Looks okay to me - I see you left a "printf" statement in plm_base_launch_support.c, so you might want to make that an opal_output_verbose or something.
On Feb 3, 2014, at 12:19 PM, Adrian Reber <adrian_at_[hidden]> wrote:
> This patch
> introduces unique collective ids for the checkpoint/restart code and
> with this applied it seems to work pretty good. As this patch also
> touches non-CR code it would be good if someone could have a look at it.
> With this patch applied the code seems to work up to the point where
> orterun actually pauses all processes and tries to create the
> checkpoints. The checkpoint creation does not work for me as CRS does
> not yet include support for checkpoint/restart using CRIU which would be
> my next step.
> devel mailing list