mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/netdev/net.git/
synced 2026-04-18 06:33:43 -04:00
btrfs: mark ordered extent insertion failure checks as unlikely
We never expect an ordered extent insertion to fail due to already having another ordered extent in the tree for the same file offset, since we always wait for existing ordered extents in a range to complete before writing into the range again. So mark the failure checks for the results of tree_insert() as unlikely, to make it clear it's never expected (save exceptional causes like bugs or memory corruptions) and to serve as a hint for the compiler to possibly generate better code. Reviewed-by: Josef Bacik <josef@toxicpanda.com> Reviewed-by: Qu Wenruo <wqu@suse.com> Reviewed-by: Johannes Thumshirn <johannes.thumshirn@wdc.com> Signed-off-by: Filipe Manana <fdmanana@suse.com> Reviewed-by: David Sterba <dsterba@suse.com> Signed-off-by: David Sterba <dsterba@suse.com>
This commit is contained in:
committed by
David Sterba
parent
cb3cd62454
commit
b7ac1acbdd
@@ -224,7 +224,7 @@ static void insert_ordered_extent(struct btrfs_ordered_extent *entry)
|
||||
spin_lock_irq(&inode->ordered_tree_lock);
|
||||
node = tree_insert(&inode->ordered_tree, entry->file_offset,
|
||||
&entry->rb_node);
|
||||
if (node)
|
||||
if (unlikely(node))
|
||||
btrfs_panic(fs_info, -EEXIST,
|
||||
"inconsistency in ordered tree at offset %llu",
|
||||
entry->file_offset);
|
||||
@@ -1303,7 +1303,7 @@ struct btrfs_ordered_extent *btrfs_split_ordered_extent(
|
||||
}
|
||||
|
||||
node = tree_insert(&inode->ordered_tree, new->file_offset, &new->rb_node);
|
||||
if (node)
|
||||
if (unlikely(node))
|
||||
btrfs_panic(fs_info, -EEXIST,
|
||||
"zoned: inconsistency in ordered tree at offset %llu",
|
||||
new->file_offset);
|
||||
|
||||
Reference in New Issue
Block a user