All of lore.kernel.org
 help / color / mirror / Atom feed
From: Gilles Chanteperdrix <gilles.chanteperdrix@xenomai.org>
To: Jan Kiszka <jan.kiszka@domain.hid>
Cc: xenomai-core <xenomai@xenomai.org>
Subject: Re: [Xenomai-core] [PATCH 0/6] Registry-related cleanups and	improvements
Date: Tue, 26 Aug 2008 15:40:13 +0200	[thread overview]
Message-ID: <48B407BD.2070204@domain.hid> (raw)
In-Reply-To: <48B3ED52.7080101@domain.hid>

Jan Kiszka wrote:
> Here comes the complete series of patches to improve the Xenomai
> registry. #1, #5, and #6 are resends of already posted patches, the
> middle part is fresh from the editor. The series reduces overhead
> and simplifies the registry code. Overall diffstat:
> 
>  include/asm-generic/hal.h  |   10 +-
>  include/nucleus/registry.h |   10 --
>  ksrc/arch/generic/hal.c    |   18 +--
>  ksrc/arch/generic/nmi.c    |    6 -
>  ksrc/arch/x86/nmi_32.c     |    6 -
>  ksrc/nucleus/module.c      |   58 ++++-------
>  ksrc/nucleus/registry.c    |  223 ++++++++++++++++++++++-----------------------
>  ksrc/skins/native/alarm.c  |   22 +---
>  ksrc/skins/native/buffer.c |   15 ---
>  ksrc/skins/native/cond.c   |   14 --
>  ksrc/skins/native/event.c  |   15 ---
>  ksrc/skins/native/heap.c   |   14 --
>  ksrc/skins/native/intr.c   |   17 ---
>  ksrc/skins/native/mutex.c  |   15 ---
>  ksrc/skins/native/pipe.c   |   14 --
>  ksrc/skins/native/queue.c  |   12 --
>  ksrc/skins/native/sem.c    |   14 --
>  ksrc/skins/native/task.c   |   16 ---
>  18 files changed, 182 insertions(+), 317 deletions(-)
> 
> So a net gain of more than 130 LOC.
> 
> The series passed basic tests on my virtual target.

Ok. Now that we have discussed the potential xnsynch issue, the patches
are Ok for me.

-- 
                                                 Gilles.


  parent reply	other threads:[~2008-08-26 13:40 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-08-26 12:08 [Xenomai-core] [PATCH 0/6] Registry-related cleanups and improvements Jan Kiszka
2008-08-26 11:49 ` [Xenomai-core] [PATCH 1/6] Remove / shorten critical sections in registry code Jan Kiszka
2008-08-26 11:51 ` [Xenomai-core] [PATCH 2/6] Allow key-less registry entries Jan Kiszka
2008-08-26 11:53 ` [Xenomai-core] [PATCH 3/6] Native skin: Use " Jan Kiszka
2008-08-26 11:55 ` [Xenomai-core] [PATCH 4/6] Eliminate xnobjhash Jan Kiszka
2008-08-26 11:56 ` [Xenomai-core] [RESEND][PATCH 5/6] Avoid add_proc_leaf duplicates Jan Kiszka
2008-08-26 11:58 ` [Xenomai-core] [RESEND][PATCH 6/6] Report registry slot usage via /proc Jan Kiszka
2008-08-26 16:08   ` Philippe Gerum
2008-08-26 16:19     ` Jan Kiszka
2008-08-26 16:22       ` Philippe Gerum
2008-08-26 17:08         ` Jan Kiszka
2008-08-26 13:40 ` Gilles Chanteperdrix [this message]
2008-08-26 16:03   ` [Xenomai-core] [PATCH 0/6] Registry-related cleanups and improvements Philippe Gerum
2008-08-27  9:35 ` Philippe Gerum

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=48B407BD.2070204@domain.hid \
    --to=gilles.chanteperdrix@xenomai.org \
    --cc=jan.kiszka@domain.hid \
    --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.