Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756543AbZAGAJZ (ORCPT ); Tue, 6 Jan 2009 19:09:25 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1753566AbZAGAJG (ORCPT ); Tue, 6 Jan 2009 19:09:06 -0500 Received: from e38.co.us.ibm.com ([32.97.110.159]:35199 "EHLO e38.co.us.ibm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753535AbZAGAJF (ORCPT ); Tue, 6 Jan 2009 19:09:05 -0500 Subject: Re: [RFC][PATCH 2/4] sunrpc: Use utsnamespaces From: Matt Helsley To: Trond Myklebust Cc: "Serge E. Hallyn" , Linux Containers , linux-nfs@vger.kernel.org, Linux Kernel Mailing List , "J. Bruce Fields" , Chuck Lever , "Eric W. Biederman" , Linux Containers , Cedric Le Goater In-Reply-To: <1231281732.4173.6.camel@heimdal.trondhjem.org> References: <20090106011314.534653345@us.ibm.com> <20090106011314.961946803@us.ibm.com> <20090106200229.GA17031@us.ibm.com> <1231274682.20316.65.camel@heimdal.trondhjem.org> <20090106215831.GE18147@us.ibm.com> <1231281732.4173.6.camel@heimdal.trondhjem.org> Content-Type: text/plain Date: Tue, 06 Jan 2009 16:08:50 -0800 Message-Id: <1231286930.14345.196.camel@localhost> Mime-Version: 1.0 X-Mailer: Evolution 2.22.3.1 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1116 Lines: 27 On Tue, 2009-01-06 at 17:42 -0500, Trond Myklebust wrote: > On Tue, 2009-01-06 at 15:58 -0600, Serge E. Hallyn wrote: > > > So should we use patch 2/4, plus (as someone - was it you? - suggested) > > using a DEFAULT instead of init_utsname()->nodename when > > current->utsname() == NULL? > > No. I'm don't think that 2/4 is correct either. Basically, 2/4 is saying > that the container that first mounts the filesystem 'owns' it. However > at the same time we know that the lifetime of the filesystem is in no > way bounded by the lifetime of the container, and that's what gets you > into trouble with 'umount' in the first place. > > IMO, the current code is the most correct approach, in that it assumes > that the filesystems are owned by the 'init' namespace. IMHO This seems more incorrect than trying to use a more proximal namespace. Cheers, -Matt Helsley -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/