From: Philippe Gerum <rpm@xenomai.org>
To: Jan Kiszka <jan.kiszka@domain.hid>
Cc: xenomai-core <xenomai@xenomai.org>
Subject: Re: [Xenomai-core] Pending patches
Date: Thu, 15 Jan 2009 13:09:02 +0100 [thread overview]
Message-ID: <496F275E.90407@domain.hid> (raw)
In-Reply-To: <496F1CF2.1030401@domain.hid>
Jan Kiszka wrote:
> Philippe Gerum wrote:
>> Jan Kiszka wrote:
>>> commit 728fc8970e2032b3280971788f1223f3ad82d80d
>>> Author: Jan Kiszka <jan.kiszka@domain.hid>
>>> Date: Thu Jan 15 11:10:24 2009 +0100
>>>
>>> xnpipe: Fix racy callback handlers
>>>
>>> Invocation of input, output and alloc handler must take place under
>>> nklock to properly synchronize with xnpipe_disconnect. Change all
>>> callers to comply with this policy.
>>>
>> That one is under investigation. I agree on the bug report (thanks btw), but I
>> disagree on the fix. Basically, we can't run all hooks under nklock. For
>> instance, the alloc_handler may issue kmalloc() calls when issued from the Linux
>> write endpoint.
>
> You mean it /could/? Because no in-tree user (ie. native) calls
> rt-unsafe services from its alloc_handler.
>
When you export a public interface, it is better not to make it incompatible
unless there is no other way to fix a situation. Doing so is last resort for me.
> Jan
>
--
Philippe.
next prev parent reply other threads:[~2009-01-15 12:09 UTC|newest]
Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-01-15 10:19 [Xenomai-core] Pending patches Jan Kiszka
2009-01-15 10:30 ` Gilles Chanteperdrix
2009-01-15 11:00 ` Gilles Chanteperdrix
2009-01-15 11:31 ` Jan Kiszka
2009-01-15 13:02 ` Gilles Chanteperdrix
2009-01-15 14:12 ` Jan Kiszka
2009-01-15 11:28 ` Jan Kiszka
2009-01-15 13:09 ` Gilles Chanteperdrix
2009-01-15 14:17 ` Jan Kiszka
2009-01-15 21:46 ` Jan Kiszka
2009-01-15 10:49 ` Philippe Gerum
2009-01-15 11:24 ` Jan Kiszka
2009-01-15 12:09 ` Philippe Gerum [this message]
2009-01-15 13:55 ` Jan Kiszka
2009-01-15 14:16 ` Philippe Gerum
2009-01-19 23:18 ` Philippe Gerum
2009-01-19 23:35 ` Jan Kiszka
2009-01-20 8:49 ` Philippe Gerum
2009-01-20 9:06 ` Jan Kiszka
2009-01-20 9:18 ` Jan Kiszka
2009-01-20 9:22 ` Philippe Gerum
2009-01-27 13:10 ` Jan Kiszka
2009-01-27 13:26 ` Jan Kiszka
2009-01-27 13:28 ` Gilles Chanteperdrix
2009-01-27 13:29 ` Philippe Gerum
2009-01-27 13:40 ` Gilles Chanteperdrix
2009-01-27 13:40 ` 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=496F275E.90407@domain.hid \
--to=rpm@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.