summaryrefslogtreecommitdiffstats
path: root/fs/btrfs/file.c
diff options
context:
space:
mode:
authorGoldwyn Rodrigues <rgoldwyn@suse.com>2019-09-11 11:45:15 -0500
committerDavid Sterba <dsterba@suse.com>2019-11-18 12:46:45 +0100
commit9cf35f673583ccc9f3e2507498b3079d56614ad3 (patch)
tree32eb38a32f613b4aae675b26f08e5c7af0b9e5ba /fs/btrfs/file.c
parentaf42d3466bdc8f39806b26f593604fdc54140bcb (diff)
downloadlinux-9cf35f673583ccc9f3e2507498b3079d56614ad3.tar.gz
linux-9cf35f673583ccc9f3e2507498b3079d56614ad3.tar.bz2
linux-9cf35f673583ccc9f3e2507498b3079d56614ad3.zip
btrfs: simplify inode locking for RWF_NOWAIT
This is similar to 942491c9e6d6 ("xfs: fix AIM7 regression"). Apparently our current rwsem code doesn't like doing the trylock, then lock for real scheme. This causes extra contention on the lock and can be measured eg. by AIM7 benchmark. So change our read/write methods to just do the trylock for the RWF_NOWAIT case. Fixes: edf064e7c6fe ("btrfs: nowait aio support") Signed-off-by: Goldwyn Rodrigues <rgoldwyn@suse.com> Reviewed-by: David Sterba <dsterba@suse.com> [ update changelog ] Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs/file.c')
-rw-r--r--fs/btrfs/file.c5
1 files changed, 3 insertions, 2 deletions
diff --git a/fs/btrfs/file.c b/fs/btrfs/file.c
index 435a502a3226..25ce1b6dbda9 100644
--- a/fs/btrfs/file.c
+++ b/fs/btrfs/file.c
@@ -1903,9 +1903,10 @@ static ssize_t btrfs_file_write_iter(struct kiocb *iocb,
(iocb->ki_flags & IOCB_NOWAIT))
return -EOPNOTSUPP;
- if (!inode_trylock(inode)) {
- if (iocb->ki_flags & IOCB_NOWAIT)
+ if (iocb->ki_flags & IOCB_NOWAIT) {
+ if (!inode_trylock(inode))
return -EAGAIN;
+ } else {
inode_lock(inode);
}