improved rcl_wait in the area of timeout computation and spurious wakeups#1135
Merged
mjcarroll merged 4 commits intoros2:rollingfrom Mar 29, 2024
Merged
improved rcl_wait in the area of timeout computation and spurious wakeups#1135mjcarroll merged 4 commits intoros2:rollingfrom
mjcarroll merged 4 commits intoros2:rollingfrom
Conversation
added 3 commits
March 7, 2024 20:47
Signed-off-by: Janosch Machowinski <j.machowinski@nospam.org>
Added special handling for timers with a clock that has time override enabled. For theses timer we should not compute a timeout, as the waitset is waken up by the associated guard condition. Before this change, the waitset could wait up, because of an expected ready timer, that was acutally not ready, as the time update to the ROS_TIME had not yet arrived. Signed-off-by: Janosch Machowinski <j.machowinski@nospam.org>
Signed-off-by: Janosch Machowinski <j.machowinski@nospam.org>
Contributor
Author
|
@sloretz You added the TODO, perhaps you want to have a look at this. |
Member
|
Member
|
@jmachowinski some linting as well as test failures on this one, can you take a look? |
jmachowinski
force-pushed
the
timer_improvements
branch
2 times, most recently
from
ab72207 to
6fac0d8
Compare
This commit changes the computation of the timer timeout, to be more precise, in the case, of many registered timers. Signed-off-by: Janosch Machowinski <J.Machowinski@cellumation.com>
jmachowinski
force-pushed
the
timer_improvements
branch
from
6fac0d8 to
dd8ab57
Compare
Contributor
Author
|
@mjcarroll fixed |
Member
|
Member
|
mjcarroll
approved these changes
Mar 29, 2024
Member
|
Note that this was reverted in #1142 (comment), and we'll be working on re-applying this change with a fix to avoid broken tests asap. |
Contributor
Author
|
Found the bug, and also some bugs in the test. I will push a PR tomorrow. |
Contributor
Author
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
These patches fix unneeded wake ups in case timer override is active.
Also the improve the timeout computation in case of many registered timers.