On Wed, Apr 24, 2019 at 11:46:55AM +0300, Nikolay Borisov wrote:
> btrfs_trim_free_extents always caps the range it's going to trim based
> on the size of the device. This happens if find_first_clear_extent_bit
> detects that untrimmed range is past the last allocated range. Since it
> doesn't have knowledge of the size of the device it just returns (u64)-1
> for end. Then btrfs_trim_free_extents caps this to device->total_bytes.
> However, btrfs_trim_free_extent calculates 'len' based off of the
> actual usable end byte - in this case total_bytes - 1.
> 
> Signed-off-by: Nikolay Borisov <nbori...@suse.com>
> 
> Fixes: 4d877687cbbc ("btrfs: Switch btrfs_trim_free_extents to 
> find_first_clear_extent_bit")
> ---
> 
> David you might want to squash that in the 'Fixes' commit. With this I see 
> generic/500 passing and also full xfstest didn't uncover any new regressions. 

Folded, thanks.

Reply via email to