OpenVZ Forum


Home » Mailing lists » Devel » [PATCH 0/13] Pid namespaces (OpenVZ view)
Re: [PATCH 11/13] Changes to show virtual ids to user [message #13626 is a reply to message #13605] Thu, 31 May 2007 07:57 Go to previous messageGo to previous message
Pavel Emelianov is currently offline  Pavel Emelianov
Messages: 1149
Registered: September 2006
Senior Member
Cedric Le Goater wrote:
> Hello !
>
>>>> The worst case I can see with pid == 0. Is that it would be a bug
>>>> that we can fix later. For other cases it would seem to be a user
>>>> space API thing that we get stuck with for all time.
>>> We cannot trust userspace application to expect some pid other than
>>> positive. All that we can is either use some always-absent pid or
>>> send the signal as SI_KERNEL.
>>>
>>> Our experience show that making decisions like above causes random
> <>> applications failures that are hard (or even impossible) to debug.
>
>> Ok. So I guess I see what you are proposing is picking an arbitrary
>> pid, say pid == 2, and reserving that in all pid namespaces and using
>> it when we have a pid that does not map to a specific namespace. I'm
>> fine with that.
>>
>> All I care about is that we have a solution, preferably simple,
>> to the non-mapped pid problem.
>
> Pavel, are you against using pid == 0 and setting si_code to SI_KERNEL ?

I think I am. A quick grep through the code revealed one place where
this can happen, so I believe application are (have to be) somehow
prepared to this.

> C.
>
 
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Previous Topic: Re: [ckrm-tech] [RFC] [PATCH 0/3] Add group fairness to CFS
Next Topic: [RFC][PATCH 0/16] Enable cloning of pid namespace
Goto Forum:
  


Current Time: Tue Jul 29 10:40:15 GMT 2025

Total time taken to generate the page: 0.34753 seconds