Skip to content

Conversation

@jsquyres
Copy link
Member

@jsquyres jsquyres commented Apr 21, 2017

The usNIC BTL does not use more than 1 iov, so be sure to set it to 1 so that we don't allocate cq/rq/sq entries based on a default (i.e., >1) number of iovs per entry.

Signed-off-by: Jeff Squyres [email protected]
(cherry picked from commit 7bd2de9)

@jsquyres jsquyres added the bug label Apr 21, 2017
@jsquyres jsquyres added this to the v3.0.0 milestone Apr 21, 2017
@jsquyres jsquyres requested a review from bturrubiates April 21, 2017 01:56
@bwbarrett-bot
Copy link

Test Passed

The usNIC BTL does not use more than 1 iov, so be sure to set it to 1
so that we don't allocate cq/rq/sq entries based on a default (i.e.,
>1) number of iovs per entry.

Signed-off-by: Jeff Squyres <[email protected]>
(cherry picked from commit 7bd2de9)
Follow on to 7bd2de9: move setting
the iov_limit to 1 earlier in the startup sequence.

Signed-off-by: Jeff Squyres <[email protected]>
(cherry picked from commit 1d5e08f)
@jsquyres jsquyres force-pushed the pr/v3.0.x/usnic-fix-iov-len branch from 54689ad to e8d0e15 Compare April 21, 2017 16:16
@bwbarrett
Copy link
Member

bot:ompi:retest

1 similar comment
@bwbarrett
Copy link
Member

bot:ompi:retest

@bwbarrett
Copy link
Member

FYI: the "default" check was a configuration error in Jenkins and will never move out of pending. I can't figure out how to remove it, so please ignore when waiting to merge.

@bwbarrett
Copy link
Member

bot:ompi:retest

@hppritcha
Copy link
Member

ignoring weirdness at AWS.

@hppritcha hppritcha merged commit 2e6fd7c into open-mpi:v3.x Apr 24, 2017
@jsquyres jsquyres deleted the pr/v3.0.x/usnic-fix-iov-len branch October 4, 2019 21:37
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.

5 participants