Skip to content

Conversation

@alex-mikheev
Copy link
Contributor

@yosefe @jladd-mlnx please review

@yosefe
Copy link
Contributor

yosefe commented Feb 28, 2017

@ggouaillardet pls take a look

Signed-off-by: Alex Mikheev <alexm@mellanox.com>
@alex-mikheev alex-mikheev force-pushed the topic/mpi_unpack_fix_v2.x branch from 45045aa to 24aab62 Compare February 28, 2017 14:58
Copy link
Member

@jladd-mlnx jladd-mlnx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

@ggouaillardet
Copy link
Contributor

ggouaillardet commented Feb 28, 2017

should we cherry-pick 7b803ac instead ?
i clearly forgot to do it ...

@alex-mikheev
Copy link
Contributor Author

I does not look like just cherry picking 7b803ac will fix it

@jsquyres jsquyres added this to the v2.1.0 milestone Feb 28, 2017
@jsquyres jsquyres changed the title ompi: fixes crash in mpi_unpack v2.x: ompi: fixes crash in mpi_unpack Feb 28, 2017
@jsquyres jsquyres mentioned this pull request Feb 28, 2017
@hppritcha hppritcha merged commit 8d2c643 into open-mpi:v2.x Mar 1, 2017
@yosefe yosefe deleted the topic/mpi_unpack_fix_v2.x branch March 1, 2017 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.

6 participants