From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from cantor2.suse.de ([195.135.220.15]:50252 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751207Ab3IJPg7 (ORCPT ); Tue, 10 Sep 2013 11:36:59 -0400 Date: Tue, 10 Sep 2013 08:36:55 -0700 From: Mark Fasheh To: Christoph Hellwig Cc: Josef Bacik , viro@ZenIV.linux.org.uk, linux-fsdevel@vger.kernel.org, linux-btrfs@vger.kernel.org, Chris Mason , Andrew Vagin Subject: Re: [PATCH][RESEND] vfs: allow /proc/PID/maps to get device from stat Message-ID: <20130910153655.GH31381@wotan.suse.de> Reply-To: Mark Fasheh References: <20130807195718.GC31381@wotan.suse.de> <20130807201826.GA23804@infradead.org> <20130807205146.GE2397@localhost.localdomain> <20130808121349.GC5180@infradead.org> <20130808130207.GA16712@localhost.localdomain> <20130808134805.GB24181@infradead.org> <20130808154454.GF16712@localhost.localdomain> <20130812114752.GA2390@infradead.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <20130812114752.GA2390@infradead.org> Sender: linux-btrfs-owner@vger.kernel.org List-ID: On Mon, Aug 12, 2013 at 04:47:52AM -0700, Christoph Hellwig wrote: > On Thu, Aug 08, 2013 at 11:44:54AM -0400, Josef Bacik wrote: > > On Thu, Aug 08, 2013 at 06:48:05AM -0700, Christoph Hellwig wrote: > > > On Thu, Aug 08, 2013 at 09:02:07AM -0400, Josef Bacik wrote: > > > > This won't work, try having 10000 subvolumes with dirty inodes and do sync then > > > > go skiing, you'll have time :). Thanks, > > > > > > Why would the dirty inodes make any difference? If you share the bdi > > > between the subvolumes the sync workflow should be exactly the same > > > still. > > > > > > > If we could dis-entangle vfsmounts from sb's and have it so you could have > > multiple vfsmounts with just one sb that would solve at least the in-kernel > > confusion, but I think we still have the userspace confusion. Thanks, > > I think it would mostly solve userspace confusion, as userspace only > sees mounts and the device names. > > But please fix this up properly instead of propagating the effects of > the nasty btrfs hack that should never have been merged in that form > further up the stack. Can one of you explain how this solves the problem that userspace is getting different devices for the same inode? Seriously, I've been looking into it and I'm a bit lost. I followed the converstaion until here but I don't see how any of the proposed changes actually *fix* anything? Also, what is the relationship between vfsmounts and sb today? Wouldn't a bind mount produce the situation of more than 1 vfsmount per sb that is described above? Sincerely, someone who would like to fix this ABI breakage that has been going on for years. --Mark -- Mark Fasheh