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.

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

ORTE/OOB/UD set CQE size without querying the MAX supported size #1301

Closed
shamisp opened this issue Jan 13, 2016 · 4 comments
Closed

ORTE/OOB/UD set CQE size without querying the MAX supported size #1301

shamisp opened this issue Jan 13, 2016 · 4 comments
Assignees
@igor-ivanov
Labels
bug
Milestone
Future

Comments

@shamisp
Copy link
Contributor

shamisp commented Jan 13, 2016

https://github.com/open-mpi/ompi/blob/master/orte/mca/oob/ud/oob_ud_qp.c#L53

You can't use static values since it may not work on a certain platforms. You have to query the size first and then use it.

(Link to the original report: openucx/ucx#551 (comment))
@shamisp shamisp added the bug label Jan 13, 2016
@shamisp shamisp added this to the Future milestone Jan 13, 2016
@shamisp shamisp mentioned this issue Jan 13, 2016
Closed
@jsquyres
Copy link
Member

Per 16 Jan 2016 teleconf, @jladd-mlnx will assign this to someone in Mellanox.

@jladd-mlnx
Copy link
Member

@artpol84 could you, please, address this.

@jladd-mlnx jladd-mlnx assigned igor-ivanov and unassigned artpol84 Jan 19, 2016
@jladd-mlnx
Copy link
Member

Assigning this to @igor-ivanov instead. Igor, please address this ASAP.

igor-ivanov added a commit to igor-ivanov/ompi that referenced this issue Jan 20, 2016
@igor-ivanov
orte/oob: Fix issue open-mpi#1301
34d861d 
Signed-off-by: Igor Ivanov <Igor.Ivanov@itseez.com>
@igor-ivanov igor-ivanov mentioned this issue Jan 20, 2016
Merged
@jladd-mlnx
Copy link
Member

Fixed via #1301

hjelmn added a commit that referenced this issue Jan 25, 2016
@hjelmn
Merge pull request #1313 from igor-ivanov/pr/issue-1301
aec3060 
orte/oob: Fix issue #1301
@igor-ivanov igor-ivanov mentioned this issue Jan 26, 2016
Merged
artpol84 pushed a commit to artpol84/ompi that referenced this issue Jan 26, 2016
@igor-ivanov @artpol84
orte/oob: Fix issue open-mpi#1301
39bbfdd 
Signed-off-by: Igor Ivanov <Igor.Ivanov@itseez.com>
jsquyres pushed a commit to jsquyres/ompi that referenced this issue Aug 23, 2016
@igor-ivanov
orte/oob: Fix issue open-mpi#1301
9311bab 
Signed-off-by: Igor Ivanov <Igor.Ivanov@itseez.com>
jsquyres added a commit to jsquyres/ompi that referenced this issue Aug 23, 2016
@jsquyres
Merge pull request open-mpi#921 from igor-ivanov/pr/issue-1301
bc1420c 
orte/oob: Fix issue open-mpi#1301
jsquyres added a commit to jsquyres/ompi that referenced this issue Aug 23, 2016
@jsquyres
Merge pull request open-mpi#1301 from jsquyres/pr/fix-unweighted-values
d7db39c 
mpi.h: fix types of MPI_UNWEIGHTED and MPI_WEIGHTS_EMPTY
bosilca pushed a commit to bosilca/ompi that referenced this issue Oct 3, 2016
@igor-ivanov @bosilca
orte/oob: Fix issue open-mpi#1301
b36b457 
Signed-off-by: Igor Ivanov <Igor.Ivanov@itseez.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@igor-ivanov igor-ivanov

Labels
bug
Projects
None yet
Milestone
Future
Development

No branches or pull requests
6 participants
@jsquyres @hjelmn @shamisp @artpol84 @jladd-mlnx @igor-ivanov