diff options
author | Naohiro Aota <naohiro.aota@wdc.com> | 2024-03-26 14:39:21 +0900 |
---|---|---|
committer | David Sterba <dsterba@suse.com> | 2024-04-09 23:20:29 +0200 |
commit | 073bda7a541731f41ed08f32d286394236c74005 (patch) | |
tree | 383dee0217648b88d8a27cfc7764620cc5845a88 /fs/btrfs/extent-tree.c | |
parent | 68879386180c0efd5a11e800b0525a01068c9457 (diff) | |
download | linux-stable-073bda7a541731f41ed08f32d286394236c74005.tar.gz linux-stable-073bda7a541731f41ed08f32d286394236c74005.tar.bz2 linux-stable-073bda7a541731f41ed08f32d286394236c74005.zip |
btrfs: zoned: add ASSERT and WARN for EXTENT_BUFFER_ZONED_ZEROOUT handling
Add an ASSERT to catch a faulty delayed reference item resulting from
prematurely cleared extent buffer.
Also, add a WARN to detect if we try to dirty a ZEROOUT buffer again, which
is suspicious as its update will be lost.
Reviewed-by: Johannes Thumshirn <johannes.thumshirn@wdc.com>
Signed-off-by: Naohiro Aota <naohiro.aota@wdc.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs/extent-tree.c')
-rw-r--r-- | fs/btrfs/extent-tree.c | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/fs/btrfs/extent-tree.c b/fs/btrfs/extent-tree.c index beedd6ed64d3..257d044bca91 100644 --- a/fs/btrfs/extent-tree.c +++ b/fs/btrfs/extent-tree.c @@ -3464,6 +3464,14 @@ void btrfs_free_tree_block(struct btrfs_trans_handle *trans, if (root_id != BTRFS_TREE_LOG_OBJECTID) { struct btrfs_ref generic_ref = { 0 }; + /* + * Assert that the extent buffer is not cleared due to + * EXTENT_BUFFER_ZONED_ZEROOUT. Please refer + * btrfs_clear_buffer_dirty() and btree_csum_one_bio() for + * detail. + */ + ASSERT(btrfs_header_bytenr(buf) != 0); + btrfs_init_generic_ref(&generic_ref, BTRFS_DROP_DELAYED_REF, buf->start, buf->len, parent, btrfs_header_owner(buf)); |