Re: [PATCH 2/3] iomap: decouple simple direct I/O reads from iomap_dio_rw

Christoph Hellwig <[email protected]> Tue, 28 Jul 2026 05:52:17 +0200
Newsgroups org.kernel.vger.linux-ext4,org.kernel.vger.linux-fsdevel,org.kernel.vger.linux-xfs
Message-ID <[email protected]>
On Fri, Jul 24, 2026 at 02:39:49PM -0700, Joanne Koong wrote:
> > > -     ret = iomap_dio_rw(iocb, to, &ext4_iomap_ops, NULL, 0, NULL, 0);
> > > +     ret = iomap_dio_read_simple(iocb, to, ext4_iomap_begin);
> >
> > I was kinda wondering if you can pass ext4_iomap_ops.iomap_begin here
> > to avoid the declaration in ext4.h?
> 
> I think it ends up being a bit cleaner to do it here, otherwise the
> subsequent ext4 patch that replaces .iomap_begin()/.iomap_end() with
> .iomap_next() will have to do it.

Yes, we'd just do the work twice otherwise.