]> git.baikalelectronics.ru Git - kernel.git/commit
cifs: truncate the inode and mapping when we simulate fcollapse
authorRonnie Sahlberg <lsahlber@redhat.com>
Wed, 23 Feb 2022 01:14:16 +0000 (11:14 +1000)
committerSteve French <stfrench@microsoft.com>
Sat, 19 Mar 2022 04:06:06 +0000 (23:06 -0500)
commit914c251cace25acb70633936be24b1a87fd4a4a2
tree33daee729f9ebc734034dd4bc77a79914f0d8482
parentd8f0a7b2939d0694493b9bcdc6d6d967269f5b6c
cifs: truncate the inode and mapping when we simulate fcollapse

RHBZ:1997367

When we collapse a range in smb3_collapse_range() we must make sure
we update the inode size and pagecache accordingly.

If not, both inode size and pagecahce may be stale until it is refreshed.

This can be demonstrated for the inode size by running :

xfs_io -i -f -c "truncate 320k" -c "fcollapse 64k 128k" -c "fiemap -v"  \
/mnt/testfile

where we can see the result of stale data in the fiemap output.
The third line of the output is wrong, all this data should be truncated.

 EXT: FILE-OFFSET      BLOCK-RANGE      TOTAL FLAGS
   0: [0..127]:        hole               128
   1: [128..383]:      128..383           256   0x1
   2: [384..639]:      hole               256

And the correct output, when the inode size has been updated correctly should
look like this:

 EXT: FILE-OFFSET      BLOCK-RANGE      TOTAL FLAGS
   0: [0..127]:        hole               128
   1: [128..383]:      128..383           256   0x1

Reported-by: Xiaoli Feng <xifeng@redhat.com>
Reported-by: kernel test robot <lkp@intel.com>
Cc: stable@vger.kernel.org
Signed-off-by: Ronnie Sahlberg <lsahlber@redhat.com>
Signed-off-by: Steve French <stfrench@microsoft.com>
fs/cifs/smb2ops.c