[CRIU] [PATCH 9/9] test: zdtm -- Add proc-self test

Pavel Emelyanov xemul at parallels.com
Fri May 17 10:27:43 EDT 2013


On 05/17/2013 06:20 PM, Cyrill Gorcunov wrote:
> On Fri, May 17, 2013 at 06:13:09PM +0400, Pavel Emelyanov wrote:
>> On 05/17/2013 06:11 PM, Cyrill Gorcunov wrote:
>>> On Fri, May 17, 2013 at 06:03:44PM +0400, Pavel Emelyanov wrote:
>>>>
>>>> This can't PASS strictly speaking. On restore we create _another_ namespace and
>>>> do not guarantee that kernel creates the same ID for namespace. See
>>>
>>> It doesn't check for NS ID, but for NS name. We've opened "net", an it must
>>> remain "net" after restore.
>>
>> The it's also wrong. It should not only remain "net", but remain "my net".
> 
> But we don't support yet same ID restore, so at least minimum check is needed,
> if you prefer I can simply drop this test out.

No, I prefer if test after restore opens /proc/self/net again and compares
it with what is opened.

> .
> 




More information about the CRIU mailing list