Bug #187

PeriodicTask does not guarantee fixed scheduling with long processing time

Added by J. Wienke about 13 years ago. Updated almost 12 years ago.

Status:ResolvedStart date:02/14/2011
Priority:NormalDue date:
Assignee:-% Done:

100%

Category:ThreadingEstimated time:1.51 hour
Target version:0.7

Description

The processing time is not used for sleeping. Provide two versions:
  1. as it is to have a fixed sleep time
  2. one where processing time is used to calculate the sleep time.

Associated revisions

Revision 9817a02d
Added by Arne Nordmann over 12 years ago

Jitter significantly decreased and reduced number of timestamp requests to one per cycle
Jitter does no longer adds up over time
Added example for peridodic task
refs #187

History

#1 Updated by J. Wienke about 13 years ago

  • Estimated time changed from 1.50 to 1.51

#2 Updated by J. Wienke almost 13 years ago

  • Target version changed from 0.2 to 0.3

#3 Updated by J. Wienke almost 13 years ago

  • Target version changed from 0.3 to 0.4

#4 Updated by Anonymous almost 13 years ago

  • % Done changed from 0 to 30

Increased precision with Revision r299, but still an offset of ~60us remaining.

#5 Updated by J. Wienke over 12 years ago

  • Target version changed from 0.4 to 0.5

#6 Updated by J. Moringen over 12 years ago

  • Target version changed from 0.5 to 0.6

#7 Updated by J. Wienke about 12 years ago

  • Subject changed from PeriodicTask does not guarantee fixed schedulign with long processing time to PeriodicTask does not guarantee fixed scheduling with long processing time

We would like to release 0.6 this weekend. Is there a chance to get this done before?

#8 Updated by S. Wrede about 12 years ago

  • Status changed from New to Feedback
  • Target version changed from 0.6 to 0.7

Postponed due to inactivity. Should not block 0.6 release.

@Arne: Could you please elaborate a bit on the state of this issue?

#9 Updated by Anonymous about 12 years ago

Already done as far as I recall.
Needs to be tested and approved for release though. Could do that until the end of this week.

#10 Updated by J. Wienke almost 12 years ago

Can you please do the checking before leaving? Otherwise this will not get in the 0.7 release.

#11 Updated by Anonymous almost 12 years ago

  • Status changed from Feedback to Resolved
  • % Done changed from 30 to 100

Checked to be working. Periodic task accounting processing time of the task by default (can be disabled when creating the task, though, to provide previous behavior).
See source:examples/PeriodicTask.cpp

Also available in: Atom PDF