From: Philippe Gerum <rpm@xenomai.org>
To: xenomai@xenomai.org
Subject: Re: [Xenomai-help] Interrupt numbers
Date: Fri, 23 Dec 2011 19:13:20 +0100 [thread overview]
Message-ID: <4EF4C4C0.9010100@domain.hid> (raw)
In-Reply-To: <E9F947B915764962864D7E7291CA601F@TerryHP>
On 12/23/2011 05:16 PM, Terry Fryar wrote:
> I would imagine it was nice, however, to have a userspace interrupt ISR so
> that flaky code could be debugged in userspace before making it into a
> driver?
>
Most interrupts are level sensitive these days, which means that you
cannot safely step into the code which is supposed to ack the source
device using a debugger, your kernel would be stormed by IRQs before you
reach the point where the device request is acked. Remember that
userland always runs with hw interrupts enabled, regardless of the
domain. Even edge triggered IRQs would not give you any guarantee with
respect to device timing requirements.
GDB aside, also think about a transition to secondary mode for whatever
reason while running in userland prior to acking the device: this would
be another source of unexpected delays in the acknowledge path.
Debugging work is likely to introduce these issues, unless one refrains
from using anything else than rt_printf() for logging/observing the
runtime state, but that would not help with level sensitive IRQs anyway.
You may want to handle the main application logic that responds to an
interrupt in userland through, in which case you need some RTDM driver
handling the bottom half of real-time interrupts, which would in turn
unblock a task sleeping on some read() or ioctl(), to process the event
in userland (i.e. UIO-like for real-time IRQs).
The bottom line is that you want the IRQ to be acknowledged at device
level from kernel space. Keeping it masked in the PIC while
transitioning to user-space would be another option, assuming it is not
shared with the regular kernel (sharing between rt and non-rt would be
just wrong anyway), if the device permits (infinitely) delayed
acknowledges, but I would not recommend this. Typically, a user-space
code can be wiped off at any time, leaving the device in a weird state.
These are the reasons why I have killed the rt_intr_* API in 3.x, it was
way too easy to shoot oneself in the foot (and believe me, I saw quite a
few damaged feet in the past years due to this issue). What was missing
in this API is a clear hint that some user-provided code should live in
kernel space to ack each particular device controlled from userland.
Using RTDM to implement such code and synchronize with the application
logic in userland is a safe, sane and simple solution.
> -----Original Message-----
> From: xenomai-help-bounces@domain.hid [mailto:xenomai-help-bounces@domain.hid] On
> Behalf Of Gilles Chanteperdrix
> Sent: Friday, December 23, 2011 9:53 AM
> To: Makarand Pradhan
> Cc: xenomai@xenomai.org
> Subject: Re: [Xenomai-help] Interrupt numbers
>
> On 12/23/2011 04:26 PM, Makarand Pradhan wrote:
>> On 23/12/11 04:45 AM, Gilles Chanteperdrix wrote:
>>>
>>> Xenomai uses the same interrupt numbers as linux. But rt_intr_create
>>> is deprecated in user-space, you should instead write a driver using
>>> the rtdm skin. The enable bit is handled by xenomai when you request
>>> the irq at xenomai level.
>>>
>> Hi Gilles,
>>
>> We use rt_intr_create in our code. So I am trying to understand the
>> reasons for it being deprecated. So far, I have not been able to see
>> any comments in the code regarding the deprecation or anything in the git
> log.
>
> Splitting your code between driver and application enforces a clean
> separation between the two, which helps maintenance, so is good on the long
> run.
>
>>
>> Can you pl comment on the reasons for deprecating rt_intr_create? Will
>> it be removed in the next release?
>
> We never change ABI in a branch, so, all releases in the 2.6 branch are
> guaranteed to support the same services as xenomai 2.6.0.
>
> But in xenomai 3.0, rt_intr_create will certainly no longer be available.
--
Philippe.
next prev parent reply other threads:[~2011-12-23 18:13 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-12-23 0:38 [Xenomai-help] Interrupt numbers Terry Fryar
2011-12-23 9:45 ` Gilles Chanteperdrix
2011-12-23 14:04 ` Terry Fryar
2011-12-23 15:26 ` Makarand Pradhan
2011-12-23 15:53 ` Gilles Chanteperdrix
2011-12-23 16:16 ` Terry Fryar
2011-12-23 18:13 ` Philippe Gerum [this message]
2011-12-23 23:06 ` Terry Fryar
2012-01-19 19:41 ` Michael Pustylnik
2012-01-19 22:59 ` Philippe Gerum
2011-12-23 16:17 ` Makarand Pradhan
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=4EF4C4C0.9010100@domain.hid \
--to=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.