fuse: keep inode->i_blkbits constant
authorJoanne Koong <joannelkoong@gmail.com>
Thu, 7 Aug 2025 17:50:15 +0000 (10:50 -0700)
committerChristian Brauner <brauner@kernel.org>
Mon, 11 Aug 2025 12:51:50 +0000 (14:51 +0200)
With fuse now using iomap for writeback handling, inode blkbits changes
are problematic because iomap relies on inode->i_blkbits for its
internal bitmap logic. Currently we change inode->i_blkbits in fuse to
match the attr->blksize value passed in by the server.

This commit keeps inode->i_blkbits constant in fuse. Any attr->blksize
values passed in by the server will not update inode->i_blkbits. The
client-side behavior for stat is unaffected, stat will still reflect the
blocksize passed in by the server.

Signed-off-by: Joanne Koong <joannelkoong@gmail.com>
Link: https://lore.kernel.org/20250807175015.515192-1-joannelkoong@gmail.com
Fixes: ef7e7cbb32 ("fuse: use iomap for writeback")
Signed-off-by: Christian Brauner <brauner@kernel.org>
fs/fuse/inode.c

index ecb869e895ab1d644d88868290f4a7dc69e36a2c..67c2318bfc42941602345ca6b626801845241417 100644 (file)
@@ -289,11 +289,6 @@ void fuse_change_attributes_common(struct inode *inode, struct fuse_attr *attr,
                }
        }
 
-       if (attr->blksize != 0)
-               inode->i_blkbits = ilog2(attr->blksize);
-       else
-               inode->i_blkbits = inode->i_sb->s_blocksize_bits;
-
        /*
         * Don't set the sticky bit in i_mode, unless we want the VFS
         * to check permissions.  This prevents failures due to the