diff options
| author | Kent Overstreet <kent.overstreet@linux.dev> | 2023-10-22 11:19:34 -0400 |
|---|---|---|
| committer | Kent Overstreet <kent.overstreet@linux.dev> | 2023-10-31 12:18:37 -0400 |
| commit | 2d39081291470750cc605c917531d7cd85aebf94 (patch) | |
| tree | 04626dd638b12ed50c047e04a09d4e3eb564612f /fs/bcachefs/io_write.c | |
| parent | e677179b35b7ecbe3cefe33011b69d45171e5e9f (diff) | |
| download | lwn-2d39081291470750cc605c917531d7cd85aebf94.tar.gz lwn-2d39081291470750cc605c917531d7cd85aebf94.zip | |
bcachefs: Ensure we don't exceed encoded_extent_max
The write path may (rarely) see an encoded (checksummed) extent that
exceeds encoded_extent_max - this can happen when we're moving an
existing extent that was not checksummed, but was given a checksum by
bch2_write_rechecksum().
Signed-off-by: Kent Overstreet <kent.overstreet@linux.dev>
Diffstat (limited to 'fs/bcachefs/io_write.c')
| -rw-r--r-- | fs/bcachefs/io_write.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/fs/bcachefs/io_write.c b/fs/bcachefs/io_write.c index 6e4f85eb6ec8..4a666f4d2dcc 100644 --- a/fs/bcachefs/io_write.c +++ b/fs/bcachefs/io_write.c @@ -816,6 +816,7 @@ static enum prep_encoded_ret { /* Can we just write the entire extent as is? */ if (op->crc.uncompressed_size == op->crc.live_size && + op->crc.uncompressed_size <= c->opts.encoded_extent_max >> 9 && op->crc.compressed_size <= wp->sectors_free && (op->crc.compression_type == bch2_compression_opt_to_type(op->compression_opt) || op->incompressible)) { |