[CRIU] [PATCH 2/5] rpc: rename criu_dump_resp to criu_opts and add restore type of req

[Andrew Vagin]

On Mon, Sep 30, 2013 at 03:06:23PM +0400, Ruslan Kuprieiev wrote:
> Signed-off-by: Ruslan Kuprieiev <kupruser at gmail.com>

> ---
> diff --git a/protobuf/rpc.proto b/protobuf/rpc.proto
> index a07b05f..1ce6d49 100644
> --- a/protobuf/rpc.proto
> +++ b/protobuf/rpc.proto
> @@ -1,6 +1,6 @@
> -message criu_dump_req {
> +message criu_opts {
>  	required int32 images_dir_fd	= 1;
> -	optional int32 pid		= 2; //if not set, will dump requesting process
> +	optional int32 pid		= 2; //if not set on dump, will dump requesting process
>  
>  	optional bool leave_running	= 3;
>  	optional bool ext_unix_sk	= 4;
> @@ -8,16 +8,22 @@ message criu_dump_req {
>  	optional bool evasive_devices	= 6;
>  	optional bool shell_job		= 7;
>  	optional bool file_locks	= 8;
> -	optional int32 log_level	= 9 [default = 2];
> +	optional string log_file	= 9;
> +	optional int32 log_level	= 10 [default = 2];
>  }

Do you understand, that you break backward compatibility?
>  
>  message criu_dump_resp {
>  	optional bool restored		= 1;
>  }
>  
> +message criu_restore_resp {
> +	required int32 pid		= 1;
> +}
> +
>  enum criu_req_type {
>  	EMPTY		= 0;
>  	DUMP		= 1;
> +	RESTORE		= 2;
>  }
>  
>  /*
> @@ -28,7 +34,7 @@ enum criu_req_type {
>  message criu_req {
>  	required criu_req_type type	= 1;
>  
> -	optional criu_dump_req dump	= 2;
> +	optional criu_opts opts		= 2;
>  }
>  
>  /*
> @@ -41,4 +47,5 @@ message criu_resp {
>  	required bool success		= 2;
>  
>  	optional criu_dump_resp	dump	= 3;
> +	optional criu_restore_resp restore = 4;
>  }
> 

> _______________________________________________
> CRIU mailing list
> CRIU at openvz.org
> https://lists.openvz.org/mailman/listinfo/criu