Re: [Xenomai-core] Potential problem with rt_eepro100

2010-11-04 Thread Gilles Chanteperdrix
Gilles Chanteperdrix wrote: > Jan Kiszka wrote: >> Am 05.11.2010 00:25, Gilles Chanteperdrix wrote: >>> Jan Kiszka wrote: Am 04.11.2010 23:08, Gilles Chanteperdrix wrote: > Jan Kiszka wrote: >> rework. Safer for now is likely to revert 56ff4329ff, keeping nucleus >> debugging off.

Re: [Xenomai-core] Potential problem with rt_eepro100

2010-11-04 Thread Gilles Chanteperdrix
Jan Kiszka wrote: > Am 05.11.2010 00:24, Gilles Chanteperdrix wrote: >> Jan Kiszka wrote: >>> Am 04.11.2010 23:06, Gilles Chanteperdrix wrote: Jan Kiszka wrote: >>> At first sight, here you are more breaking things than cleaning them. >> Still, it has the SMP record for my test program

Re: [Xenomai-core] Potential problem with rt_eepro100

2010-11-04 Thread Gilles Chanteperdrix
Jan Kiszka wrote: > Am 05.11.2010 00:46, Gilles Chanteperdrix wrote: >> Jan Kiszka wrote: >>> Am 05.11.2010 00:25, Gilles Chanteperdrix wrote: Jan Kiszka wrote: > Am 04.11.2010 23:08, Gilles Chanteperdrix wrote: >> Jan Kiszka wrote: >>> rework. Safer for now is likely to revert 56f

Re: [Xenomai-core] Potential problem with rt_eepro100

2010-11-04 Thread Jan Kiszka
Am 05.11.2010 00:46, Gilles Chanteperdrix wrote: > Jan Kiszka wrote: >> Am 05.11.2010 00:25, Gilles Chanteperdrix wrote: >>> Jan Kiszka wrote: Am 04.11.2010 23:08, Gilles Chanteperdrix wrote: > Jan Kiszka wrote: >> rework. Safer for now is likely to revert 56ff4329ff, keeping nucleus >

Re: [Xenomai-core] Potential problem with rt_eepro100

2010-11-04 Thread Gilles Chanteperdrix
Jan Kiszka wrote: > Am 05.11.2010 00:25, Gilles Chanteperdrix wrote: >> Jan Kiszka wrote: >>> Am 04.11.2010 23:08, Gilles Chanteperdrix wrote: Jan Kiszka wrote: > rework. Safer for now is likely to revert 56ff4329ff, keeping nucleus > debugging off. That is not enough. >>> It is,

Re: [Xenomai-core] Potential problem with rt_eepro100

2010-11-04 Thread Jan Kiszka
Am 05.11.2010 00:24, Gilles Chanteperdrix wrote: > Jan Kiszka wrote: >> Am 04.11.2010 23:06, Gilles Chanteperdrix wrote: >>> Jan Kiszka wrote: >> At first sight, here you are more breaking things than cleaning them. > Still, it has the SMP record for my test program, still runs with ftrace

Re: [Xenomai-core] Potential problem with rt_eepro100

2010-11-04 Thread Jan Kiszka
Am 05.11.2010 00:25, Gilles Chanteperdrix wrote: > Jan Kiszka wrote: >> Am 04.11.2010 23:08, Gilles Chanteperdrix wrote: >>> Jan Kiszka wrote: rework. Safer for now is likely to revert 56ff4329ff, keeping nucleus debugging off. >>> That is not enough. >> >> It is, I've reviewed the code t

Re: [Xenomai-core] Potential problem with rt_eepro100

2010-11-04 Thread Gilles Chanteperdrix
Jan Kiszka wrote: > Am 04.11.2010 23:08, Gilles Chanteperdrix wrote: >> Jan Kiszka wrote: >>> rework. Safer for now is likely to revert 56ff4329ff, keeping nucleus >>> debugging off. >> That is not enough. > > It is, I've reviewed the code today. The fallouts I am talking about are: 47dac49c71e89

Re: [Xenomai-core] Potential problem with rt_eepro100

