[CRIU] [PATCH] service: add "one connection -- multiple requests" support

Pavel Emelyanov xemul at parallels.com
Fri Nov 22 09:28:52 PST 2013


On 11/23/2013 12:22 AM, Ruslan Kuprieiev wrote:
> On 22.11.2013 21:20, Pavel Emelyanov wrote:
>>>>> -	default: {
>>>>> -		CriuResp resp = CRIU_RESP__INIT;
>>>>> +		if (ret == 0)
>>>>> +			break;
>>>>> +
>>>>> +		pid = fork();
>>>> NAK. Each request should be synchronous withing session.
>>> Oh, yeah. I will add wait() after loop. Ok?
>> Why not just omiting fork() here?
> Because if we have restore request further request won't be processed, 
> as worker is replaced by restored process.

No, worker cannot be replaced by restored process, as the restore_root_task
does fork() itself.

> .
> 




More information about the CRIU mailing list