Commit dbfdff40 authored Feb 22, 2024 by Steve French

smb3: update allocation size more accurately on write completion

Changes to allocation size are approximated for extending writes of cached
files until the server returns the actual value (on SMB3 close or query info
for example), but it was setting the estimated value for number of blocks
to larger than the file size even if the file is likely sparse which
breaks various xfstests (e.g. generic/129, 130, 221, 228).

When i_size and i_blocks are updated in write completion do not increase
allocation size more than what was written (rounded up to 512 bytes).

Signed-off-by: Steve French <stfrench@microsoft.com>

parent 13c2e308

fs/smb/client/file.c

+8 −1

Original line number	Diff line number	Diff line
		@@ -3094,8 +3094,15 @@ static int cifs_write_end(struct file file, struct address_space mapping,
		if (rc > 0) {
		spin_lock(&inode->i_lock);
		if (pos > inode->i_size) {
		loff_t additional_blocks = (512 - 1 + copied) >> 9;

		i_size_write(inode, pos);
		inode->i_blocks = (512 - 1 + pos) >> 9;
		/*
		* Estimate new allocation size based on the amount written.
		* This will be updated from server on close (and on queryinfo)
		*/
		inode->i_blocks = min_t(blkcnt_t, (512 - 1 + pos) >> 9,
		inode->i_blocks + additional_blocks);
		}
		spin_unlock(&inode->i_lock);
		}