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

Topic/thread framework #7

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Topic/thread framework #7

wants to merge 6 commits into from

Conversation

hppritcha
Copy link
Owner

@hppritcha hppritcha commented Mar 23, 2019

Add a framework to support different types of threading models including
user space thread packages such as Qthreads and argobots:

https://github.com/pmodels/argobots

https://github.com/Qthreads/qthreads

The default threading model is pthreads. Alternate thread models are
specificed at configure time using the --with-threads=X option.

The framework is static. The theading model to use is selected at
Open MPI configure/build time.

npe9 and others added 6 commits March 23, 2019 08:14
Add a framework to support different types of threading models including
user space thread packages such as Qthreads and argobot:

https://github.com/pmodels/argobots

https://github.com/Qthreads/qthreads

The default threading model is pthreads.  Alternate thread models are
specificed at configure time using the --with-threads=X option.

The framework is static.  The theading model to use is selected at
Open MPI configure/build time.
use the OPAL_CONFIG_POSIX_THREADS macro

Signed-off-by: Howard Pritchard <[email protected]>
remove workaround introduced into the wrappers Makefile.
This can be done due to the configury refactor.

rework the threads framework configury to move a good chunk
of setup to the top config directory to make sure thread
package specific compile/link args are defined early in
the configure process.

Signed-off-by: Howard Pritchard <[email protected]>
config: fix thread configury

- Add double quotations
- Change Argobot to Argobots
config: implement Argobots check

If the poll time is too long, MPI hangs.

This quick fix just sets it to 0, but it is not good for the
Pthreads version. Need to find a good way to abstract it.

Note that even 1 (= 1 millisecond) causes disastrous performance
degradation.
hppritcha added a commit that referenced this pull request Sep 7, 2019
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