Skip to content

usnic should not send an empty modex when there are no usNIC devices #248

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.

Sign up for GitHub

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

Jump to bottom
Closed
jsquyres opened this issue Oct 22, 2014 · 1 comment
Closed

usnic should not send an empty modex when there are no usNIC devices #248

jsquyres opened this issue Oct 22, 2014 · 1 comment
Assignees
@jsquyres
Milestone
Open MPI 1.8.5

Comments

@jsquyres
Copy link
Member

Per #235, @ggouaillardet says that the usNIC BTL is sending around an empty modex when there are no usNIC devices (in v1.8).

Ralph says that this may be necessary in v1.8; need to investigate and see if usnic actually needs to do this. If not, usnic should not send around empty modex messages.

On master, a quick look indicates that usnic is probably not sending an empty modex around, but that should be double checked.
@jsquyres jsquyres self-assigned this Oct 22, 2014
@jsquyres jsquyres added this to the Open MPI 1.8.5 milestone Oct 22, 2014
@jsquyres
Copy link
Member Author

Confirmed: if the probe for usnic devices succeeds but turns up no devices, we'll send an empty modex message around (both master and v1.8). Doh!

ggouaillardet added a commit that referenced this issue May 6, 2016
@ggouaillardet
coll/libnbc: fix MPI_Ireduce_scatter_block for one task communicator
6c9d65c 
Thanks Lisandro Dalcin for the report

Fixes #248
ggouaillardet added a commit to ggouaillardet/ompi-release that referenced this issue May 6, 2016
@ggouaillardet
coll/libnbc: fix MPI_Ireduce_scatter_block for one task communicator
8bd2810 
Thanks Lisandro Dalcin for the report

Fixes open-mpi/ompi#248

(cherry picked from commit open-mpi/ompi@6c9d65c)
@ggouaillardet ggouaillardet mentioned this issue May 6, 2016
Merged
jsquyres pushed a commit to jsquyres/ompi that referenced this issue Sep 21, 2016
Merge pull request open-mpi#248 from jsquyres/pr/rsh-mca-param-levels
fe69987 
plm rsh: assign some levels to the rsh PLM MCA params
bosilca pushed a commit to bosilca/ompi that referenced this issue Oct 3, 2016
@ggouaillardet @bosilca
coll/libnbc: fix MPI_Ireduce_scatter_block for one task communicator
c104b62 
Thanks Lisandro Dalcin for the report

Fixes open-mpi#248
dong0321 pushed a commit to ICLDisco/ompi that referenced this issue Oct 8, 2019
@rhc54
Merge pull request open-mpi#248 from pmix/topic/duh
f6de289 
Don't always include the host daemon in collective
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jsquyres jsquyres

Labels
None yet
Projects
None yet
Milestone
Open MPI 1.8.5
Development

No branches or pull requests
1 participant
@jsquyres