From mboxrd@z Thu Jan 1 00:00:00 1970 From: Al Viro Subject: Re: [patch 00/13] vfs: add helpers to check r/o bind mounts Date: Thu, 24 Apr 2008 15:48:15 +0100 Message-ID: <20080424144815.GI15214@ZenIV.linux.org.uk> References: <20080424113950.818688067@szeredi.hu> <20080424124245.GC15214@ZenIV.linux.org.uk> <20080424134826.GD15214@ZenIV.linux.org.uk> <20080424140000.GE15214@ZenIV.linux.org.uk> <20080424143537.GG15214@ZenIV.linux.org.uk> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: akpm@linux-foundation.org, torvalds@linux-foundation.org, dave@linux.vnet.ibm.com, ezk@cs.sunysb.edu, mhalcrow@us.ibm.com, linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org To: Miklos Szeredi Return-path: Received: from zeniv.linux.org.uk ([195.92.253.2]:47382 "EHLO ZenIV.linux.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752245AbYDXOsU (ORCPT ); Thu, 24 Apr 2008 10:48:20 -0400 Content-Disposition: inline In-Reply-To: Sender: linux-fsdevel-owner@vger.kernel.org List-ID: On Thu, Apr 24, 2008 at 04:42:52PM +0200, Miklos Szeredi wrote: > > You are right - we do have races there. Always had. > > And nfsd_permission() is the next target for continuation of ro-bind > > series. Assuming that we don't simply make r/w export to hold will_write > > all along, in which case all these checks around calls of vfs_...() in > > there simply go away - that's also an arguable option. > > Yes. And that _still_ doesn't make the path_*() interface wrong. It would make it bloody useless for nfsd. With ecryptfs being a piss-poor argument in favour of anything other than git rm, what's left? Another stack frame in fs/namei.c syscalls?