[lxc-devel] [PATCH] lxc-checkpoint -r should actually wait for the restore to happen
Serge Hallyn
serge.hallyn at ubuntu.com
Tue Mar 24 20:00:42 UTC 2015
Quoting Tycho Andersen (tycho.andersen at canonical.com):
> On Tue, Mar 24, 2015 at 07:40:48PM +0000, Serge Hallyn wrote:
> > Quoting Tycho Andersen (tycho.andersen at canonical.com):
> > > + if (pid != 0)
> > > + wait_for_pid(pid);
> >
> > Sorry - I suspect some package builds will fail on the ignore of
> > return value here. Could this be
> >
> > if (pid != 0)
> > return wait_for_pid(pid) == 0;
>
> Sure,
>
>
> From ca37de3207433bc0d2c55b466c99d931334389b3 Mon Sep 17 00:00:00 2001
> From: Tycho Andersen <tycho.andersen at canonical.com>
> Date: Fri, 20 Mar 2015 10:17:31 -0600
> Subject: [PATCH] lxc-checkpoint -r should actually wait for the restore to
> happen
>
> v2: use wait_for_pid
> v2: check wait_for_pid's return value
>
> Signed-off-by: Tycho Andersen <tycho.andersen at canonical.com>
Acked-by: Serge E. Hallyn <serge.hallyn at ubuntu.com>
> ---
> src/lxc/lxc_checkpoint.c | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/src/lxc/lxc_checkpoint.c b/src/lxc/lxc_checkpoint.c
> index cfa08fc..86f1f4b 100644
> --- a/src/lxc/lxc_checkpoint.c
> +++ b/src/lxc/lxc_checkpoint.c
> @@ -27,6 +27,7 @@
> #include "config.h"
> #include "lxc.h"
> #include "arguments.h"
> +#include "utils.h"
>
> static char *checkpoint_dir = NULL;
> static bool stop = false;
> @@ -168,6 +169,9 @@ bool restore(struct lxc_container *c)
>
> lxc_container_put(c);
>
> + if (pid != 0)
> + return wait_for_pid(pid) == 0;
> +
> return ret;
> }
>
> --
> 2.1.0
>
> _______________________________________________
> lxc-devel mailing list
> lxc-devel at lists.linuxcontainers.org
> http://lists.linuxcontainers.org/listinfo/lxc-devel
More information about the lxc-devel
mailing list