Re: [Xenomai] [PATCH v3] nucleus: Convert intrlock to a sleeping Linux lock

2013-01-06 Thread Jan Kiszka
On 2013-01-06 16:26, Gilles Chanteperdrix wrote: > On 01/06/2013 04:19 PM, Jan Kiszka wrote: > >> 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 ipi

Re: [Xenomai] [PATCH v3] nucleus: Convert intrlock to a sleeping Linux lock

2013-01-06 Thread Gilles Chanteperdrix
On 01/06/2013 04:19 PM, Jan Kiszka wrote: > 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

Re: [Xenomai] [PATCH v3] nucleus: Convert intrlock to a sleeping Linux lock

2013-01-06 Thread Jan Kiszka
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, th

Re: [Xenomai] [PATCH v3] nucleus: Convert intrlock to a sleeping Linux lock

2013-01-06 Thread Gilles Chanteperdrix
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_virtual

Re: [Xenomai] [PATCH v3] nucleus: Convert intrlock to a sleeping Linux lock

2013-01-06 Thread Jan Kiszka
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,

Re: [Xenomai] [PATCH v3] nucleus: Convert intrlock to a sleeping Linux lock

2013-01-06 Thread Gilles Chanteperdrix
On 01/06/2013 03:46 PM, Jan Kiszka wrote: > On 2013-01-06 15:26, Gilles Chanteperdrix wrote: >> On 01/06/2013 03:20 PM, Jan Kiszka wrote: >> >>> On 2013-01-06 14:55, Gilles Chanteperdrix wrote: On 01/06/2013 12:03 PM, Jan Kiszka wrote: > On 2013-01-06 11:55, Jan Kiszka wrote: >>

Re: [Xenomai] [PATCH v3] nucleus: Convert intrlock to a sleeping Linux lock

2013-01-06 Thread Jan Kiszka
On 2013-01-06 15:26, Gilles Chanteperdrix wrote: > On 01/06/2013 03:20 PM, Jan Kiszka wrote: > >> On 2013-01-06 14:55, Gilles Chanteperdrix wrote: >>> On 01/06/2013 12:03 PM, Jan Kiszka wrote: >>> On 2013-01-06 11:55, Jan Kiszka wrote: > On 2013-01-06 11:14, Philippe Gerum wrote: >> O

Re: [Xenomai] [PATCH v3] nucleus: Convert intrlock to a sleeping Linux lock

2013-01-06 Thread Gilles Chanteperdrix
On 01/06/2013 03:26 PM, Gilles Chanteperdrix wrote: > On 01/06/2013 03:20 PM, Jan Kiszka wrote: > >> On 2013-01-06 14:55, Gilles Chanteperdrix wrote: >>> On 01/06/2013 12:03 PM, Jan Kiszka wrote: >>> On 2013-01-06 11:55, Jan Kiszka wrote: > On 2013-01-06 11:14, Philippe Gerum wrote:

Re: [Xenomai] [PATCH v3] nucleus: Convert intrlock to a sleeping Linux lock

2013-01-06 Thread Gilles Chanteperdrix
On 01/06/2013 03:20 PM, Jan Kiszka wrote: > On 2013-01-06 14:55, Gilles Chanteperdrix wrote: >> On 01/06/2013 12:03 PM, Jan Kiszka wrote: >> >>> On 2013-01-06 11:55, Jan Kiszka wrote: On 2013-01-06 11:14, Philippe Gerum wrote: > On 01/03/2013 03:09 PM, Jan Kiszka wrote: > >> +stat

Re: [Xenomai] [PATCH v3] nucleus: Convert intrlock to a sleeping Linux lock

2013-01-06 Thread Jan Kiszka
On 2013-01-06 14:55, Gilles Chanteperdrix wrote: > On 01/06/2013 12:03 PM, Jan Kiszka wrote: > >> On 2013-01-06 11:55, Jan Kiszka wrote: >>> On 2013-01-06 11:14, Philippe Gerum wrote: On 01/03/2013 03:09 PM, Jan Kiszka wrote: > +static struct xnvfile_lock_ops vfile_stat_lockops = { >

Re: [Xenomai] [PATCH v3] nucleus: Convert intrlock to a sleeping Linux lock

2013-01-06 Thread Gilles Chanteperdrix
On 01/06/2013 12:03 PM, Jan Kiszka wrote: > On 2013-01-06 11:55, Jan Kiszka wrote: >> On 2013-01-06 11:14, Philippe Gerum wrote: >>> On 01/03/2013 03:09 PM, Jan Kiszka wrote: >>> +static struct xnvfile_lock_ops vfile_stat_lockops = { + .get = xnintr_get_query_lock, + .put = xnintr

Re: [Xenomai] [PATCH v3] nucleus: Convert intrlock to a sleeping Linux lock

2013-01-06 Thread Jan Kiszka
On 2013-01-06 11:55, Jan Kiszka wrote: > On 2013-01-06 11:14, Philippe Gerum wrote: >> On 01/03/2013 03:09 PM, Jan Kiszka wrote: >> >>> +static struct xnvfile_lock_ops vfile_stat_lockops = { >>> + .get = xnintr_get_query_lock, >>> + .put = xnintr_put_query_lock, >>> +}; >>> + >>> static struct

Re: [Xenomai] [PATCH v3] nucleus: Convert intrlock to a sleeping Linux lock

2013-01-06 Thread Jan Kiszka
On 2013-01-06 11:14, Philippe Gerum wrote: > On 01/03/2013 03:09 PM, Jan Kiszka wrote: > >> +static struct xnvfile_lock_ops vfile_stat_lockops = { >> +.get = xnintr_get_query_lock, >> +.put = xnintr_put_query_lock, >> +}; >> + >> static struct xnvfile_snapshot stat_vfile = { >> .priv

Re: [Xenomai] [PATCH v3] nucleus: Convert intrlock to a sleeping Linux lock

2013-01-06 Thread Philippe Gerum
On 01/03/2013 03:09 PM, Jan Kiszka wrote: > +static struct xnvfile_lock_ops vfile_stat_lockops = { > + .get = xnintr_get_query_lock, > + .put = xnintr_put_query_lock, > +}; > + > static struct xnvfile_snapshot stat_vfile = { > .privsz = sizeof(struct vfile_stat_priv), > .datas

[Xenomai] [PATCH v3] nucleus: Convert intrlock to a sleeping Linux lock

2013-01-03 Thread Jan Kiszka
Conceptually, all users of this lock are supposed to run over Linux context already. Moreover, latest I-pipe patches complain that ipipe_virtualize_irq is called with the Xenomai domain stalled which invalidates that useful internal bug check. So rework the locking for dumping scheduling statistic