diff options
| author | Waiman Long <longman@redhat.com> | 2024-09-09 14:29:05 -0400 |
|---|---|---|
| committer | Peter Zijlstra <peterz@infradead.org> | 2024-09-10 12:02:33 +0200 |
| commit | d00b83d416e73bc3fa4d21b14bec920e88b70ce6 (patch) | |
| tree | 68909c23e7ef5f4e4f0dabb1a86c85a41271dbdc /block | |
| parent | 39dea484e2bb9066abbc01e2c5e03b6917b0b775 (diff) | |
| download | linux-d00b83d416e73bc3fa4d21b14bec920e88b70ce6.tar.gz linux-d00b83d416e73bc3fa4d21b14bec920e88b70ce6.tar.bz2 linux-d00b83d416e73bc3fa4d21b14bec920e88b70ce6.zip | |
locking/rwsem: Move is_rwsem_reader_owned() and rwsem_owner() under CONFIG_DEBUG_RWSEMS
Both is_rwsem_reader_owned() and rwsem_owner() are currently only used when
CONFIG_DEBUG_RWSEMS is defined. This causes a compilation error with clang
when `make W=1` and CONFIG_WERROR=y:
kernel/locking/rwsem.c:187:20: error: unused function 'is_rwsem_reader_owned' [-Werror,-Wunused-function]
187 | static inline bool is_rwsem_reader_owned(struct rw_semaphore *sem)
| ^~~~~~~~~~~~~~~~~~~~~
kernel/locking/rwsem.c:271:35: error: unused function 'rwsem_owner' [-Werror,-Wunused-function]
271 | static inline struct task_struct *rwsem_owner(struct rw_semaphore *sem)
| ^~~~~~~~~~~
Fix this by moving these two functions under the CONFIG_DEBUG_RWSEMS define.
Reported-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: Waiman Long <longman@redhat.com>
Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Tested-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Link: https://lore.kernel.org/r/20240909182905.161156-1-longman@redhat.com
Diffstat (limited to 'block')
0 files changed, 0 insertions, 0 deletions