All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Lennart Sorensen" <lsorense@csclub.uwaterloo.ca>
To: Philippe Gerum <rpm@xenomai.org>
Cc: xenomai@xenomai.org
Subject: Re: [Xenomai] I-pipe error when requesting irq that is on omap gpio
Date: Mon, 17 Nov 2014 14:12:37 -0500	[thread overview]
Message-ID: <20141117191237.GS24110@csclub.uwaterloo.ca> (raw)
In-Reply-To: <545D12D2.2020104@xenomai.org>

On Fri, Nov 07, 2014 at 07:43:30PM +0100, Philippe Gerum wrote:
> For this particular issue, Xenomai 2.x is deadly wrong. Creating,
> deleting, enabling or disabling an IRQ channel must be done from
> secondary mode. The syscall mode bits for these native API calls are
> wrong, should be as follows instead:
> 
> diff --git a/ksrc/skins/native/syscall.c b/ksrc/skins/native/syscall.c
> index 950b092..dff7f4d 100644
> --- a/ksrc/skins/native/syscall.c
> +++ b/ksrc/skins/native/syscall.c
> @@ -4145,12 +4145,12 @@ static xnsysent_t __systab[] = {
>  	[__native_alarm_stop] = {&__rt_alarm_stop, __xn_exec_any},
>  	[__native_alarm_wait] = {&__rt_alarm_wait, __xn_exec_primary},
>  	[__native_alarm_inquire] = {&__rt_alarm_inquire, __xn_exec_any},
> -	[__native_intr_create] = {&__rt_intr_create, __xn_exec_any},
> +	[__native_intr_create] = {&__rt_intr_create, __xn_exec_lostage},
>  	[__native_intr_bind] = {&__rt_intr_bind, __xn_exec_conforming},
> -	[__native_intr_delete] = {&__rt_intr_delete, __xn_exec_any},
> +	[__native_intr_delete] = {&__rt_intr_delete, __xn_exec_lostage},
>  	[__native_intr_wait] = {&__rt_intr_wait, __xn_exec_primary},
> -	[__native_intr_enable] = {&__rt_intr_enable, __xn_exec_any},
> -	[__native_intr_disable] = {&__rt_intr_disable, __xn_exec_any},
> +	[__native_intr_enable] = {&__rt_intr_enable, __xn_exec_lostage},
> +	[__native_intr_disable] = {&__rt_intr_disable, __xn_exec_lostage},
>  	[__native_intr_inquire] = {&__rt_intr_inquire, __xn_exec_any},
>  	[__native_pipe_create] = {&__rt_pipe_create, __xn_exec_lostage},
>  	[__native_pipe_bind] = {&__rt_pipe_bind, __xn_exec_conforming},

Are you sure all 4 of those had to change?  I can see intr_create and
intr_delete, but intr_enable and disable?  We are seeing insane latency
from realtime irqs on the omap gpio irqs (a few ms response to an irq).
Of course it could be the gpio irq is just doing something wrong.
Not sure yet.  Of course before doing this change we sometimes saw it
take close to 1s to call the realtime irq handler which made no sense
at all.  gic interrupts are working as expected and see no delays that
we are aware of at least.

-- 
Len Sorensen


  parent reply	other threads:[~2014-11-17 19:12 UTC|newest]

Thread overview: 65+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-10-01 14:24 [Xenomai] I-pipe error when requesting irq that is on omap gpio Lennart Sorensen
2014-10-01 14:57 ` Gilles Chanteperdrix
2014-10-01 15:02   ` Lennart Sorensen
2014-10-01 15:28     ` Gilles Chanteperdrix
2014-10-01 15:46       ` Lennart Sorensen
2014-10-01 15:52         ` Lennart Sorensen
2014-10-01 16:02           ` Lennart Sorensen
2014-10-01 17:31             ` Gilles Chanteperdrix
2014-10-01 17:40               ` Lennart Sorensen
2014-10-01 17:46                 ` Gilles Chanteperdrix
2014-10-01 17:54                   ` Lennart Sorensen
2014-10-01 20:33                     ` Lennart Sorensen
2014-10-01 21:39                     ` Gilles Chanteperdrix
2014-10-01 21:43                       ` Lennart Sorensen
2014-10-02 13:56                         ` [Xenomai] Xenomai on Cortex R (or Rreal Time) family mobin Motallebizadeh
2014-10-02 14:00                           ` Gilles Chanteperdrix
2014-10-02 14:22                             ` mobin Motallebizadeh
2014-10-02 14:26                               ` Lennart Sorensen
2014-10-02 19:34                 ` [Xenomai] I-pipe error when requesting irq that is on omap gpio Lennart Sorensen
2014-11-07 18:26 ` Lennart Sorensen
2014-11-07 18:43   ` Philippe Gerum
2014-11-07 20:21     ` Lennart Sorensen
2014-11-07 20:24     ` Lennart Sorensen
2014-11-07 20:32       ` Gilles Chanteperdrix
2014-11-07 21:26         ` Lennart Sorensen
2014-11-17 19:12     ` Lennart Sorensen [this message]
2014-11-17 19:44       ` Philippe Gerum
2014-11-17 20:11         ` Lennart Sorensen
2014-11-17 20:14       ` Gilles Chanteperdrix
2014-11-17 20:24         ` Lennart Sorensen
2014-11-17 20:27           ` Gilles Chanteperdrix
2014-11-17 21:47             ` Lennart Sorensen
2014-11-17 21:54               ` Gilles Chanteperdrix
2014-11-17 22:10                 ` Lennart Sorensen
2014-11-17 22:15                   ` Gilles Chanteperdrix
2014-11-18 15:24                     ` Lennart Sorensen
2014-11-18 15:26                       ` Gilles Chanteperdrix
2014-11-18 18:48                         ` Lennart Sorensen
2014-11-18 18:51                           ` Gilles Chanteperdrix
2014-11-18 18:56                             ` Lennart Sorensen
2014-11-18 19:00                               ` Gilles Chanteperdrix
2014-11-18 19:24                                 ` Lennart Sorensen
2014-11-18 19:28                                   ` Gilles Chanteperdrix
2014-11-18 20:28                                     ` Gilles Chanteperdrix
2014-11-18 20:53                                       ` Lennart Sorensen
2014-11-18 20:56                                         ` Gilles Chanteperdrix
2014-11-18 21:17                                           ` Gilles Chanteperdrix
2014-11-18 21:39                                             ` Lennart Sorensen
2014-11-18 22:05                                               ` Gilles Chanteperdrix
2014-11-18 22:31                                                 ` Lennart Sorensen
2014-11-19  5:40                                                   ` Gilles Chanteperdrix
2014-11-19 17:12                                               ` Lennart Sorensen
2014-11-19 22:38                                                 ` Gilles Chanteperdrix
2014-11-20 20:01                                                   ` Lennart Sorensen
2014-11-20 20:06                                                     ` Gilles Chanteperdrix
2014-11-20 22:38                                                       ` Lennart Sorensen
2014-11-20 22:41                                                         ` Gilles Chanteperdrix
2014-11-21  7:59                                                         ` Gilles Chanteperdrix
2014-11-21 15:11                                                           ` Lennart Sorensen
2014-11-18 19:14                           ` Gilles Chanteperdrix
2014-11-18 19:25                             ` Lennart Sorensen
2014-11-18 19:46                               ` Gilles Chanteperdrix
2014-11-18 21:48                                 ` Lennart Sorensen
2014-11-17 20:30           ` Gilles Chanteperdrix
2014-11-17 20:43             ` Lennart Sorensen

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=20141117191237.GS24110@csclub.uwaterloo.ca \
    --to=lsorense@csclub.uwaterloo.ca \
    --cc=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.