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
Adding enable_elasticity option to pool configuration #3019
Conversation
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.
Thanks, LGTM
a15c9ff
to
55ea677
Compare
- fixing performance problem related to get_used_processing_units() for non-elastic pools
55ea677
to
5a082bb
Compare
@@ -14,7 +14,8 @@ namespace hpx { namespace threads { namespace policies | |||
do_background_work = 0x1, | |||
reduce_thread_priority = 0x02, | |||
delay_exit = 0x04, | |||
fast_idle_mode = 0x08 | |||
fast_idle_mode = 0x08, | |||
enable_elasticity = 0x10 |
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'd be nice to see a few lines of comment added here to let the next person know what each of these flags/options is for.
@biddisco your comments have been addressed. |
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.
Lovely thank you
get_used_processing_units()
fornon-elastic pools
This specifically leaves open the issue of
get_used_processing_units()
returning an unprotected reference to the mask. This has to be resolved separately.