-
Notifications
You must be signed in to change notification settings - Fork 3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Robustify equeue multithread unit test #12126
Conversation
Rather than wait 10ms before breaking the other thread, and expecting at least one event to have been run, wait for one event to be run, then break. Should avoid some spurious failures that have been seen.
4e0f61e
to
2d37aa2
Compare
Failure seen today testing #12097 and others. |
@kjbracey-arm, thank you for your changes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just one question
void touch() | ||
{ | ||
lock(); | ||
if (count < 200) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why 200? What is the significance of this value ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's from the original code.
Don't think it's particularly magic - it's using an 8-bit counter for "how many times was this called", but stops incrementing at 200 to avoid any wrap possibility.
CI started |
Test run: SUCCESSSummary: 1 of 1 test jobs passed |
That wasn't requested for 5.15.x as far as i can tell? |
Test change. Lets pull it in the next patch. |
Summary of changes
Rather than wait 10ms before breaking the other thread, and expecting at
least one event to have been run, wait for one event to be run, then
break.
Should avoid some spurious failures that have been seen.
Impact of changes
Migration actions required
Documentation
None
Pull request type
Test results
Reviewers