All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jan Kiszka <jan.kiszka@web.de>
To: Gilles Chanteperdrix <gilles.chanteperdrix@xenomai.org>
Cc: "Mauerer, Wolfgang" <wolfgang.mauerer@siemens.com>,
	Xenomai <xenomai@xenomai.org>
Subject: Re: [Xenomai] [PATCH v3] nucleus: Convert intrlock to a sleeping Linux lock
Date: Sun, 06 Jan 2013 16:19:07 +0100	[thread overview]
Message-ID: <50E995EB.8040300@web.de> (raw)
In-Reply-To: <50E9951E.4030809@xenomai.org>

On 2013-01-06 16:15, Gilles Chanteperdrix wrote:
> On 01/06/2013 04:11 PM, Jan Kiszka wrote:
> 
>> On 2013-01-06 16:09, Gilles Chanteperdrix wrote:
>>>> If you prefer to skip the warning in ipipe, then I will send a
>>>> corresponding patch. The lock conversion is still necessary for forge,
>>>> though.
>>>
>>>
>>> There are other ways around, ipipe_virtualize_irq is already a wrapper
>>> used only by 2.6, so we can add some code here. So, we can only check
>>> ipipe_root_p here, and disable the warning in ipipe_request_irq.
>>
>> Err, we rather need unchecked __ipipe_request_irq, called by
>> ipipe_request_irq after testing the context and by ipipe_virtualize_irq
>> without that test.
> 
> 
> ipipe_root_only does two tests, it tests whether the current domain is
> root, and whether root is stalled. The problem, as far as I understood,
> come from the second test, what I mean is that we can arrange to keep
> the first test for ipipe_virtualize_irq.

Well, if we disallow non-root invocations of ipipe_virtualize_irq, then
we can also convert the lock to a Linux version. I thought your concerns
are related to restricting the usage of that services (or services that
call it) in stable 2.6.

Jan


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 261 bytes
Desc: OpenPGP digital signature
URL: <http://www.xenomai.org/pipermail/xenomai/attachments/20130106/b6b0d47e/attachment.pgp>

  reply	other threads:[~2013-01-06 15:19 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-01-02 18:07 [Xenomai] [PATCH v2] nucleus: Convert intrlock to a sleeping Linux lock Jan Kiszka
2013-01-02 20:25 ` Jeff Webb
2013-01-03 10:46   ` Jan Kiszka
2013-01-03 14:09 ` [Xenomai] [PATCH v3] " Jan Kiszka
2013-01-06 10:14   ` Philippe Gerum
2013-01-06 10:55     ` Jan Kiszka
2013-01-06 11:03       ` Jan Kiszka
2013-01-06 13:55         ` Gilles Chanteperdrix
2013-01-06 14:20           ` Jan Kiszka
2013-01-06 14:26             ` Gilles Chanteperdrix
2013-01-06 14:35               ` Gilles Chanteperdrix
2013-01-06 14:46               ` Jan Kiszka
2013-01-06 15:09                 ` Gilles Chanteperdrix
2013-01-06 15:11                   ` Jan Kiszka
2013-01-06 15:15                     ` Gilles Chanteperdrix
2013-01-06 15:19                       ` Jan Kiszka [this message]
2013-01-06 15:26                         ` Gilles Chanteperdrix
2013-01-06 15:31                           ` 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=50E995EB.8040300@web.de \
    --to=jan.kiszka@web.de \
    --cc=gilles.chanteperdrix@xenomai.org \
    --cc=wolfgang.mauerer@siemens.com \
    --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.