mbox series

[v4,0/3] iomap: zero range flush fixes

Message ID 20241115200155.593665-1-bfoster@redhat.com (mailing list archive)
Headers show
Series iomap: zero range flush fixes | expand

Message

Brian Foster Nov. 15, 2024, 8:01 p.m. UTC
Hi all,

Here's v4 of the zero range flush improvements. No real major changes
here, mostly minor whitespace, naming issues, etc.

Brian

v4:
- Whitespace and variable naming fixes.
- Split off patch 4 to a separate post.
v3: https://lore.kernel.org/linux-fsdevel/20241108124246.198489-1-bfoster@redhat.com/
- Added new patch 1 to always reset per-iter state in iomap_iter.
- Dropped iomap_iter_init() helper.
- Misc. cleanups.
- Appended patch 4 to warn on zeroing beyond EOF.
v2: https://lore.kernel.org/linux-fsdevel/20241031140449.439576-1-bfoster@redhat.com/
- Added patch 1 to lift zeroed mapping handling code into caller.
- Split unaligned start range handling at the top level.
- Retain existing conditional flush behavior (vs. unconditional flush)
  for the remaining range.
v1: https://lore.kernel.org/linux-fsdevel/20241023143029.11275-1-bfoster@redhat.com/

Brian Foster (3):
  iomap: reset per-iter state on non-error iter advances
  iomap: lift zeroed mapping handling into iomap_zero_range()
  iomap: elide flush from partial eof zero range

 fs/iomap/buffered-io.c | 88 +++++++++++++++++++++---------------------
 fs/iomap/iter.c        | 11 +++---
 2 files changed, 50 insertions(+), 49 deletions(-)

Comments

Christian Brauner Nov. 20, 2024, 8:33 a.m. UTC | #1
On Fri, 15 Nov 2024 15:01:52 -0500, Brian Foster wrote:
> Here's v4 of the zero range flush improvements. No real major changes
> here, mostly minor whitespace, naming issues, etc.
> 
> Brian
> 
> v4:
> - Whitespace and variable naming fixes.
> - Split off patch 4 to a separate post.
> v3: https://lore.kernel.org/linux-fsdevel/20241108124246.198489-1-bfoster@redhat.com/
> - Added new patch 1 to always reset per-iter state in iomap_iter.
> - Dropped iomap_iter_init() helper.
> - Misc. cleanups.
> - Appended patch 4 to warn on zeroing beyond EOF.
> v2: https://lore.kernel.org/linux-fsdevel/20241031140449.439576-1-bfoster@redhat.com/
> - Added patch 1 to lift zeroed mapping handling code into caller.
> - Split unaligned start range handling at the top level.
> - Retain existing conditional flush behavior (vs. unconditional flush)
>   for the remaining range.
> v1: https://lore.kernel.org/linux-fsdevel/20241023143029.11275-1-bfoster@redhat.com/
> 
> [...]

Applied to the vfs.fixes branch of the vfs/vfs.git tree.
Patches in the vfs.fixes branch should appear in linux-next soon.

Please report any outstanding bugs that were missed during review in a
new review to the original patch series allowing us to drop it.

It's encouraged to provide Acked-bys and Reviewed-bys even though the
patch has now been applied. If possible patch trailers will be updated.

Note that commit hashes shown below are subject to change due to rebase,
trailer updates or similar. If in doubt, please check the listed branch.

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/vfs/vfs.git
branch: vfs.fixes

[1/3] iomap: reset per-iter state on non-error iter advances
      https://git.kernel.org/vfs/vfs/c/cad47157a55d
[2/3] iomap: lift zeroed mapping handling into iomap_zero_range()
      https://git.kernel.org/vfs/vfs/c/46538f0b405b
[3/3] iomap: elide flush from partial eof zero range
      https://git.kernel.org/vfs/vfs/c/c07ba2d5979b