btrfs/035: check for data loss
authorFilipe Manana <fdmanana@suse.com>
Wed, 14 Oct 2015 03:19:34 +0000 (14:19 +1100)
committerDave Chinner <david@fromorbit.com>
Wed, 14 Oct 2015 03:19:34 +0000 (14:19 +1100)
commit0e6ead559169260d0a2621ec22edcd0e63b84a88
tree99fb5f364ef857162883907abab1d344b5ec5975
parentddb4e4cfccfa5fc36975c12e9a66a24d3b7829bd
btrfs/035: check for data loss

The test currently verifies that cloning one file with an inline extent
with a size of 10 bytes into a file with an inline extent that has a size
of 20 bytes succeeds. But this results in data loss, because the btrfs
clone operation drops the 20 bytes inline extent from the destination
inode and then copies the 10 bytes inline extent from the source file
into the destination file, resulting in data loss of the last 10 bytes
of data that the destination file had.

Fixing btrfs to correctly operate for this case (not resulting in data
loss) is actually a lot of work and brings a lot of complexity, specially
considering that any of the inline extents can be compressed. For the
moment there's a fix to make the clone operation return the errno
EOPNOTSUPP and not touch any of the inodes. This is the same approach
we do for other cases involving operation against inline extents, so
this just adds one more case that should have never been allowed.
Cloning inline extents is a rare operation and pointless, since it
involves copying them and not doing any actual deduplication or saving
space.

The btrfs patch for the linux kernel that prevents this data loss,
and fixes some file corruption cases, is titled:

  "Btrfs: fix file corruption and data loss after cloning inline extents"

Signed-off-by: Filipe Manana <fdmanana@suse.com>
Reviewed-by: Dave Chinner <dchinner@redhat.com>
Signed-off-by: Dave Chinner <david@fromorbit.com>
tests/btrfs/035
tests/btrfs/035.out