btrfs: Use ihold instead of igrab in cow_file_range_async
authorNikolay Borisov <nborisov@suse.com>
Thu, 3 Jan 2019 08:50:01 +0000 (10:50 +0200)
committerDavid Sterba <dsterba@suse.com>
Mon, 25 Feb 2019 13:13:19 +0000 (14:13 +0100)
commitbd4691a0e866e774c9690f09fec573e845495f91
tree4f4179bb92f11f58a073f8695cc49d8266b90917
parent62b37622718c87db3d992fe03b25f6f3b18264f1
btrfs: Use ihold instead of igrab in cow_file_range_async

ihold is supposed to be used when the caller already has a reference to
the inode. In the case of cow_file_range_async this invariants holds,
since the 3 call chains leading to this function all take a reference:

btrfs_writepage  <--- does igrab
 extent_write_full_page
  __extent_writepage
   writepage_delalloc
     btrfs_run_delalloc_range
      cow_file_range_async

extent_write_cache_pages <--- does igrab
 __extent_writepage (same callchain as above)

and

submit_compressed_extents <-- already called from async CoW submit path,
      which would have done ihold.
 extent_write_locked_range
  __extent_writepage

Signed-off-by: Nikolay Borisov <nborisov@suse.com>
Reviewed-by: David Sterba <dsterba@suse.com>
[ add comment ]
Signed-off-by: David Sterba <dsterba@suse.com>
fs/btrfs/inode.c