diff options
| author | Joanne Koong <joannelkoong@gmail.com> | 2025-07-23 16:08:50 -0700 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2025-07-28 16:14:18 -0700 |
| commit | 595d7ebeaf39ee08180e00fdecf3576a2d702e01 (patch) | |
| tree | 30057c4f57656aff5b0a57f9cb73bf37774878f2 /fs/fuse | |
| parent | Merge tag 'vfs-6.17-rc1.iomap' of git://git.kernel.org/pub/scm/linux/kernel/g... (diff) | |
| download | linux-595d7ebeaf39ee08180e00fdecf3576a2d702e01.tar.gz linux-595d7ebeaf39ee08180e00fdecf3576a2d702e01.zip | |
fuse: remove page alignment check for writeback len
Remove incorrect page alignment check for the writeback len arg in
fuse_iomap_writeback_range(). len will always be block-aligned as
passed in by iomap.
On regular fuse filesystems, i_blkbits is set to PAGE_SHIFT so this is
not a problem but for fuseblk filesystems, the block size is set to a
default of 512 bytes or a block size passed in at mount time.
Please note that non-page-aligned lengths are fine for the logic in
fuse_iomap_writeback_range(). The check was originally added as a
safeguard to detect conspicuously wrong ranges.
Signed-off-by: Joanne Koong <joannelkoong@gmail.com>
Fixes: ef7e7cbb323f ("fuse: use iomap for writeback")
Reported-by: Linux Kernel Functional Testing <lkft@linaro.org>
Link: https://lore.kernel.org/linux-fsdevel/CA+G9fYs5AdVM-T2Tf3LciNCwLZEHetcnSkHsjZajVwwpM2HmJw@mail.gmail.com/
Reported-by: Sasha Levin <sashal@kernel.org>
Reviewed-by: Darrick J. Wong <djwong@kernel.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'fs/fuse')
| -rw-r--r-- | fs/fuse/file.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/fs/fuse/file.c b/fs/fuse/file.c index 617fd1b562fd..95275a1e2f54 100644 --- a/fs/fuse/file.c +++ b/fs/fuse/file.c @@ -2154,8 +2154,6 @@ static ssize_t fuse_iomap_writeback_range(struct iomap_writepage_ctx *wpc, loff_t offset = offset_in_folio(folio, pos); WARN_ON_ONCE(!data); - /* len will always be page aligned */ - WARN_ON_ONCE(len & (PAGE_SIZE - 1)); if (!data->ff) { data->ff = fuse_write_file_get(fi); |