From mboxrd@z Thu Jan 1 00:00:00 1970 From: Al Viro Subject: Re: [PATCH 0/6 v7] overlay filesystem - request for inclusion Date: Tue, 22 Mar 2011 19:00:02 +0000 Message-ID: <20110322190002.GW22723@ZenIV.linux.org.uk> References: <20110322152602.053930811@szeredi.hu> <20110322183919.GV22723@ZenIV.linux.org.uk> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Linus Torvalds , linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org, akpm@linux-foundation.org, apw@canonical.com, nbd@openwrt.org, neilb@suse.de To: Miklos Szeredi Return-path: Received: from zeniv.linux.org.uk ([195.92.253.2]:50939 "EHLO ZenIV.linux.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751705Ab1CVTAF (ORCPT ); Tue, 22 Mar 2011 15:00:05 -0400 Content-Disposition: inline In-Reply-To: Sender: linux-fsdevel-owner@vger.kernel.org List-ID: On Tue, Mar 22, 2011 at 07:58:17PM +0100, Miklos Szeredi wrote: > On Tue, 22 Mar 2011, Linus Torvalds wrote: > > On Tue, Mar 22, 2011 at 11:39 AM, Al Viro wrote: > > > > > > Locking analysis would be really nice; AFAICS, it violates locking order > > > when called from e.g. ->setattr() > > Locking order is always: > > -> overlayfs locks > -> upper fs locks > -> lower fs locks > > So it's really pretty simple and easy to validate. In which *order* on the upper fs? > Protection is exactly as for userspace callers. AFAICT. Pardon? You traverse the chain of ancestors; fine, but who says it stays anywhere near being relevant as you go?