2010-11-04 Thread Gilles Chanteperdrix
Jan Kiszka wrote: > Am 04.11.2010 23:06, Gilles Chanteperdrix wrote: >> Jan Kiszka wrote: > At first sight, here you are more breaking things than cleaning them. Still, it has the SMP record for my test program, still runs with ftrace on (after 2 hours, where it previously failed aft

Re: [Xenomai-core] Potential problem with rt_eepro100

2010-11-04 Thread Jan Kiszka
Am 04.11.2010 23:06, Gilles Chanteperdrix wrote: > Jan Kiszka wrote: At first sight, here you are more breaking things than cleaning them. >>> Still, it has the SMP record for my test program, still runs with ftrace >>> on (after 2 hours, where it previously failed after maximum 23 minutes).

Re: [Xenomai-core] Potential problem with rt_eepro100

2010-11-04 Thread Jan Kiszka
Am 04.11.2010 23:08, Gilles Chanteperdrix wrote: > Jan Kiszka wrote: >> rework. Safer for now is likely to revert 56ff4329ff, keeping nucleus >> debugging off. > > That is not enough. It is, I've reviewed the code today. > This commit was followed by several others to "fix > the fix". You know h

Re: [Xenomai-core] Potential problem with rt_eepro100

2010-11-04 Thread Gilles Chanteperdrix
Jan Kiszka wrote: > rework. Safer for now is likely to revert 56ff4329ff, keeping nucleus > debugging off. That is not enough. This commit was followed by several others to "fix the fix". You know how things are, someone proposes a fix, which fixes things for him, but it breaks in the other people

Re: [Xenomai-core] Potential problem with rt_eepro100

2010-11-04 Thread Gilles Chanteperdrix
Jan Kiszka wrote: >>> At first sight, here you are more breaking things than cleaning them. >> Still, it has the SMP record for my test program, still runs with ftrace >> on (after 2 hours, where it previously failed after maximum 23 minutes). > > My version was indeed still buggy, I'm reworking

Re: [Xenomai-core] Potential problem with rt_eepro100

2010-11-04 Thread Jan Kiszka
Am 04.11.2010 15:53, Anders Blomdell wrote: > Jan Kiszka wrote: >> Am 04.11.2010 14:18, Anders Blomdell wrote: >>> Gilles Chanteperdrix wrote: Jan Kiszka wrote: > Am 04.11.2010 10:26, Jan Kiszka wrote: >> Am 04.11.2010 10:16, Gilles Chanteperdrix wrote: >>> Jan Kiszka wrote: >>

Re: [Xenomai-core] Potential problem with rt_eepro100

2010-11-04 Thread Anders Blomdell
Jan Kiszka wrote: Am 04.11.2010 14:18, Anders Blomdell wrote: Gilles Chanteperdrix wrote: Jan Kiszka wrote: Am 04.11.2010 10:26, Jan Kiszka wrote: Am 04.11.2010 10:16, Gilles Chanteperdrix wrote: Jan Kiszka wrote: Take a step back and look at the root cause for this issue again. Unlocked

Re: [Xenomai-core] Potential problem with rt_eepro100

2010-11-04 Thread Jan Kiszka
Am 04.11.2010 14:18, Anders Blomdell wrote: > Gilles Chanteperdrix wrote: >> Jan Kiszka wrote: >>> Am 04.11.2010 10:26, Jan Kiszka wrote: Am 04.11.2010 10:16, Gilles Chanteperdrix wrote: > Jan Kiszka wrote: >> Take a step back and look at the root cause for this issue again. >> Un

Re: [Xenomai-core] Potential problem with rt_eepro100

2010-11-04 Thread Anders Blomdell
Gilles Chanteperdrix wrote: Jan Kiszka wrote: Am 04.11.2010 10:26, Jan Kiszka wrote: Am 04.11.2010 10:16, Gilles Chanteperdrix wrote: Jan Kiszka wrote: Take a step back and look at the root cause for this issue again. Unlocked if need-resched __xnpod_schedule is inhe

Re: [Xenomai-core] Potential problem with rt_eepro100

2010-11-04 Thread Gilles Chanteperdrix
Jan Kiszka wrote: > Am 04.11.2010 10:26, Jan Kiszka wrote: >> Am 04.11.2010 10:16, Gilles Chanteperdrix wrote: >>> Jan Kiszka wrote: Take a step back and look at the root cause for this issue again. Unlocked if need-resched __xnpod_schedule is inherently

Re: [Xenomai-core] Potential problem with rt_eepro100

2010-11-04 Thread Anders Blomdell
Jan Kiszka wrote: Am 04.11.2010 10:26, Jan Kiszka wrote: Am 04.11.2010 10:16, Gilles Chanteperdrix wrote: Jan Kiszka wrote: Take a step back and look at the root cause for this issue again. Unlocked if need-resched __xnpod_schedule is inherently racy and will always b

Re: [Xenomai-core] Potential problem with rt_eepro100

2010-11-04 Thread Jan Kiszka
Am 04.11.2010 10:26, Jan Kiszka wrote: > Am 04.11.2010 10:16, Gilles Chanteperdrix wrote: >> Jan Kiszka wrote: >>> Take a step back and look at the root cause for this issue again. Unlocked >>> >>> if need-resched >>> __xnpod_schedule >>> >>> is inherently racy and will always be (n

Re: [Xenomai-core] Potential problem with rt_eepro100

2010-11-04 Thread Jan Kiszka
Am 04.11.2010 10:16, Gilles Chanteperdrix wrote: > Jan Kiszka wrote: >> Take a step back and look at the root cause for this issue again. Unlocked >> >> if need-resched >> __xnpod_schedule >> >> is inherently racy and will always be (not only for the remote >> reschedule case BTW)

Re: [Xenomai-core] Potential problem with rt_eepro100

2010-11-04 Thread Gilles Chanteperdrix
Gilles Chanteperdrix wrote: > Jan Kiszka wrote: >> Take a step back and look at the root cause for this issue again. Unlocked >> >> if need-resched >> __xnpod_schedule >> >> is inherently racy and will always be (not only for the remote >> reschedule case BTW). > > Ok, let us exa

Re: [Xenomai-core] Potential problem with rt_eepro100

2010-11-04 Thread Gilles Chanteperdrix
Anders Blomdell wrote: > Probably being daft here; why not stop fiddling with remote CPU status > bits and always do a reschedule on IPI irq's? That is what we had been doing for a long time, and stopped between 2.5.4 and 2.5.5, and this is what I say: maybe this was not such a good idea. --

Re: [Xenomai-core] Potential problem with rt_eepro100

2010-11-04 Thread Gilles Chanteperdrix
Jan Kiszka wrote: > Take a step back and look at the root cause for this issue again. Unlocked > > if need-resched > __xnpod_schedule > > is inherently racy and will always be (not only for the remote > reschedule case BTW). Ok, let us examine what may happen with this code i

Re: [Xenomai-core] Potential problem with rt_eepro100

2010-11-04 Thread Jan Kiszka
Am 04.11.2010 09:45, Anders Blomdell wrote: > Jan Kiszka wrote: >> Am 04.11.2010 01:13, Gilles Chanteperdrix wrote: >>> Jan Kiszka wrote: Am 04.11.2010 00:56, Gilles Chanteperdrix wrote: > Jan Kiszka wrote: >> Am 04.11.2010 00:44, Gilles Chanteperdrix wrote: >>> Jan Kiszka wrote: >

Re: [Xenomai-core] Potential problem with rt_eepro100

2010-11-04 Thread Anders Blomdell
Jan Kiszka wrote: Am 04.11.2010 01:13, Gilles Chanteperdrix wrote: Jan Kiszka wrote: Am 04.11.2010 00:56, Gilles Chanteperdrix wrote: Jan Kiszka wrote: Am 04.11.2010 00:44, Gilles Chanteperdrix wrote: Jan Kiszka wrote: Am 04.11.2010 00:18, Gilles Chanteperdrix wrote: Jan Kiszka wrote: Am

Re: [Xenomai-core] Potential problem with rt_eepro100

2010-11-04 Thread Jan Kiszka
Am 04.11.2010 01:13, Gilles Chanteperdrix wrote: > Jan Kiszka wrote: >> Am 04.11.2010 00:56, Gilles Chanteperdrix wrote: >>> Jan Kiszka wrote: Am 04.11.2010 00:44, Gilles Chanteperdrix wrote: > Jan Kiszka wrote: >> Am 04.11.2010 00:18, Gilles Chanteperdrix wrote: >>> Jan Kiszka wro