From mboxrd@z Thu Jan 1 00:00:00 1970 From: ebiederm-aS9lmoZGLiVWk0Htik3J/w@public.gmane.org (Eric W. Biederman) Subject: Re: [PATCH 3/9] pid: Implement ns_of_pid. Date: Wed, 12 Dec 2007 18:25:44 -0700 Message-ID: References: <20071213005945.GB27896@us.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: In-Reply-To: <20071213005945.GB27896-r/Jw6+rmf7HQT0dZR+AlfA@public.gmane.org> (sukadev-r/Jw6+rmf7HQT0dZR+AlfA@public.gmane.org's message of "Wed, 12 Dec 2007 16:59:45 -0800") List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: containers-bounces-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org Errors-To: containers-bounces-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org To: sukadev-r/Jw6+rmf7HQT0dZR+AlfA@public.gmane.org Cc: Linux Containers , Andrew Morton , Oleg Nesterov , Pavel Emelyanov List-Id: containers.vger.kernel.org sukadev-r/Jw6+rmf7HQT0dZR+AlfA@public.gmane.org writes: > > My patch refers to this function as pid_active_pid_ns() - I have > been meaning to send that out on top of your signals patch. > Since a pid has many namespaces, we have been using 'active pid ns' > to refer to this ns. Currently we don't ask for any of the others, and the namespace the pid came from is special. That fundamentally is the namespace of the pid. The rest byproducts of being in that pid namespace, as we could derive them by walking the namespace's parent list. > Even your next patch modifies task_active_pid_ns() to use this. > So can we rename this functio to pid_active_pid_ns() ? I'd be more inclined to rename task_active_pid_ns to task_pid_ns. And to rename pid_in_pid_ns that Pavel has issues with to pid_in_ns. When I read active_pid_ns I wonder what the other namespaces are that we are distinguishing this from. They do exist in the implementation but so far it is a complete don't care. So I expect being as terse as we can while still conveying all of the relevant information is the most maintainable long term. Eric