linux.git/fs/btrfs/compression.c, branch v5.9

btrfs: remove fail label in check_compressed_csum

2020-07-27T10:55:42+00:00

Signed-off-by: Nikolay Borisov 
Reviewed-by: David Sterba 
Signed-off-by: David Sterba

btrfs: increment corrupt device counter during compressed read

2020-07-27T10:55:40+00:00

If a compressed read fails due to checksum error only a line is printed
to dmesg, device corrupt counter is not modified.

Reviewed-by: Josef Bacik 
Reviewed-by: Johannes Thumshirn 
Signed-off-by: Nikolay Borisov 
Reviewed-by: David Sterba 
Signed-off-by: David Sterba

btrfs: remove needless ASSERT check of orig_bio in end_compressed_bio_read

2020-07-27T10:55:40+00:00

compressed_bio::orig_bio is always set in btrfs_submit_compressed_read
before any bio submission is performed. Since that function is always
called with a valid bio it renders the ASSERT unnecessary.

Reviewed-by: Josef Bacik 
Reviewed-by: Johannes Thumshirn 
Signed-off-by: Nikolay Borisov 
Reviewed-by: David Sterba 
Signed-off-by: David Sterba

btrfs: make btrfs_submit_compressed_write take btrfs_inode

2020-07-27T10:55:31+00:00

Majority of its uses are for btrfs_inode so take it as an argument
directly.

Signed-off-by: Nikolay Borisov 
Reviewed-by: David Sterba 
Signed-off-by: David Sterba

btrfs: make btrfs_csum_one_bio takae btrfs_inode

2020-07-27T10:55:26+00:00

Will enable converting btrfs_submit_compressed_write to btrfs_inode more
easily.

Signed-off-by: Nikolay Borisov 
Reviewed-by: David Sterba 
Signed-off-by: David Sterba

btrfs: unexport btrfs_compress_set_level()

2020-05-25T09:25:37+00:00

btrfs_compress_set_level() can be static function in the file
compression.c.

Reviewed-by: Johannes Thumshirn 
Signed-off-by: Anand Jain 
Reviewed-by: David Sterba 
Signed-off-by: David Sterba

btrfs: use crypto_shash_digest() instead of open coding

2020-05-25T09:25:27+00:00

Use crypto_shash_digest() instead of crypto_shash_init() +
crypto_shash_update() + crypto_shash_final().  This is more efficient.

Signed-off-by: Eric Biggers 
Reviewed-by: David Sterba 
Signed-off-by: David Sterba

btrfs: use larger zlib buffer for s390 hardware compression

2020-01-31T18:30:40+00:00

In order to benefit from s390 zlib hardware compression support,
increase the btrfs zlib workspace buffer size from 1 to 4 pages (if s390
zlib hardware support is enabled on the machine).

This brings up to 60% better performance in hardware on s390 compared to
the PAGE_SIZE buffer and much more compared to the software zlib
processing in btrfs.  In case of memory pressure, fall back to a single
page buffer during workspace allocation.

The data compressed with larger input buffers will still conform to zlib
standard and thus can be decompressed also on a systems that uses only
PAGE_SIZE buffer for btrfs zlib.

Link: http://lkml.kernel.org/r/20200108105103.29028-1-zaslonko@linux.ibm.com
Signed-off-by: Mikhail Zaslonko 
Reviewed-by: David Sterba 
Cc: Chris Mason 
Cc: Josef Bacik 
Cc: David Sterba 
Cc: Richard Purdie 
Cc: Heiko Carstens 
Cc: Vasily Gorbik 
Cc: Christian Borntraeger 
Cc: Eduard Shishkin 
Cc: Ilya Leoshkevich 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

btrfs: get rid of at_offset parameter to btrfs_lookup_bio_sums()

2020-01-20T15:40:54+00:00

We can encode this in the offset parameter: -1 means use the page
offsets, anything else is a valid offset.

Signed-off-by: Omar Sandoval 
Signed-off-by: David Sterba

btrfs: get rid of trivial __btrfs_lookup_bio_sums() wrappers

2020-01-20T15:40:53+00:00

Currently, we have two wrappers for __btrfs_lookup_bio_sums():
btrfs_lookup_bio_sums_dio(), which is used for direct I/O, and
btrfs_lookup_bio_sums(), which is used everywhere else. The only
difference is that the _dio variant looks up csums starting at the given
offset instead of using the page index, which isn't actually direct
I/O-specific. Let's clean up the signature and return value of
__btrfs_lookup_bio_sums(), rename it to btrfs_lookup_bio_sums(), and get
rid of the trivial helpers.

Reviewed-by: Nikolay Borisov 
Signed-off-by: Omar Sandoval 
Signed-off-by: David Sterba