All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jan Kiszka <jan.kiszka@domain.hid>
To: xenomai-core <xenomai@xenomai.org>
Subject: [Xenomai-core] xnregistry_fetch & friends
Date: Mon, 25 Aug 2008 22:19:01 +0200	[thread overview]
Message-ID: <48B313B5.9050308@domain.hid> (raw)

[-- Attachment #1: Type: text/plain, Size: 849 bytes --]

Hi,

trying to select a sane kernel-side looking scheme for fast native
mutexes, I had a closer look at the registry usage in that skin (and
many others). The typical pattern is

object = xnregistry_fetch(handle);
perform_operation(object);

There is no lock around those two, both services do nklock acquisition
only internally. So this is a bit racy against concurrent object
destruction and memory releasing / object reconstruction. Well, I guess
the rational is: we test against object magics and the underlying memory
is normally not vanishing (immediately) on destruction, right? Remains
just object reconstruction. Not a real-life issue?

But then I wonder

 a) why xnregistry_fetch uses nklock at all (even for totally uncritical
    XNOBJECT_SELF!)

 b) what the ideas/plans on unused xnregistry_put/get are.

Jan


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 257 bytes --]

             reply	other threads:[~2008-08-25 20:19 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-08-25 20:19 Jan Kiszka [this message]
2008-08-25 22:11 ` [Xenomai-core] xnregistry_fetch & friends Philippe Gerum
2008-08-25 22:58   ` Jan Kiszka
2008-08-26  8:06     ` Philippe Gerum
2008-08-26  8:27       ` Jan Kiszka
2008-08-26  8:41         ` Philippe Gerum
2008-08-26  8:52           ` Jan Kiszka
2008-08-26  9:09             ` Philippe Gerum
2008-08-26 12:49             ` Gilles Chanteperdrix
2008-08-26 13:08               ` Jan Kiszka
2008-08-26 13:13                 ` Gilles Chanteperdrix
2008-08-26 13:18                   ` Gilles Chanteperdrix
2008-08-26 13:32                     ` Jan Kiszka
2008-08-26 13:38                       ` Gilles Chanteperdrix

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=48B313B5.9050308@domain.hid \
    --to=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.