Skip to content
This repository was archived by the owner on Sep 30, 2022. It is now read-only.

Conversation

@ggouaillardet
Copy link
Contributor

test for existence of struct ibv_exp_device_attr.exp_atomic_cap.
That was previously mistyped struct ibv_exp_device_attr.ext_atomic_cap

(back-ported from commit open-mpi/ompi@1b3be0a)

@mellanox-github
Copy link

Test PASSed.
See http://bgate.mellanox.com/jenkins/job/gh-ompi-release-pr/2002/ for details.

@ggouaillardet
Copy link
Contributor Author

:bot:assign: @hjelmn
:bot:milestone:v2.0.1
:bot🏷️but

@ompiteam-bot ompiteam-bot added this to the v2.0.1 milestone Aug 1, 2016
@ompiteam-bot
Copy link

OMPIBot error: Label "but" does not exist.

@ggouaillardet
Copy link
Contributor Author

:bot🏷️bug

@mellanox-github
Copy link

Test PASSed.
See http://bgate.mellanox.com/jenkins/job/gh-ompi-release-pr/2003/ for details.

@jsquyres
Copy link
Member

jsquyres commented Aug 2, 2016

Note that with this PR, I still get 2 warnings in the openib BTL (admittedly, one is unrelated):

btl_openib_component.c: In function 'init_one_port':
btl_openib_component.c:783:5: warning: "HAVE_STRUCT_IBV_EXP_DEVICE_ATTR_EXT_ATOM" is not defined [-Wundef]
 #if HAVE_STRUCT_IBV_EXP_DEVICE_ATTR_EXT_ATOM
     ^
btl_openib_component.c: In function 'init_one_device':
btl_openib_component.c:2128:17: warning: unused variable 'index' [-Wunused-variable]
             int index;
                 ^

ggouaillardet and others added 3 commits August 4, 2016 14:14
test for existence of struct ibv_exp_device_attr.exp_atomic_cap.
That was previously mistyped struct ibv_exp_device_attr.ext_atomic_cap

(back-ported from commit open-mpi/ompi@1b3be0a)
(back-ported from commit open-mpi/ompi@d5a2ac6)

Conflicts:
	opal/mca/btl/openib/btl_openib_component.c
@ggouaillardet ggouaillardet force-pushed the topic/v2.x/openib_exp_atomic_cap branch from 09c6ced to f3149f2 Compare August 4, 2016 05:27
@ggouaillardet
Copy link
Contributor Author

the two warnings were unrelated indeed. i picked some two more commits, and we are good now

:bot:nolabel:pushed-back

@mellanox-github
Copy link

Test PASSed.
See http://bgate.mellanox.com/jenkins/job/gh-ompi-release-pr/2018/ for details.

@jsquyres
Copy link
Member

jsquyres commented Aug 4, 2016

👍

@hppritcha I think this is good to go.

@hppritcha hppritcha merged commit 007ee59 into open-mpi:v2.x Aug 5, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants