Skip to content

Conversation

rhc54
Copy link
Contributor

@rhc54 rhc54 commented Dec 1, 2017

Only selectable when specifically requested via "-mca odls pspawn"

Note that there are several concerns:

  • we aren't getting SIGCHLD calls when the procs terminate
  • we aren't seeing the IO pipes close on termination, though
    we are getting output forwarded to mpirun
  • I haven't found a way to bind the child process prior to exec.
    If we want to use this method, we probably need someone to
    implement a cgroup component for the orte/rtc framework

Signed-off-by: Ralph Castain [email protected]

Only selectable when specifically requested via "-mca odls pspawn"

Note that there are several concerns:
  * we aren't getting SIGCHLD calls when the procs terminate
  * we aren't seeing the IO pipes close on termination, though
    we are getting output forwarded to mpirun
  * I haven't found a way to bind the child process prior to exec.
    If we want to use this method, we probably need someone to
    implement a cgroup component for the orte/rtc framework

Signed-off-by: Ralph Castain <[email protected]>
@rhc54
Copy link
Contributor Author

rhc54 commented Dec 1, 2017

@bosilca @abouteiller @jjhursey @ggouaillardet

This implementation isn't complete - someone needs to at least resolve the SIGCHLD and IO termination issues. In addition, we would need to resolve the binding question.

I will say that I did find it faster to spawn the proc this way, so if someone cares to help resolve the problems, it might be an attractive option. I probably won't have time to continue this for the next few weeks at least.

@rhc54 rhc54 merged commit 4225b89 into open-mpi:master Dec 1, 2017
@rhc54 rhc54 deleted the topic/pspawn branch December 1, 2017 04:23
@ggouaillardet
Copy link
Contributor

ggouaillardet commented Dec 1, 2017

@rhc54 i pushed a4755b6
now i can mpirun hello_c and it does not hang

@rhc54
Copy link
Contributor Author

rhc54 commented Dec 1, 2017

oh, that is embarrassing - thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants