diff options
| author | Nikolay Borisov | 2018-06-27 15:38:23 +0200 |
|---|---|---|
| committer | David Sterba | 2018-08-06 13:12:54 +0200 |
| commit | 07e21c4dad42be8d2a6b8cd0b98cf8e6931e12fd (patch) | |
| tree | 6d262c09157e3cf7c0eede5de259634a835d914f /fs/btrfs/extent_io.c | |
| parent | btrfs: rename btrfs_release_extent_buffer_page (diff) | |
| download | kernel-qcow2-linux-07e21c4dad42be8d2a6b8cd0b98cf8e6931e12fd.tar.gz kernel-qcow2-linux-07e21c4dad42be8d2a6b8cd0b98cf8e6931e12fd.tar.xz kernel-qcow2-linux-07e21c4dad42be8d2a6b8cd0b98cf8e6931e12fd.zip | |
btrfs: Document locking requirement via lockdep_assert_held
Remove stale comment since there is no longer an eb->eb_lock and
document the locking expectation with a lockdep_assert_held statement.
No functional changes.
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/extent_io.c')
| -rw-r--r-- | fs/btrfs/extent_io.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/fs/btrfs/extent_io.c b/fs/btrfs/extent_io.c index 6a4013ddcf73..5ab401f79683 100644 --- a/fs/btrfs/extent_io.c +++ b/fs/btrfs/extent_io.c @@ -5066,9 +5066,10 @@ static inline void btrfs_release_extent_buffer_rcu(struct rcu_head *head) __free_extent_buffer(eb); } -/* Expects to have eb->eb_lock already held */ static int release_extent_buffer(struct extent_buffer *eb) { + lockdep_assert_held(&eb->refs_lock); + WARN_ON(atomic_read(&eb->refs) == 0); if (atomic_dec_and_test(&eb->refs)) { if (test_and_clear_bit(EXTENT_BUFFER_IN_TREE, &eb->bflags)) { |