android/android_kernel_xiaomi_sm8450
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fs: allow deduplication of eof block into the end of the destination file

Browse Source 
commit a5e6ea18e3 upstream.

We always round down, to a multiple of the filesystem's block size, the
length to deduplicate at generic_remap_check_len().  However this is only
needed if an attempt to deduplicate the last block into the middle of the
destination file is requested, since that leads into a corruption if the
length of the source file is not block size aligned.  When an attempt to
deduplicate the last block into the end of the destination file is
requested, we should allow it because it is safe to do it - there's no
stale data exposure and we are prepared to compare the data ranges for
a length not aligned to the block (or page) size - in fact we even do
the data compare before adjusting the deduplication length.

After btrfs was updated to use the generic helpers from VFS (by commit
34a28e3d77 ("Btrfs: use generic_remap_file_range_prep() for cloning
and deduplication")) we started to have user reports of deduplication
not reflinking the last block anymore, and whence users getting lower
deduplication scores.  The main use case is deduplication of entire
files that have a size not aligned to the block size of the filesystem.

We already allow cloning the last block to the end (and beyond) of the
destination file, so allow for deduplication as well.

Link: https://lore.kernel.org/linux-btrfs/2019-1576167349.500456@svIo.N5dq.dFFD/
CC: stable@vger.kernel.org # 5.1+
Reviewed-by: Josef Bacik <josef@toxicpanda.com>
Reviewed-by: Darrick J. Wong <darrick.wong@oracle.com>
Signed-off-by: Filipe Manana <fdmanana@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
Filipe Manana 2019-12-16 18:26:55 +00:00 committed by Greg Kroah-Hartman
parent 5fefc9b3e3
commit a421f51377
1 changed files with 4 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
fs/read_write.c
View File

@ -1777,10 +1777,9 @@ static int remap_verify_area(struct file *file, loff_t pos, loff_t len,
* else. Assume that the offsets have already been checked for block * else. Assume that the offsets have already been checked for block
* alignment. * alignment.
* *
* For deduplication we always scale down to the previous block because we * For clone we only link a partial EOF block above or at the destination file's
* can't meaningfully compare post-EOF contents. * EOF. For deduplication we accept a partial EOF block only if it ends at the
* * destination file's EOF (can not link it into the middle of a file).
* For clone we only link a partial EOF block above the destination file's EOF.
* *
* Shorten the request if possible. * Shorten the request if possible.
*/ */
@ -1796,8 +1795,7 @@ static int generic_remap_check_len(struct inode *inode_in,
if ((*len & blkmask) == 0) if ((*len & blkmask) == 0)
return 0; return 0;
if ((remap_flags & REMAP_FILE_DEDUP) || if (pos_out + *len < i_size_read(inode_out))
pos_out + *len < i_size_read(inode_out))
new_len &= ~blkmask; new_len &= ~blkmask;
if (new_len == *len) if (new_len == *len)