Skip to content
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

Add timed busy-wait option to pika::barrier wait and arrive_and_wait #685

Merged
merged 1 commit into from
May 12, 2023

Conversation

msimberg
Copy link
Contributor

This allows us to start doing eth-cscs/DLA-Future#833 in DLA-Future. The default behaviour of wait and arrive_and_wait with these changes is exactly as before. Both functions now have an optional second parameter to specify a busy-wait time during which the barrier does not yield (thanks to #672). This option should be used with care, but can noticeably improve performance when there are few tasks running outside of those participating in the barrier.

@msimberg msimberg added this to the 0.16.0 milestone May 11, 2023
@msimberg msimberg self-assigned this May 11, 2023
@msimberg msimberg marked this pull request as ready for review May 11, 2023 14:49
@msimberg
Copy link
Contributor Author

bors try

bors bot added a commit that referenced this pull request May 11, 2023
@bors
Copy link
Contributor

bors bot commented May 11, 2023

try

Build failed:

Copy link
Contributor

@aurianer aurianer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM thanks!

@msimberg
Copy link
Contributor Author

bors try

bors bot added a commit that referenced this pull request May 12, 2023
@bors
Copy link
Contributor

bors bot commented May 12, 2023

try

Build failed:

@msimberg msimberg merged commit fdbb461 into main May 12, 2023
@bors bors bot deleted the barrier-no-yield branch May 12, 2023 10:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants