[PATCH v3 0/2] locking/rwsem: Fix DEBUG_RWSEM warning from thaw_super()
From: Waiman Long
Date: Tue May 15 2018 - 13:42:05 EST
v3:
- Modify patch 1 to not expose any new rwsem owner related function.
- Modify patch 2 to make percpu_rwsem_release() and
percpu_rwsem_acquire() set the owner field directly, if applicable.
This patchset aims to fix the DEBUG_RWSEM warning in the filesystem
freezing/thawing code. A new macro RWSEM_OWNER_UNKNOWN (-1) is exposed in
the linux/rwsem.h file to indicate that a rwsem is currently owned by an
unknown writer. Other than that, there is no externally visible changes.
The new RWSEM_OWNER_UNKNOWN macro has no dependency on internal rwsem
header. What is important is that bit 1 of the owner field is set which
is the marker used by the internal rwsem code to determine if the owner
is unknown or not.
The original code of clearing the owner field in percpu_rwsem_release()
isn't correct as a NULL owner field should correspond to an unlocked
rwsem which is not the case here. In addition, optimistic spinning
won't be stopped by a NULL owner value.
Testings are welcome.
Waiman Long (2):
locking/rwsem: Add a new RWSEM_ANONYMOUSLY_OWNED flag
locking/percpu-rwsem: Annotate rwsem ownership transfer by setting
RWSEM_OWNER_UNKNOWN
include/linux/percpu-rwsem.h | 6 +++++-
include/linux/rwsem.h | 5 +++++
kernel/locking/rwsem-xadd.c | 17 ++++++++---------
kernel/locking/rwsem.c | 5 ++---
kernel/locking/rwsem.h | 33 ++++++++++++++++++++++++---------
5 files changed, 44 insertions(+), 22 deletions(-)
--
1.8.3.1