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 jsquyres@cisco.com
(cherry picked from commit 7bd2de9)

@jsquyres jsquyres added the bug label Apr 21, 2017
@jsquyres jsquyres added this to the v2.1.1 milestone Apr 21, 2017
@jsquyres jsquyres requested a review from bturrubiates April 21, 2017 01:54
@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 <jsquyres@cisco.com>
(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 <jsquyres@cisco.com>
(cherry picked from commit 1d5e08f)
@jsquyres jsquyres force-pushed the pr/v2.1.x/usnic-fix-iov-len branch from 6ce6f9e to 3c5b86d Compare April 21, 2017 16:17
@jsquyres
Copy link
Member Author

@hppritcha Good to go when CI finishes.

@hppritcha hppritcha merged commit 74ff15b into open-mpi:v2.x Apr 22, 2017
@jsquyres jsquyres deleted the pr/v2.1.x/usnic-fix-iov-len branch August 2, 2018 16:57
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.

4 participants