Date: Tue, 7 Mar 2006 06:35:14 -0500
From: Alexander Viro <aviro@redhat.com>
To: David Howells <dhowells@redhat.com>
Cc: Linux filesystem caching discussion list 
	<linux-cachefs@redhat.com>,
       torvalds@osdl.org, akpm@osdl.org, steved@redhat.com,
       trond.myklebust@fys.uio.no, aviro@redhat.com,
       linux-fsdevel@vger.kernel.org, nfsv4@linux-nfs.org,
       linux-kernel@vger.kernel.org
Subject: Re: [PATCH] Fix multiple blockdev-based filesystem mounts
Message-ID: <20060307113514.GX20691@devserv.devel.redhat.com>
References: <20060302213356.7282.26463.stgit@warthog.cambridge.redhat.com> <15962.1141729721@warthog.cambridge.redhat.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <15962.1141729721@warthog.cambridge.redhat.com>
User-Agent: Mutt/1.4.1i
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1126
Lines: 26

On Tue, Mar 07, 2006 at 11:08:41AM +0000, David Howells wrote:
> 
> The attached patch fixes multiple mounts of the same blockdev-based filesystem
> (such as EXT3). The problem is that the path through the function that deals
> with a second mount of an already extant superblock is going out through the
> error path and not the success path that sets up the vfsmount.
> 
> The error can be checked by doing something like:
> 
> 	mount /dev/hda6 /a
> 	mount /dev/hda6 /b
> 
> Where /dev/hda6 has something like an EXT3 filesystem on it. The second mount
> should succeed, but doesn't without this patch.
> 
> This patch is dependent on the getsb patch submitted earlier.
> 
> Signed-Off-By: David Howells <dhowells@redhat.com>

Please, merge them in one; this one fixes a bug in ->get_sb() patch and since
it wasn't merged in mainline yet, there's no reason to keep them separate.
-
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/