Re: [Xenomai-help] Application broken 2.3.4 ---> 2.4.0

[Jan Kiszka <jan.kiszka@domain.hid>]

Ignacio García Pérez wrote:
> Ignacio García Pérez escribió:
> 
> Ok, replying myself with further findings:
> 
> It turns out that the following code miserably fails to set the thread 
> periodic:
> 
> rt_task_set_periodic(&_blink_thread, rt_timer_read(), mili2count(250));
> 
> Where this actually works:
> 
> rt_task_set_periodic(&_blink_thread, TM_NOW, mili2count(250));
> 
> Which makes me think that rt_task_set_periodic() fails when the passed 
> idate is in the past.
> 
> And you may say: why the hell do you use rt_timer_read() when you can 
> use TM_NOW ???
> 
> Well, I think that rt_task_set_periodic should word (AND DID) both ways, 
> but actually I have a very good reason to (sometimes) use an idate in 
> the past: I want the blinking synchronized to the "event" as closely as 
> possible. If there is a delay D from my recorded time for the event to 
> the actual call to rt_task_set_periodic, I can:
> 
> a) Use rt_task_set_periodic(TM_NOD)
> 
> b) Use rt_task_set_periodic(recorded_time)
> 
> In the first case there will be a delay of D from every period to the 
> "event", while in the second case that delay will be only in the FIRST 
> execution of the periodic task.
> 
> Anyway, something is truly screwed up when 
> rt_task_set_periodic(&_blink_thread, rt_timer_read(), mili2count(250)) 
> fails to make the task periodic...

That's due to the reworked timer subsystem: Starting timers (like the
periodic task timer) in the past is reported as error - up to the
application in this case. One may discuss if this case can be considered
as an undocumented API change (I haven't re-read the docs in this regard
yet).

However, what about this to resolve your issue:

while start_date < now()
	start_date += period

Of course you can write this more smartly than using a loop (e.g. if you
know that already start_date + period > now()).

Jan


-- 
Siemens AG, Corporate Technology, CT SE 2
Corporate Competence Center Embedded Linux