From mboxrd@z Thu Jan 1 00:00:00 1970 From: NAGARATHNAM MUTHUSAMY Subject: Re: [REVIEW][PATCH 09/11] ipc/shm: Fix shmctl(..., IPC_STAT, ...) between pid namespaces. Date: Fri, 23 Mar 2018 14:41:29 -0700 Message-ID: <1091a91e-f8ee-b091-6d95-78b33520fb2d@oracle.com> References: <87vadmobdw.fsf_-_@xmission.com> <20180323191614.32489-9-ebiederm@xmission.com> <7df62190-2407-bfd4-d144-7304a8ea8ae3@oracle.com> <87lgeio4tb.fsf@xmission.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <87lgeio4tb.fsf-aS9lmoZGLiVWk0Htik3J/w@public.gmane.org> Content-Language: en-US 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: "Eric W. Biederman" Cc: esyr-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org, jannh-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org, khlebnikov-XoJtRXgx1JseBXzfvpsJ4g@public.gmane.org, linux-api-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Linux Containers , serge.hallyn-GeWIH/nMZzLQT0dZR+AlfA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, prakash.sangappa-QHcLZuEGTsvQT0dZR+AlfA@public.gmane.org, linux-security-module-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, luto-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, oleg-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org, akpm-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org, Pavel Emelyanov List-Id: linux-api@vger.kernel.org On 3/23/2018 2:33 PM, ebiederm-aS9lmoZGLiVWk0Htik3J/w@public.gmane.org wrote: > NAGARATHNAM MUTHUSAMY writes: > >> Thanks! >> >> Reviewed-by: Nagarathnam Muthusamy > Does this look like it will address the issue you have been fighting > with pids? We do use IPC shared memory but it is a single large one, shared by multiple levels. We are currently looking into using a similar solution based on file locks. When a new level is created, a file representing that level could be created in a common path which could be locked by the init process of that level. Parent levels could query the locking pid of that file to get the pid translation of the init process of the required level. Then it could open a file descriptor and use the translate_pid API for further translations. Thanks, Nagarathnam. > > Eric