[CRIU] [PATCH 4/6] dump: dump signals before infecting with parasite

Pavel Emelyanov xemul at parallels.com
Wed Aug 6 00:02:45 PDT 2014


On 08/05/2014 12:35 PM, Ruslan Kuprieiev wrote:

This patch requires comment why we do this.

> Signed-off-by: Ruslan Kuprieiev <kupruser at gmail.com>
> ---
>  cr-dump.c | 12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)
> 
> diff --git a/cr-dump.c b/cr-dump.c
> index d8ad0fc..c8be0bd 100644
> --- a/cr-dump.c
> +++ b/cr-dump.c
> @@ -1498,6 +1498,12 @@ static int dump_one_task(struct pstree_item *item)
>  		goto err;
>  	}
>  
> +	ret = dump_task_signals(pid, item, cr_fdset);
> +	if (ret) {
> +		pr_err("Dump %d signals failed %d\n", pid, ret);
> +		goto err;
> +	}
> +
>  	ret = -1;
>  	parasite_ctl = parasite_infect_seized(pid, item, &vmas, dfds, proc_args.timer_n);
>  	if (!parasite_ctl) {
> @@ -1632,12 +1638,6 @@ static int dump_one_task(struct pstree_item *item)
>  		goto err;
>  	}
>  
> -	ret = dump_task_signals(pid, item, cr_fdset);
> -	if (ret) {
> -		pr_err("Dump %d signals failed %d\n", pid, ret);
> -		goto err;
> -	}
> -
>  	close_cr_fdset(&cr_fdset);
>  err:
>  	close_pid_proc();
> 



More information about the CRIU mailing list