Quoting Dan Smith ([email protected]):
> This allows the caller to set the CHECKPOINT_NONETNS flag on the checkpoint
> operation, thus skipping the checkpointing of network namespaces entirely.
> 
> Signed-off-by: Dan Smith <[email protected]>

Thanks, Dan.  Can we add '-N' as a synonym for my lazy fingers?

Also, args does get memset to 0 before parse_args() gets called so
you don't need to set args->flag to 0 there.

But

Acked-by: Serge Hallyn <[email protected]>

> ---
>  app-checkpoint.h  |    1 +
>  checkpoint-main.c |    7 +++++++
>  2 files changed, 8 insertions(+), 0 deletions(-)
> 
> diff --git a/app-checkpoint.h b/app-checkpoint.h
> index f740085..c078f04 100644
> --- a/app-checkpoint.h
> +++ b/app-checkpoint.h
> @@ -5,6 +5,7 @@ struct app_checkpoint_args {
>       int uerrfd;
>       int container;
>       int verbose;
> +     int flags;
>  };
> 
>  struct app_restart_args {
> diff --git a/checkpoint-main.c b/checkpoint-main.c
> index f6faa32..bef6bfe 100644
> --- a/checkpoint-main.c
> +++ b/checkpoint-main.c
> @@ -30,6 +30,7 @@ static char usage_str[] =
>  "  -l,--logfile=FILE     write error and debug data to FILE (default=none)\n"
>  "     --logile-fd=FD     write error and debug data to file descriptor FD\n"
>  "  -c,--container        require the PID is a container-init\n"
> +"     --nonetns          do not checkpoint network namespace(s)\n"
>  "  -v,--verbose          verbose output\n"
>  "";
> 
> @@ -61,6 +62,7 @@ static void parse_args(struct app_checkpoint_args *args, 
> int argc, char *argv[])
>               { "logfile-fd", required_argument,      NULL, 2 },
>               { "container",  no_argument,            NULL, 'c' },
>               { "verbose",    no_argument,            NULL, 'v' },
> +             { "nonetns",    no_argument,            NULL, 3 },
>               { NULL,         0,                      NULL, 0 }
>       };
>       static char optc[] = "hvco:l:";
> @@ -71,6 +73,7 @@ static void parse_args(struct app_checkpoint_args *args, 
> int argc, char *argv[])
>       args->outfd = -1;
>       args->logfd = -1;
>       args->uerrfd = fileno(stderr);
> +     args->flags = 0;
>       output = NULL;
>       logfile = NULL;
> 
> @@ -109,6 +112,9 @@ static void parse_args(struct app_checkpoint_args *args, 
> int argc, char *argv[])
>               case 'v':
>                       args->verbose = 1;
>                       break;
> +             case 3:
> +                     args->flags |= CHECKPOINT_NONETNS;
> +                     break;
>               default:
>                       usage(usage_str);
>               }
> @@ -164,6 +170,7 @@ int main(int argc, char *argv[])
>               exit(1);
>       }
> 
> +     flags = args.flags;
>       if (!args.container)
>               flags |= CHECKPOINT_SUBTREE;
> 
> -- 
> 1.6.3.3
> 
> _______________________________________________
> Containers mailing list
> [email protected]
> https://lists.linux-foundation.org/mailman/listinfo/containers
_______________________________________________
Containers mailing list
[email protected]
https://lists.linux-foundation.org/mailman/listinfo/containers

_______________________________________________
Devel mailing list
[email protected]
https://openvz.org/mailman/listinfo/devel

Reply via email to