Skip to content

Conversation

@hjelmn
Copy link
Member

@hjelmn hjelmn commented Mar 7, 2017

It is not possible to hook madvise at this time due to a deadlock when
using glibc.

Signed-off-by: Nathan Hjelm hjelmn@lanl.gov
(cherry picked from commit 3caeda2)
Signed-off-by: Nathan Hjelm hjelmn@lanl.gov

It is not possible to hook madvise at this time due to a deadlock when
using glibc.

Signed-off-by: Nathan Hjelm <hjelmn@lanl.gov>
(cherry picked from commit 3caeda2)
Signed-off-by: Nathan Hjelm <hjelmn@lanl.gov>
@hjelmn hjelmn added this to the v2.0.3 milestone Mar 7, 2017
@hjelmn hjelmn requested a review from bosilca March 7, 2017 23:30
@jsquyres
Copy link
Member

jsquyres commented Mar 8, 2017

@jladd-mlnx @artpol84 Mellanox Jenkins fails with this error:

19:16:57 [jenkins03:19952] Error: pshmem_init.c:74 - _shmem_init() SHMEM failed to initialize - aborting

Is that a real error, or a false positive?

@jsquyres
Copy link
Member

jsquyres commented Mar 8, 2017

It looks like the Mellanox Jenkins may have pulled in a zillion commits for this PR. That doesn't seem right -- there should be a single commit...?

@artpol84
Copy link
Contributor

artpol84 commented Mar 8, 2017

bot:mellanox:retest

2 similar comments
@Di0gen
Copy link

Di0gen commented Mar 8, 2017

bot:mellanox:retest

@Di0gen
Copy link

Di0gen commented Mar 8, 2017

bot:mellanox:retest

@Di0gen
Copy link

Di0gen commented Mar 8, 2017

The Mellanox's Jenkins pulled right commit,
But on Jenkins page, it printed the diff HEAD vs master. This is the reason of this long commits list.
Settings changed, now it isn't printing the chanlog list.

@jsquyres
Copy link
Member

jsquyres commented Mar 8, 2017

@Di0gen @artpol84 Thanks for fixing. Can you please check the Mellanox Jenkins for #3123, too? It looks like it also pulled a zillion commits. Just to make sure: we have PRs actively being filed against 4 different branches these days:

  • v1.10
  • master
  • v2.0.x
  • v2.x

Shortly (within a week or three), we'll have a 5th branch (v3.x), but the activity on the v1.10 branch will likely die off.

@jsquyres
Copy link
Member

jsquyres commented Mar 8, 2017

@hppritcha We agreed on this PR yesterday on the call, so I'm going to merge.

@jsquyres jsquyres merged commit 5ddaa4a into open-mpi:v2.0.x Mar 8, 2017
@jsquyres jsquyres changed the title memory/patcher: do not hook madvise v2.0.x: memory/patcher: do not hook madvise Mar 8, 2017
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