From: Jan Kiszka <jan.kiszka@domain.hid>
To: Philippe Gerum <rpm@xenomai.org>
Cc: xenomai@xenomai.org
Subject: [Xenomai-core] [PATCH v3 0/9] heap setup/cleanup fixes, refactorings & more
Date: Tue, 20 Oct 2009 13:37:25 +0200 [thread overview]
Message-ID: <20091020113724.9069.23594.stgit@domain.hid> (raw)
Third round of this series. This time I dug deeper into the heap
management, trying to simplify its use which should remove a few race
conditions of the existing code.
The central change is patch 5: xnheap_destroy_mapped will no longer
return an error, it will always start deferred deletion in case the
heap's memory is still in use. This simplifies the use a lot, allowing
among other things to drop -EBUSY from the list of possible return codes
of rt_queue/heap_delete.
The series furthermore contains an attempt to fix RTAI's shm code, fixes
the know leakages of rt_mutex/queue/heap auto-deletion and introduces
complete heap statistics (this time without using a Linux spin lock).
Please pull the series (or cherry-pick individual patches) from
git://xenomai.org/xenomai-jki.git for-upstream
if there are no concerns.
Jan Kiszka (9):
native: Release fastlock to the proper heap
nucleus: Use Linux spin lock for heap list management
nucleus: Fix race window in heap mapping procedure
nucleus: xnheap_destroy does not fail
nucleus: Avoid returning errors from xnheap_destroy_mapped
rtai: Try to fix _shm_free
native: Do not requeue on auto-cleanup errors
native: Fix memory leak on heap/queue auto-deletion
nucleus: Include all heaps in statistics
include/asm-generic/bits/heap.h | 2 +-
include/asm-generic/system.h | 2 +-
include/native/ppd.h | 16 +--
include/nucleus/heap.h | 32 +++--
ksrc/drivers/ipc/iddp.c | 3 +-
ksrc/drivers/ipc/xddp.c | 6 +-
ksrc/nucleus/heap.c | 258 +++++++++++++++++++++++++++------------
ksrc/nucleus/module.c | 2 +-
ksrc/nucleus/pod.c | 5 +-
ksrc/nucleus/shadow.c | 5 +-
ksrc/skins/native/heap.c | 41 +++---
ksrc/skins/native/mutex.c | 14 ++-
ksrc/skins/native/pipe.c | 4 +-
ksrc/skins/native/queue.c | 34 +++---
ksrc/skins/native/syscall.c | 25 +---
ksrc/skins/posix/shm.c | 4 +-
ksrc/skins/psos+/rn.c | 6 +-
ksrc/skins/rtai/shm.c | 47 ++++---
ksrc/skins/vrtx/heap.c | 6 +-
ksrc/skins/vrtx/syscall.c | 3 +-
20 files changed, 317 insertions(+), 198 deletions(-)
[1] http://thread.gmane.org/gmane.linux.real-time.xenomai.devel/6559
next reply other threads:[~2009-10-20 11:37 UTC|newest]
Thread overview: 33+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-10-20 11:37 Jan Kiszka [this message]
2009-10-20 11:37 ` [Xenomai-core] [PATCH v3 3/9] nucleus: Fix race window in heap mapping procedure Jan Kiszka
2009-10-20 11:37 ` [Xenomai-core] [PATCH v3 4/9] nucleus: xnheap_destroy does not fail Jan Kiszka
2009-10-20 11:37 ` [Xenomai-core] [PATCH v3 2/9] nucleus: Use Linux spin lock for heap list management Jan Kiszka
2009-10-20 11:37 ` [Xenomai-core] [PATCH v3 1/9] native: Release fastlock to the proper heap Jan Kiszka
2009-10-20 11:37 ` [Xenomai-core] [PATCH v3 5/9] nucleus: Avoid returning errors from xnheap_destroy_mapped Jan Kiszka
2009-10-24 17:22 ` Philippe Gerum
2009-11-02 16:04 ` Philippe Gerum
2009-11-02 16:41 ` Jan Kiszka
2009-11-02 16:51 ` Philippe Gerum
2009-11-02 16:57 ` Jan Kiszka
2009-11-02 18:01 ` Jan Kiszka
2009-11-02 18:19 ` [Xenomai-help] Xenomai on ARMadeus Pierre Ficheux
2009-11-02 18:22 ` Gilles Chanteperdrix
2009-11-02 18:38 ` Gilles Chanteperdrix
2009-11-02 19:19 ` gwenhael.goavec
2009-11-02 22:29 ` Gilles Chanteperdrix
2009-11-03 7:36 ` gwenhael.goavec
[not found] ` <20091103082204.248eed59@domain.hid>
2009-11-04 13:14 ` Gilles Chanteperdrix
[not found] ` <fbc4f538a6f4d84cfe514aba0985a525.squirrel@domain.hid>
2009-11-12 14:59 ` Gilles Chanteperdrix
2009-11-02 18:26 ` [Xenomai-core] [PATCH v3 5/9] nucleus: Avoid returning errors from xnheap_destroy_mapped Philippe Gerum
2009-11-03 8:26 ` Jan Kiszka
2009-10-20 11:37 ` [Xenomai-core] [PATCH v3 6/9] rtai: Try to fix _shm_free Jan Kiszka
2009-10-24 17:25 ` Philippe Gerum
2009-10-20 11:37 ` [Xenomai-core] [PATCH v3 7/9] native: Do not requeue on auto-cleanup errors Jan Kiszka
2009-10-20 11:37 ` [Xenomai-core] [PATCH v3 9/9] nucleus: Include all heaps in statistics Jan Kiszka
2009-10-20 23:41 ` Philippe Gerum
2009-10-22 10:52 ` Jan Kiszka
2009-11-11 12:59 ` Jan Kiszka
2009-11-15 17:38 ` Philippe Gerum
2009-11-16 12:38 ` Jan Kiszka
2009-10-20 11:37 ` [Xenomai-core] [PATCH v3 8/9] native: Fix memory leak on heap/queue auto-deletion Jan Kiszka
2009-10-22 10:30 ` [Xenomai-core] [PATCH] native: Avoid double release on queue/heap auto-cleanup Jan Kiszka
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20091020113724.9069.23594.stgit@domain.hid \
--to=jan.kiszka@domain.hid \
--cc=rpm@xenomai.org \
--cc=xenomai@xenomai.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.