On Thu, Jun 25, 2026 at 10:27:40AM -0700, Darrick J. Wong wrote: > > > > + bio->bi_end_io = end_io; > > if (iter->iomap.flags & IOMAP_F_INTEGRITY) > > fs_bio_integrity_alloc(bio); > > Ah, so the bug here is that all the pagecache readers should have been > allocating integrity information for the bio before submitting it?
Well, all the ones that set IOMAP_F_INTEGRITY, which really is just XFS at the momen. And because if the iomap refactoring for fuse XFS is now using it's own submit callback and does not get it.. > And > because it doesn't, iomap_finish_ioend won't do the read verification? Yeah. > So the block layer does it for us, and that's why we don't use the ioend > chaining? And (I guess) the future userspace interface won't have any > means to get at the integrity data? Yeah.
