Skip to content

Conversation

@markalle
Copy link
Contributor

This is mostly based off recent UCX additions to their patcher:
openucx/ucx#2703

They added triggers for

  • mmap when (flags & MAP_FIXED) && (addr != NULL)
  • shmat when (shmflg & SHM_REMAP) && (shmaddr != NULL)

Beyond that I noticed they already had a trigger for

  • madvise when (advice == MADV_FREE)
    that we didn't so I added that.

And the other main thing is we didn't really have shmat/shmdt
active for some systems because we only had a path for
syscall(SYS_shmdt, ) but we needed to also have a path for
syscall(SYS_ipc, IPCOP_shmdt, ) and same for shmat.

Signed-off-by: Mark Allen markalle@us.ibm.com
(cherry picked from commit eb88811)

This is mostly based off recent UCX additions to their patcher:
    openucx/ucx#2703

They added triggers for
* mmap when (flags & MAP_FIXED) && (addr != NULL)
* shmat when (shmflg & SHM_REMAP) && (shmaddr != NULL)

Beyond that I noticed they already had a trigger for
* madvise when (advice == MADV_FREE)
that we didn't so I added that.

And the other main thing is we didn't really have shmat/shmdt
active for some systems because we only had a path for
syscall(SYS_shmdt, ) but we needed to also have a path for
syscall(SYS_ipc, IPCOP_shmdt, ) and same for shmat.

Signed-off-by: Mark Allen <markalle@us.ibm.com>
(cherry picked from commit eb88811)
@markalle markalle requested a review from yosefe May 30, 2019 17:41
@AboorvaDevarajan
Copy link
Member

Can one of the admins verify this patch?

@bwbarrett bwbarrett merged commit 59210b8 into open-mpi:v3.0.x Jun 4, 2019
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