Re: [f2fs-dev] [PATCH v14 11/21] xfs: don't report dio_mem_align and dio_offset_align for fsverity files
"Darrick J. Wong via Linux-f2fs-devel" <[email protected]> Tue, 4 Aug 2026 10:50:58 -0700
| Newsgroups | net.sourceforge.lists.linux-f2fs-devel,dev.linux.lists.fsverity,org.kernel.vger.linux-btrfs,org.kernel.vger.linux-ext4,org.kernel.vger.linux-fsdevel,org.kernel.vger.linux-xfs |
|---|---|
| Message-ID | <20260804175058.GH3556460@frogsfrogsfrogs> |
On Tue, Aug 04, 2026 at 07:43:47PM +0200, Christoph Hellwig wrote: > On Mon, Aug 03, 2026 at 10:08:01PM +0200, Andrey Albershteyn wrote: > > Sashiko.dev reported that while fsverity files falls back to the > > buffered IO for Direct I/O, they should not report non-zero values in > > dio_mem_align and dio_offset_align, meaning it's not supported. > > This doesn't make much sense to me. If we didn't want to report > we'd also want to not set STATX_DIOALIGN | STATX_DIO_READ_ALIGN. > But in the end there is very little upside of this while adding > extra special cases. directio is supported; the implementation merely falls back to reading through the page cache. Unless you're saying that xfs shouldn't shouldn't set FMODE_CAN_ODIRECT for fsverity files? But that also doesn't sound right. Personally I think sashaniko is full of it and this whole patch should be dropped. --D _______________________________________________ Linux-f2fs-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel