From mboxrd@z Thu Jan 1 00:00:00 1970 From: Steven Whitehouse Date: Wed, 12 Apr 2017 09:48:05 +0100 Subject: [Cluster-devel] [PATCH 18/25] gfs2: Convert to properly refcounting bdi In-Reply-To: <20170412081641.GR9830@infradead.org> References: <20170329105623.18241-1-jack@suse.cz> <20170329105623.18241-19-jack@suse.cz> <20170412081641.GR9830@infradead.org> Message-ID: List-Id: To: cluster-devel.redhat.com MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Hi, On 12/04/17 09:16, Christoph Hellwig wrote: > On Wed, Mar 29, 2017 at 12:56:16PM +0200, Jan Kara wrote: >> Similarly to set_bdev_super() GFS2 just used block device reference to >> bdi. Convert it to properly getting bdi reference. The reference will >> get automatically dropped on superblock destruction. > Hmm, why iisn't gfs2 simply using the generic mount_bdev code? > > Otherwise looks fine: > > Reviewed-by: Christoph Hellwig It is more or less. However we landed up copying it because we needed a slight modification in order to cope with the metafs mounts. There may be scope to factor out the common parts I guess. We cannot select the root dentry until after we've parsed the mount command line, so it is really just the last part of the function that is different, Steve.