Bug #187

PeriodicTask does not guarantee fixed scheduling with long processing time

Added by J. Wienke over 9 years ago. Updated about 8 years ago.

Status:ResolvedStart date:02/14/2011
Priority:NormalDue date:
Assignee:A. Nordmann% 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 A. Nordmann almost 9 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 over 9 years ago

  • Estimated time changed from 1.50 to 1.51

#2 Updated by J. Wienke about 9 years ago

  • Target version changed from 0.2 to 0.3

#3 Updated by J. Wienke about 9 years ago

  • Target version changed from 0.3 to 0.4

#4 Updated by A. Nordmann about 9 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 almost 9 years ago

  • Target version changed from 0.4 to 0.5

#6 Updated by J. Moringen almost 9 years ago

  • Target version changed from 0.5 to 0.6

#7 Updated by J. Wienke over 8 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 over 8 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 A. Nordmann over 8 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 about 8 years ago

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

#11 Updated by A. Nordmann about 8 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