• Qu Wenruo's avatar
    btrfs: defrag: remove an ambiguous condition for rejection · 550f133f
    Qu Wenruo authored
    From the very beginning of btrfs defrag, there is a check to reject
    extents which meet both conditions:
    
    - Physically adjacent
    
      We may want to defrag physically adjacent extents to reduce the number
      of extents or the size of subvolume tree.
    
    - Larger than 128K
    
      This may be there for compressed extents, but unfortunately 128K is
      exactly the max capacity for compressed extents.
      And the check is > 128K, thus it never rejects compressed extents.
    
      Furthermore, the compressed extent capacity bug is fixed by previous
      patch, there is no reason for that check anymore.
    
    The original check has a very small ranges to reject (the target extent
    size is > 128K, and default extent threshold is 256K), and for
    compressed extent it doesn't work at all.
    
    So it's better just to remove the rejection, and allow us to defrag
    physically adjacent extents.
    
    CC: stable@vger.kernel.org # 5.16
    Reviewed-by: default avatarFilipe Manana <fdmanana@suse.com>
    Signed-off-by: default avatarQu Wenruo <wqu@suse.com>
    Signed-off-by: default avatarDavid Sterba <dsterba@suse.com>
    550f133f
ioctl.c 129 KB