Re: remove get_super
- To: David Sterba <dsterba@suse.cz>
- Cc: Christoph Hellwig <hch@lst.de>, Al Viro <viro@zeniv.linux.org.uk>, Christian Brauner <brauner@kernel.org>, Jens Axboe <axboe@kernel.dk>, Denis Efremov <efremov@linux.com>, Josef Bacik <josef@toxicpanda.com>, Stefan Haberland <sth@linux.ibm.com>, Jan Hoeppner <hoeppner@linux.ibm.com>, Heiko Carstens <hca@linux.ibm.com>, Vasily Gorbik <gor@linux.ibm.com>, Alexander Gordeev <agordeev@linux.ibm.com>, "Darrick J . Wong" <djwong@kernel.org>, Chris Mason <clm@fb.com>, David Sterba <dsterba@suse.com>, linux-block@vger.kernel.org, nbd@other.debian.org, linux-s390@vger.kernel.org, linux-btrfs@vger.kernel.org, linux-fsdevel@vger.kernel.org
- Subject: Re: remove get_super
- From: Jan Kara <jack@suse.cz>
- Date: Thu, 14 Sep 2023 10:48:09 +0200
- Message-id: <[🔎] 20230914084809.arzw34svsvvkwivm@quack3>
- In-reply-to: <[🔎] 20230912174245.GC20408@twin.jikos.cz>
- References: <20230811100828.1897174-1-hch@lst.de> <[🔎] 20230912174245.GC20408@twin.jikos.cz>
On Tue 12-09-23 19:42:45, David Sterba wrote:
> On Fri, Aug 11, 2023 at 12:08:11PM +0200, Christoph Hellwig wrote:
> > Hi all,
> >
> > this series against the VFS vfs.super branch finishes off the work to remove
> > get_super and move (almost) all upcalls to use the holder ops.
> >
> > The first part is the missing btrfs bits so that all file systems use the
> > super_block as holder.
> >
> > The second part is various block driver cleanups so that we use proper
> > interfaces instead of raw calls to __invalidate_device and fsync_bdev.
> >
> > The last part than replaces __invalidate_device and fsync_bdev with upcalls
> > to the file system through the holder ops, and finally removes get_super.
> >
> > It leaves user_get_super and get_active_super around. The former is not
> > used for upcalls in the traditional sense, but for legacy UAPI that for
> > some weird reason take a dev_t argument (ustat) or a block device path
> > (quotactl). get_active_super is only used for calling into the file system
> > on freeze and should get a similar treatment, but given that Darrick has
> > changes to that code queued up already this will be handled in the next
> > merge window.
> >
> > A git tree is available here:
> >
> > git://git.infradead.org/users/hch/misc.git remove-get_super
>
> FYI, I've added patches 2-5 as a topic branch to btrfs for-next.
Hum, I don't see them there. Some glitch somewhere?
Honza
--
Jan Kara <jack@suse.com>
SUSE Labs, CR
Reply to: