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

Coordination Across Programming Models (OpenMP/MPI) #17

Merged
merged 2 commits into from
Mar 8, 2018
Merged

Coordination Across Programming Models (OpenMP/MPI) #17

merged 2 commits into from
Mar 8, 2018

Conversation

rhc54
Copy link
Member

@rhc54 rhc54 commented Mar 16, 2017

Signed-off-by: Ralph Castain rhc@open-mpi.org

/* register a handler specifically for notifying us if
* other programming models declare themselves */
active = -1;
PMIx_Register_event_handler(&code, 1, NULL, 0,
Copy link

Choose a reason for hiding this comment

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

Is it possible to register muliple event handler for the same event?

Copy link
Member Author

Choose a reason for hiding this comment

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

Not at the moment - however, that is being changed per the following RFC: #18

Choose a reason for hiding this comment

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

we might need to have a more flexible way to control when a programming model's callback function is invoked. Looking at section 3.2 of the OpenMP spec there are a number of calls that may lead to the OpenMP runtime adjusting resource requirements etc. that other program models might need to take in to account. I'm thinking, for example, of omp_set_nested.

@hppritcha
Copy link

It does look like one of the DOE mini-apps - snap would be a good guinea pig for testing the usefulness of this RFC. It can use nested OpenMP, and also has ports to kokkos - although its unclear if this works.

https://github.com/lanl/SNAP

rhc54 pushed a commit to rhc54/openpmix that referenced this pull request May 5, 2017
…ary integration

Refs pmix/RFCs#17
Refs open-mpi/ompi#3448

Signed-off-by: Ralph Castain <rhc@open-mpi.org>
rhc54 pushed a commit to rhc54/openpmix that referenced this pull request May 5, 2017
…ary integration

Refs pmix/RFCs#17
Refs open-mpi/ompi#3448

Signed-off-by: Ralph Castain <rhc@open-mpi.org>
Hybrid applications (i.e., applications that utilize more than one programming model, such as an MPI application that also uses OpenMP) are growing in popularity, especially as chips with increasingly large numbers of cores and processors proliferate. This RFC offers a potential solution to the problem by providing a pathway for programming models to coordinate their actions.

Signed-off-by: Ralph Castain <rhc@open-mpi.org>
@rhc54
Copy link
Member Author

rhc54 commented Mar 7, 2018

This has been "seconded" by ORNL.

Accepted

Signed-off-by: Ralph Castain <rhc@open-mpi.org>
@rhc54 rhc54 merged commit d6457e8 into pmix:master Mar 8, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants