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

Remove advice about local MPI_COMM_FREE #4

Closed
wesbland opened this issue Sep 25, 2015 · 2 comments
Closed

Remove advice about local MPI_COMM_FREE #4

wesbland opened this issue Sep 25, 2015 · 2 comments
Assignees
Labels
errata Errata items for the previous MPI Standard

Comments

@wesbland
Copy link
Member

The advice to implementors for MPI_COMM_FREE mentions this:

Though collective, it is anticipated that this operation will normally be implemented to be local, though a debugging version of an MPI library might choose to synchronize.

This text is confusing and somewhat meaningless as it basically removes the collective nature in advice. How the implementor chooses to have MPI_COMM_FREE is up to them as long as they implement the correct semantics.

@wesbland wesbland added errata Errata items for the previous MPI Standard for review labels Sep 25, 2015
@wesbland wesbland reopened this Sep 25, 2015
@wesbland wesbland added this to the 2015-12 San Jose, USA milestone Sep 25, 2015
@wesbland wesbland self-assigned this Sep 25, 2015
@wesbland wesbland modified the milestone: 2015-12 San Jose, USA Sep 25, 2015
@abouteiller
Copy link
Member

I tend to think that this advice to implementors is somewhat useful in reminding that the spec does not command that COMM_FREE is a synchronization barrier (unlike say, WIN_FREE, which is).

@wesbland
Copy link
Member Author

After more discussion, we decided to drop this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
errata Errata items for the previous MPI Standard
Projects
None yet
Development

No branches or pull requests

2 participants