From: Pavel Reichl <preichl@redhat.com>
To: linux-xfs@vger.kernel.org
Subject: [PATCH v6 1/4] xfs: Refactor xfs_isilocked()
Date: Thu, 27 Feb 2020 21:36:33 +0100 [thread overview]
Message-ID: <20200227203636.317790-2-preichl@redhat.com> (raw)
In-Reply-To: <20200227203636.317790-1-preichl@redhat.com>
Refactor xfs_isilocked() to use newly introduced __xfs_rwsem_islocked().
__xfs_rwsem_islocked() is a helper function which encapsulates checking
state of rw_semaphores hold by inode.
Signed-off-by: Pavel Reichl <preichl@redhat.com>
Suggested-by: Dave Chinner <dchinner@redhat.com>
Suggested-by: Eric Sandeen <sandeen@redhat.com>
---
Changes from V5:
Drop shared flag from __xfs_rwsem_islocked()
fs/xfs/xfs_inode.c | 42 ++++++++++++++++++++++++++----------------
fs/xfs/xfs_inode.h | 2 +-
2 files changed, 27 insertions(+), 17 deletions(-)
diff --git a/fs/xfs/xfs_inode.c b/fs/xfs/xfs_inode.c
index c5077e6326c7..4faf7827717b 100644
--- a/fs/xfs/xfs_inode.c
+++ b/fs/xfs/xfs_inode.c
@@ -345,32 +345,42 @@ xfs_ilock_demote(
}
#if defined(DEBUG) || defined(XFS_WARN)
-int
+static inline bool
+__xfs_rwsem_islocked(
+ struct rw_semaphore *rwsem,
+ bool excl)
+{
+ if (!rwsem_is_locked(rwsem))
+ return false;
+
+ if (debug_locks && excl)
+ return lockdep_is_held_type(rwsem, 1);
+
+ return true;
+}
+
+bool
xfs_isilocked(
- xfs_inode_t *ip,
+ struct xfs_inode *ip,
uint lock_flags)
{
- if (lock_flags & (XFS_ILOCK_EXCL|XFS_ILOCK_SHARED)) {
- if (!(lock_flags & XFS_ILOCK_SHARED))
- return !!ip->i_lock.mr_writer;
- return rwsem_is_locked(&ip->i_lock.mr_lock);
+ if (lock_flags & (XFS_ILOCK_EXCL | XFS_ILOCK_SHARED)) {
+ return __xfs_rwsem_islocked(&ip->i_lock.mr_lock,
+ (lock_flags & XFS_ILOCK_EXCL));
}
- if (lock_flags & (XFS_MMAPLOCK_EXCL|XFS_MMAPLOCK_SHARED)) {
- if (!(lock_flags & XFS_MMAPLOCK_SHARED))
- return !!ip->i_mmaplock.mr_writer;
- return rwsem_is_locked(&ip->i_mmaplock.mr_lock);
+ if (lock_flags & (XFS_MMAPLOCK_EXCL | XFS_MMAPLOCK_SHARED)) {
+ return __xfs_rwsem_islocked(&ip->i_mmaplock.mr_lock,
+ (lock_flags & XFS_MMAPLOCK_EXCL));
}
- if (lock_flags & (XFS_IOLOCK_EXCL|XFS_IOLOCK_SHARED)) {
- if (!(lock_flags & XFS_IOLOCK_SHARED))
- return !debug_locks ||
- lockdep_is_held_type(&VFS_I(ip)->i_rwsem, 0);
- return rwsem_is_locked(&VFS_I(ip)->i_rwsem);
+ if (lock_flags & (XFS_IOLOCK_EXCL | XFS_IOLOCK_SHARED)) {
+ return __xfs_rwsem_islocked(&VFS_I(ip)->i_rwsem,
+ (lock_flags & XFS_IOLOCK_EXCL));
}
ASSERT(0);
- return 0;
+ return false;
}
#endif
diff --git a/fs/xfs/xfs_inode.h b/fs/xfs/xfs_inode.h
index 492e53992fa9..3d7ce355407d 100644
--- a/fs/xfs/xfs_inode.h
+++ b/fs/xfs/xfs_inode.h
@@ -416,7 +416,7 @@ void xfs_ilock(xfs_inode_t *, uint);
int xfs_ilock_nowait(xfs_inode_t *, uint);
void xfs_iunlock(xfs_inode_t *, uint);
void xfs_ilock_demote(xfs_inode_t *, uint);
-int xfs_isilocked(xfs_inode_t *, uint);
+bool xfs_isilocked(xfs_inode_t *, uint);
uint xfs_ilock_data_map_shared(struct xfs_inode *);
uint xfs_ilock_attr_map_shared(struct xfs_inode *);
--
2.24.1
next prev parent reply other threads:[~2020-02-27 20:36 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-02-27 20:36 [PATCH v6 0/4] xfs: Remove wrappers for some semaphores Pavel Reichl
2020-02-27 20:36 ` Pavel Reichl [this message]
2020-02-28 17:10 ` [PATCH v6 1/4] xfs: Refactor xfs_isilocked() Darrick J. Wong
2020-03-18 17:13 ` Pavel Reichl
2020-03-18 17:46 ` Eric Sandeen
2020-03-18 18:49 ` Darrick J. Wong
2020-03-18 19:10 ` Eric Sandeen
2020-03-20 14:41 ` Pavel Reichl
2020-03-20 15:48 ` Darrick J. Wong
2020-02-27 20:36 ` [PATCH v6 2/4] xfs: clean up whitespace in xfs_isilocked() calls Pavel Reichl
2020-02-27 20:36 ` [PATCH v6 3/4] xfs: xfs_isilocked() can only check a single lock type Pavel Reichl
2020-02-27 20:36 ` [PATCH v6 4/4] xfs: replace mrlock_t with rw_semaphores Pavel Reichl
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20200227203636.317790-2-preichl@redhat.com \
--to=preichl@redhat.com \
--cc=linux-xfs@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.