diff options
| author | Nikolay Borisov <nborisov@suse.com> | 2019-10-08 16:26:16 +0300 |
|---|---|---|
| committer | David Sterba <dsterba@suse.com> | 2019-11-18 12:46:54 +0100 |
| commit | ce6d3eb6fd5cfede81c7a3d540919f6d92faaac0 (patch) | |
| tree | d0aeb26ddb7a86bca574e51b9153530cbdaf1759 /fs/btrfs | |
| parent | 67439dadb03ad9da45bfccb4cdb6ef6b1a7f8da9 (diff) | |
| download | linux-ce6d3eb6fd5cfede81c7a3d540919f6d92faaac0.tar.bz2 | |
btrfs: User assert to document transaction requirement
Using an ASSERT in btrfs_pin_extent allows to more stringently observe
whether the function is called under a transaction or not.
Reviewed-by: Johannes Thumshirn <jthumshirn@suse.de>
Signed-off-by: Nikolay Borisov <nborisov@suse.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs')
| -rw-r--r-- | fs/btrfs/extent-tree.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/fs/btrfs/extent-tree.c b/fs/btrfs/extent-tree.c index 9e5845548b76..569fd2adecaa 100644 --- a/fs/btrfs/extent-tree.c +++ b/fs/btrfs/extent-tree.c @@ -2590,14 +2590,13 @@ static int pin_down_extent(struct btrfs_block_group_cache *cache, return 0; } -/* - * this function must be called within transaction - */ int btrfs_pin_extent(struct btrfs_fs_info *fs_info, u64 bytenr, u64 num_bytes, int reserved) { struct btrfs_block_group_cache *cache; + ASSERT(fs_info->running_transaction); + cache = btrfs_lookup_block_group(fs_info, bytenr); BUG_ON(!cache); /* Logic error */ |