btrfs: compression: don't try to compress if we don't have enough pages
authorDavid Sterba <dsterba@suse.com>
Mon, 14 Jun 2021 10:45:18 +0000 (12:45 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 14 Jul 2021 14:55:40 +0000 (16:55 +0200)
commit6b00b1717f588089ea6f18d41cf84f9156517143
tree3c9862d9cd1d5e621e7d6e5c4137878672eee439
parent34172f601a9c4f360d18729d295343be5a50545a
btrfs: compression: don't try to compress if we don't have enough pages

commit f2165627319ffd33a6217275e5690b1ab5c45763 upstream.

The early check if we should attempt compression does not take into
account the number of input pages. It can happen that there's only one
page, eg. a tail page after some ranges of the BTRFS_MAX_UNCOMPRESSED
have been processed, or an isolated page that won't be converted to an
inline extent.

The single page would be compressed but a later check would drop it
again because the result size must be at least one block shorter than
the input. That can never work with just one page.

CC: stable@vger.kernel.org # 4.4+
Signed-off-by: David Sterba <dsterba@suse.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
fs/btrfs/inode.c