Skip to content

[libc++][test] adding yield in the lost wakeup test - #211261

Closed
huixie90 wants to merge 3 commits into
llvm:mainfrom
huixie90:hxie/test_wakeup
Closed

huixie90 wants to merge 3 commits into
llvm:mainfrom
huixie90:hxie/test_wakeup

Conversation

@huixie90

Copy link
Copy Markdown
Member

No description provided.

@ldionne

ldionne commented Jul 24, 2026

Copy link
Copy Markdown
Member

@mikaelholmen Could you please try out this change and see if it solves the issues you reported in #161086 (comment) ?

@mikaelholmen

Copy link
Copy Markdown
Contributor

@mikaelholmen Could you please try out this change and see if it solves the issues you reported in #161086 (comment) ?

Hi @ldionne
(I'm finally back from vacations)

Yes, I do think this helps and gets the worst case execution time down quite a lot when the machine is under heavy load.
Without this I could see execution times up to 400 seconds for the testcase, but with the patch I never saw it reach more than 30 seconds, so it looks like it has positive effect.

@huixie90 huixie90 closed this Sep 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants