Re: [PATCH v2 23/34] compiler-capability-analysis: Remove __cond_lock() function-like helper

From: Bart Van Assche
Date: Tue Mar 04 2025 - 18:26:05 EST


On 3/4/25 1:21 AM, Marco Elver wrote:
.../dev-tools/capability-analysis.rst | 2 -
Documentation/mm/process_addrs.rst | 6 +-
.../net/wireless/intel/iwlwifi/iwl-trans.c | 4 +-
.../net/wireless/intel/iwlwifi/iwl-trans.h | 6 +-
.../wireless/intel/iwlwifi/pcie/internal.h | 5 +-
.../net/wireless/intel/iwlwifi/pcie/trans.c | 4 +-
include/linux/compiler-capability-analysis.h | 41 -------------
include/linux/mm.h | 33 ++--------
include/linux/rwlock.h | 11 +---
include/linux/rwlock_api_smp.h | 14 ++++-
include/linux/rwlock_rt.h | 21 ++++---
include/linux/sched/signal.h | 14 +----
include/linux/spinlock.h | 45 +++++---------
include/linux/spinlock_api_smp.h | 20 ++++++
include/linux/spinlock_api_up.h | 61 ++++++++++++++++---
include/linux/spinlock_rt.h | 26 ++++----
kernel/signal.c | 4 +-
kernel/time/posix-timers.c | 10 +--
lib/dec_and_lock.c | 8 +--
mm/memory.c | 4 +-
mm/pgtable-generic.c | 19 +++---
tools/include/linux/compiler_types.h | 2 -
22 files changed, 160 insertions(+), 200 deletions(-)

Should this patch perhaps be split in one patch per subsystem (mm, iwlwifi, rwlock, signal, spinlock, time)? Would that make it more likely
that subsystem maintainers review this work? Can this patch be moved to
the start of this series? Anyway, if others are fine with this patch and its position in this patch series I'm also fine with this patch.

Thanks,

Bart.