[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-Id: <20200212153828.346-1-longman@redhat.com>
Date: Wed, 12 Feb 2020 10:38:25 -0500
From: Waiman Long <longman@...hat.com>
To: Peter Zijlstra <peterz@...radead.org>,
Ingo Molnar <mingo@...hat.com>,
Will Deacon <will.deacon@....com>
Cc: linux-kernel@...r.kernel.org, Bart Van Assche <bvanassche@....org>,
Waiman Long <longman@...hat.com>
Subject: [PATCH-tip 0/3] locking/lockdep: Cleaning up lockdep code
The following conditional compilation macros are used throughout the
lockdep code making it hard to understand.
- CONFIG_PROVE_LOCKING
- CONFIG_LOCK_STAT
- CONFIG_DEBUG_LOCKDEP
- CONFIG_LOCKDEP_SMALL
- CONFIG_TRACE_IRQFLAGS
In particular, almost half of the code is for CONFIG_PROVE_LOCKING and
there are several blocks of them in lockdep.c. It is very hard to see if
a function is for CONFIG_PROVE_LOCKING or not.
This patchset cleans up this mess by putting all the CONFIG_PROVE_LOCKING
functions into a separate lockdep_prove.c file making it easier to know
what they are for. It also consolidates the CONFIG_DEBUG_LOCKDEP code
to a certain extend.
This is a code relocation patchset. There is no functional change.
Waiman Long (3):
locking/lockdep: Make remove_class_from_lock_chains() depend on
CONFIG_PROVE_LOCKING
locking/lockdep: Extract CONFIG_PROVE_LOCKING code out to
lockdep_prove.c
locking/lockdep: Consolidate CONFIG_DEBUG_LOCKDEP code
kernel/locking/lockdep.c | 3300 +++-----------------------------
kernel/locking/lockdep_prove.c | 2820 +++++++++++++++++++++++++++
2 files changed, 3066 insertions(+), 3054 deletions(-)
create mode 100644 kernel/locking/lockdep_prove.c
--
2.18.1
Powered by blists - more mailing lists