diff options
| author | David Sterba | 2019-01-23 18:07:14 +0100 |
|---|---|---|
| committer | David Sterba | 2019-02-25 14:13:28 +0100 |
| commit | 766ece54f4c9c29b25eabd091a2ee939feb1669e (patch) | |
| tree | e600cdb57da7810436ca26bfbc5b7be66925ecf6 /fs/btrfs/ctree.c | |
| parent | btrfs: simplify waiting loop in btrfs_tree_lock (diff) | |
| download | kernel-qcow2-linux-766ece54f4c9c29b25eabd091a2ee939feb1669e.tar.gz kernel-qcow2-linux-766ece54f4c9c29b25eabd091a2ee939feb1669e.tar.xz kernel-qcow2-linux-766ece54f4c9c29b25eabd091a2ee939feb1669e.zip | |
btrfs: merge btrfs_set_lock_blocking_rw with it's caller
The last caller that does not have a fixed value of lock is
btrfs_set_path_blocking, that actually does the same conditional swtich
by the lock type so we can merge the branches together and remove the
helper.
Reviewed-by: Johannes Thumshirn <jthumshirn@suse.de>
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs/ctree.c')
| -rw-r--r-- | fs/btrfs/ctree.c | 13 |
1 files changed, 10 insertions, 3 deletions
diff --git a/fs/btrfs/ctree.c b/fs/btrfs/ctree.c index 3940d707fcf3..4fac5cc2e648 100644 --- a/fs/btrfs/ctree.c +++ b/fs/btrfs/ctree.c @@ -46,11 +46,18 @@ noinline void btrfs_set_path_blocking(struct btrfs_path *p) for (i = 0; i < BTRFS_MAX_LEVEL; i++) { if (!p->nodes[i] || !p->locks[i]) continue; - btrfs_set_lock_blocking_rw(p->nodes[i], p->locks[i]); - if (p->locks[i] == BTRFS_READ_LOCK) + /* + * If we currently have a spinning reader or writer lock this + * will bump the count of blocking holders and drop the + * spinlock. + */ + if (p->locks[i] == BTRFS_READ_LOCK) { + btrfs_set_lock_blocking_read(p->nodes[i]); p->locks[i] = BTRFS_READ_LOCK_BLOCKING; - else if (p->locks[i] == BTRFS_WRITE_LOCK) + } else if (p->locks[i] == BTRFS_WRITE_LOCK) { + btrfs_set_lock_blocking_write(p->nodes[i]); p->locks[i] = BTRFS_WRITE_LOCK_BLOCKING; + } } } |