Re: [lxc-devel] [PATCH 1/1] api locking: take process lock around open/close

2013-05-31 Thread Serge Hallyn
Quoting Stéphane Graber (stgra...@ubuntu.com): > On 05/29/2013 11:21 PM, Serge Hallyn wrote: > > And update the comment explaining the locking. > > > > Also take memlock in want_daemonize. > > > > Signed-off-by: Serge Hallyn > > Is that patch still relevant? > > It doesn't appear to have been

Re: [lxc-devel] [PATCH 1/1] api locking: take process lock around open/close

2013-05-31 Thread Stéphane Graber
On 05/29/2013 11:21 PM, Serge Hallyn wrote: > And update the comment explaining the locking. > > Also take memlock in want_daemonize. > > Signed-off-by: Serge Hallyn Is that patch still relevant? It doesn't appear to have been reviewed/applied to staging and touches bits that have been changed

Re: [lxc-devel] [PATCH 1/1] api locking: take process lock around open/close

2013-05-30 Thread Serge Hallyn
Quoting Dwight Engen (dwight.en...@oracle.com): > On Wed, 29 May 2013 22:21:22 -0500 > Serge Hallyn wrote: > > > And update the comment explaining the locking. > > > > Also take memlock in want_daemonize. > > Hi Serge, could you explain a bit what the locking is protecting about > open/close? I

Re: [lxc-devel] [PATCH 1/1] api locking: take process lock around open/close

2013-05-30 Thread Serge Hallyn
Quoting Dwight Engen (dwight.en...@oracle.com): > On Wed, 29 May 2013 22:21:22 -0500 > Serge Hallyn wrote: > > > And update the comment explaining the locking. > > > > Also take memlock in want_daemonize. > > Hi Serge, could you explain a bit what the locking is protecting about > open/close? I

Re: [lxc-devel] [PATCH 1/1] api locking: take process lock around open/close

2013-05-30 Thread Dwight Engen
On Wed, 29 May 2013 22:21:22 -0500 Serge Hallyn wrote: > And update the comment explaining the locking. > > Also take memlock in want_daemonize. Hi Serge, could you explain a bit what the locking is protecting about open/close? It looks like you are locking around just the open or just the clos

[lxc-devel] [PATCH 1/1] api locking: take process lock around open/close

2013-05-29 Thread Serge Hallyn
And update the comment explaining the locking. Also take memlock in want_daemonize. Signed-off-by: Serge Hallyn --- src/lxc/lxccontainer.c | 153 + 1 file changed, 115 insertions(+), 38 deletions(-) diff --git a/src/lxc/lxccontainer.c b/src/lxc/l