From: Yong Zhang <yong.zhang0@gmail.com>
To: linux-kernel@vger.kernel.org
Cc: Peter Zijlstra <peterz@infradead.org>, Ingo Molnar <mingo@elte.hu>
Subject: [RFC PATCH] lockdep: fix lockdep_stats_show() by using lockdep_state magic
Date: Mon, 14 Feb 2011 23:19:05 +0800 [thread overview]
Message-ID: <20110214151905.GA3193@zhy> (raw)
Hi Peter/Ingo,
Now lockdep_stats_show() doesn't show every thing we need, like RECLAIM_FS
status is missed. So I come up with a patch for it(but not for included).
1) In this patch, RECLAIM_FS-* locks is added:
RECLAIM_FS-safe locks: 4
RECLAIM_FS-unsafe locks: 134
RECLAIM_FS-read-safe locks: 0
RECLAIM_FS-read-unsafe locks: 74
2) But the difference is HARDIRQ, SOFRIRQ, RECLAIM_FS becomes to
upcase. Like:
HARDIRQ-safe locks: 63
HARDIRQ-unsafe locks: 590
SOFTIRQ-safe locks: 115
SOFTIRQ-unsafe locks: 506
RECLAIM_FS-safe locks: 4
RECLAIM_FS-unsafe locks: 134
Is this kind of change acceptable?
3) When looking through lockdep_stats_show(), I think there is more
thing to rewrite, like:
a) "all direct dependencies" calculation, actually I'm not very sure how
it is worked out(teach please).
b) factor is not used.
c) Is there any other message should be showed?
So could we make a decision on what lockdep_stats should be, then
I can make a patchset to reach the requirement?
BTW, the below is just to show some point, not for included.
Thanks,
Yong
---
Subject: [PATCH] lockdep: fix lockdep_stats_show() by using lockdep_state magic
Signed-off-by: Yong Zhang <yong.zhang0@gmail.com>
---
kernel/lockdep_proc.c | 79 ++++++++++++++++++++++++++-----------------------
1 files changed, 42 insertions(+), 37 deletions(-)
diff --git a/kernel/lockdep_proc.c b/kernel/lockdep_proc.c
index 1969d2f..c979679 100644
--- a/kernel/lockdep_proc.c
+++ b/kernel/lockdep_proc.c
@@ -220,12 +220,16 @@ static int lockdep_stats_show(struct seq_file *m, void *v)
struct lock_class *class;
unsigned long nr_unused = 0, nr_uncategorized = 0,
nr_irq_safe = 0, nr_irq_unsafe = 0,
- nr_softirq_safe = 0, nr_softirq_unsafe = 0,
- nr_hardirq_safe = 0, nr_hardirq_unsafe = 0,
nr_irq_read_safe = 0, nr_irq_read_unsafe = 0,
- nr_softirq_read_safe = 0, nr_softirq_read_unsafe = 0,
- nr_hardirq_read_safe = 0, nr_hardirq_read_unsafe = 0,
sum_forward_deps = 0, factor = 0;
+#define LOCKDEP_STATE(__STATE) \
+ unsigned long nr_##__STATE##_safe = 0; \
+ unsigned long nr_##__STATE##_unsafe = 0; \
+ unsigned long nr_##__STATE##_read_safe = 0; \
+ unsigned long nr_##__STATE##_read_unsafe = 0;
+
+#include "lockdep_states.h"
+#undef LOCKDEP_STATE
list_for_each_entry(class, &all_lock_classes, lock_entry) {
@@ -233,35 +237,33 @@ static int lockdep_stats_show(struct seq_file *m, void *v)
nr_unused++;
if (class->usage_mask == LOCKF_USED)
nr_uncategorized++;
+#define LOCKDEP_STATE(__STATE) \
+ if (class->usage_mask & LOCKF_USED_IN_##__STATE) \
+ nr_##__STATE##_safe++; \
+ if (class->usage_mask & LOCKF_ENABLED_##__STATE) \
+ nr_##__STATE##_unsafe++; \
+ if (class->usage_mask & LOCKF_USED_IN_##__STATE##_READ) \
+ nr_##__STATE##_read_safe++; \
+ if (class->usage_mask & LOCKF_ENABLED_##__STATE##_READ) \
+ nr_##__STATE##_read_unsafe++;
+
+#include "lockdep_states.h"
+#undef LOCKDEP_STATE
+
if (class->usage_mask & LOCKF_USED_IN_IRQ)
nr_irq_safe++;
if (class->usage_mask & LOCKF_ENABLED_IRQ)
nr_irq_unsafe++;
- if (class->usage_mask & LOCKF_USED_IN_SOFTIRQ)
- nr_softirq_safe++;
- if (class->usage_mask & LOCKF_ENABLED_SOFTIRQ)
- nr_softirq_unsafe++;
- if (class->usage_mask & LOCKF_USED_IN_HARDIRQ)
- nr_hardirq_safe++;
- if (class->usage_mask & LOCKF_ENABLED_HARDIRQ)
- nr_hardirq_unsafe++;
if (class->usage_mask & LOCKF_USED_IN_IRQ_READ)
nr_irq_read_safe++;
if (class->usage_mask & LOCKF_ENABLED_IRQ_READ)
nr_irq_read_unsafe++;
- if (class->usage_mask & LOCKF_USED_IN_SOFTIRQ_READ)
- nr_softirq_read_safe++;
- if (class->usage_mask & LOCKF_ENABLED_SOFTIRQ_READ)
- nr_softirq_read_unsafe++;
- if (class->usage_mask & LOCKF_USED_IN_HARDIRQ_READ)
- nr_hardirq_read_safe++;
- if (class->usage_mask & LOCKF_ENABLED_HARDIRQ_READ)
- nr_hardirq_read_unsafe++;
#ifdef CONFIG_PROVE_LOCKING
sum_forward_deps += lockdep_count_forward_deps(class);
#endif
}
+
#ifdef CONFIG_DEBUG_LOCKDEP
DEBUG_LOCKS_WARN_ON(debug_atomic_read(nr_unused_locks) != nr_unused);
#endif
@@ -280,7 +282,7 @@ static int lockdep_stats_show(struct seq_file *m, void *v)
*/
seq_printf(m, " all direct dependencies: %11lu\n",
nr_irq_unsafe * nr_irq_safe +
- nr_hardirq_unsafe * nr_hardirq_safe +
+ nr_HARDIRQ_unsafe * nr_HARDIRQ_safe +
nr_list_entries);
/*
@@ -312,27 +314,30 @@ static int lockdep_stats_show(struct seq_file *m, void *v)
(nr_softirq_chains + 1) *
(nr_process_chains + 1)
);
- seq_printf(m, " hardirq-safe locks: %11lu\n",
- nr_hardirq_safe);
- seq_printf(m, " hardirq-unsafe locks: %11lu\n",
- nr_hardirq_unsafe);
- seq_printf(m, " softirq-safe locks: %11lu\n",
- nr_softirq_safe);
- seq_printf(m, " softirq-unsafe locks: %11lu\n",
- nr_softirq_unsafe);
+
+#define LOCKDEP_STATE(__STATE) \
+ seq_printf(m, " %-31s%11lu\n", \
+ __stringify(__STATE)"-safe locks:", \
+ nr_##__STATE##_safe); \
+ seq_printf(m, " %-31s%11lu\n", \
+ __stringify(__STATE)"-unsafe locks:", \
+ nr_##__STATE##_unsafe);
+#include "lockdep_states.h"
+#undef LOCKDEP_STATE
seq_printf(m, " irq-safe locks: %11lu\n",
nr_irq_safe);
seq_printf(m, " irq-unsafe locks: %11lu\n",
nr_irq_unsafe);
- seq_printf(m, " hardirq-read-safe locks: %11lu\n",
- nr_hardirq_read_safe);
- seq_printf(m, " hardirq-read-unsafe locks: %11lu\n",
- nr_hardirq_read_unsafe);
- seq_printf(m, " softirq-read-safe locks: %11lu\n",
- nr_softirq_read_safe);
- seq_printf(m, " softirq-read-unsafe locks: %11lu\n",
- nr_softirq_read_unsafe);
+#define LOCKDEP_STATE(__STATE) \
+ seq_printf(m, " %-31s%11lu\n", \
+ __stringify(__STATE)"-read-safe locks:", \
+ nr_##__STATE##_read_safe); \
+ seq_printf(m, " %-31s%11lu\n", \
+ __stringify(__STATE)"-read-unsafe locks:", \
+ nr_##__STATE##_read_unsafe);
+#include "lockdep_states.h"
+#undef LOCKDEP_STATE
seq_printf(m, " irq-read-safe locks: %11lu\n",
nr_irq_read_safe);
seq_printf(m, " irq-read-unsafe locks: %11lu\n",
--
1.7.1
reply other threads:[~2011-02-14 15:19 UTC|newest]
Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20110214151905.GA3193@zhy \
--to=yong.zhang0@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@elte.hu \
--cc=peterz@infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox