On Mon, Apr 05, 2021 at 06:55:33PM +0100, Christoph Hellwig wrote: > On Mon, Apr 05, 2021 at 10:59:02AM -0400, Brian Foster wrote: > > io_list))) { > > list_del_init(&ioend->io_list); > > - iomap_ioend_try_merge(ioend, &tmp, xfs_ioend_merge_private); > > + iomap_ioend_try_merge(ioend, &tmp, NULL); > > The merge_private argument to iomap_ioend_try_merge and the io_private > field in struct ioend can go way now as well. > Indeed. I'll tack on another patch to remove all of that. Brian > Otherwise the whole series looks good to me from a very cursory look. >