diff options
author | David Howells <dhowells@redhat.com> | 2024-05-24 15:23:36 +0100 |
---|---|---|
committer | Steve French <stfrench@microsoft.com> | 2024-05-24 16:05:56 -0500 |
commit | 93a43155127fec0f8cc942d63b76668c2f8f69fa (patch) | |
tree | 7e6fd59147796ca5b181549e650c14503d787e72 /include | |
parent | 8a1607233566fef7fbe30416e5d3b0567d0d1fba (diff) | |
download | lwn-93a43155127fec0f8cc942d63b76668c2f8f69fa.tar.gz lwn-93a43155127fec0f8cc942d63b76668c2f8f69fa.zip |
cifs: Fix missing set of remote_i_size
Occasionally, the generic/001 xfstest will fail indicating corruption in
one of the copy chains when run on cifs against a server that supports
FSCTL_DUPLICATE_EXTENTS_TO_FILE (eg. Samba with a share on btrfs). The
problem is that the remote_i_size value isn't updated by cifs_setsize()
when called by smb2_duplicate_extents(), but i_size *is*.
This may cause cifs_remap_file_range() to then skip the bit after calling
->duplicate_extents() that sets sizes.
Fix this by calling netfs_resize_file() in smb2_duplicate_extents() before
calling cifs_setsize() to set i_size.
This means we don't then need to call netfs_resize_file() upon return from
->duplicate_extents(), but we also fix the test to compare against the pre-dup
inode size.
[Note that this goes back before the addition of remote_i_size with the
netfs_inode struct. It should probably have been setting cifsi->server_eof
previously.]
Fixes: cfc63fc8126a ("smb3: fix cached file size problems in duplicate extents (reflink)")
Signed-off-by: David Howells <dhowells@redhat.com>
cc: Steve French <sfrench@samba.org>
cc: Paulo Alcantara <pc@manguebit.com>
cc: Shyam Prasad N <nspmangalore@gmail.com>
cc: Rohith Surabattula <rohiths.msft@gmail.com>
cc: Jeff Layton <jlayton@kernel.org>
cc: linux-cifs@vger.kernel.org
cc: netfs@lists.linux.dev
Signed-off-by: Steve French <stfrench@microsoft.com>
Diffstat (limited to 'include')
0 files changed, 0 insertions, 0 deletions