iomap: don't lose folio dropbehind state for overwrites dontcache
authorJens Axboe <axboe@kernel.dk>
Tue, 27 May 2025 15:35:12 +0000 (09:35 -0600)
committerJens Axboe <axboe@kernel.dk>
Tue, 27 May 2025 21:19:54 +0000 (15:19 -0600)
commit3be5f84e5ae9807ae3643611935fcb0b8f74620f
tree14820456e04cdf54ddffa368ea948bb944d0272e
parent570175af81244d3eac781e5c4e65f1c6a2801e84
iomap: don't lose folio dropbehind state for overwrites

DONTCACHE I/O must have the completion punted to a workqueue, just like
what is done for unwritten extents, as the completion needs task context
to perform the invalidation of the folio(s). However, if writeback is
started off filemap_fdatawrite_range() off generic_sync() and it's an
overwrite, then the DONTCACHE marking gets lost as iomap_add_to_ioend()
don't look at the folio being added and no further state is passed down
to help it know that this is a dropbehind/DONTCACHE write.

Check if the folio being added is marked as dropbehind, and set
IOMAP_IOEND_DONTCACHE if that is the case. Then XFS can factor this into
the decision making of completion context in xfs_submit_ioend().
Additionally include this ioend flag in the NOMERGE flags, to avoid
mixing it with unrelated IO.

Since this is the 3rd flag that will cause XFS to punt the completion to
a workqueue, add a helper so that each one of them can get appropriately
commented.

This fixes extra page cache being instantiated when the write performed
is an overwrite, rather than newly instantiated blocks.

Fixes: b2cd5ae693a3 ("iomap: make buffered writes work with RWF_DONTCACHE")
Signed-off-by: Jens Axboe <axboe@kernel.dk>
fs/iomap/buffered-io.c
fs/xfs/xfs_aops.c
include/linux/iomap.h