[CRIU] Re: [PATCH 3/4] IPC: restore namespace itself

Kinsbursky Stanislav skinsbursky at openvz.org
Tue Jan 31 03:24:15 EST 2012


31.01.2012 12:07, Pavel Emelyanov пишет:
> On 01/31/2012 12:05 PM, Kinsbursky Stanislav wrote:
>> 30.01.2012 22:04, Pavel Emelyanov пишет:
>>>> +static int prepare_ipc_data(int fd)
>>>> +{
>>>> +	int ret;
>>>> +	struct ipc_ns_data ipc;
>>>> +
>>>> +	ret = read_img(fd,&ipc);
>>>> +	if (ret<= 0)
>>>> +		return -EFAULT;
>>>> +	ret = prepare_ipc_tun(&ipc.entry);
>>>> +	if (ret<   0)
>>>> +		return ret;
>>>> +	return 0;
>>> return prepare_ipc_tun(&ipc.entry);
>> Wanted to object here. But then realized, that that will be a waste of time.
> Shame on you then.

Nope. It's a kind of wisdom. :)
There are aesthetic differences between us, and I'm a little bit tired of argue 
with you about them.
Sorry.

>> Will do as requested. But this place will be rewritten soon.
> You mean that you'll add
>
> ret = prepare_ipc_sems();
> if (ret<  0)
> 	return ret;
>
> ret = prepare_ipc_msgs();
> if (ret<  0)
> 	return ret;
>
> soon?
>
> It's OK to keep this as is in that case.
>

Yep.


-- 
Best regards,
Stanislav Kinsbursky




More information about the CRIU mailing list