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

Bart Van Assche
2025-03-04T23:25:53Z

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.