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

--hpx:print-bind doesn't work with --hpx:pu-step #2313

Closed
brycelelbach opened this issue Aug 29, 2016 · 2 comments · Fixed by #2316
Closed

--hpx:print-bind doesn't work with --hpx:pu-step #2313

brycelelbach opened this issue Aug 29, 2016 · 2 comments · Fixed by #2316

Comments

@brycelelbach
Copy link
Member

--hpx:print-bind does not work with --hpx:pu-step. It would be nice if we had a diagnostic option to validate affinity settings that worked with --hpx:pu-step.

For reference, src/hpx_init.cpp line 602 is where we disable --hpx:print-bind for --hpx:pu-step (and I did try just disabling that error to confirm that --hpx:print-bind does not in fact work with --hpx:pu-step).

@brycelelbach brycelelbach changed the title --hpx:print-bind doesn't work with --hpx:print-bind doesn't work with --hpx:pu-step Aug 29, 2016
@hkaiser
Copy link
Member

hkaiser commented Aug 29, 2016

This problem has been introduced at the point when we made --hpx:bind=balanced the default (#2180). This was done by defining an implied value for this command line option, essentially making it present always.
This leads to the issue that now --hpx:pu-step is unusable as it is not allowed to be used together with --hpx:bind.

The easiest workaround for now would be to revert the patch related to --hpx:bind=balanced.

@hkaiser
Copy link
Member

hkaiser commented Aug 29, 2016

@brycelelbach I think I know how to fix this and will try to get a PR ready asap.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants