aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorLi Zefan <lizf@cn.fujitsu.com>2009-08-17 01:39:49 -0400
committerIngo Molnar <mingo@elte.hu>2009-08-17 03:54:40 -0400
commite9d65725bdf5954283625ca4d770bfc34f2ae56a (patch)
tree422219331fb039396ff24d9df6e1d5293cde1bd4
parent212274347fc4d2a7c56bf6c953b02c809e7e0be1 (diff)
lockdep: Fix missing entries in /proc/lock_chains
Two entries are missing in the output of /proc/lock_chains. One is chains[1]. When lc_next() is called the 1st time, chains[0] is returned. And when it's called the 2nd time, chains[2] is returned. The other missing ons is, when lc_start() is called the 2nd time, we should start from chains[@pos-1] but not chains[@pos], because pos == 0 is the header. Signed-off-by: Li Zefan <lizf@cn.fujitsu.com> Cc: Peter Zijlstra <peterz@infradead.org> LKML-Reference: <4A88ED25.2040306@cn.fujitsu.com> Signed-off-by: Ingo Molnar <mingo@elte.hu>
-rw-r--r--kernel/lockdep_proc.c8
1 files changed, 4 insertions, 4 deletions
diff --git a/kernel/lockdep_proc.c b/kernel/lockdep_proc.c
index 5dbe30b4e591..9a7996e371f5 100644
--- a/kernel/lockdep_proc.c
+++ b/kernel/lockdep_proc.c
@@ -160,8 +160,8 @@ static void *lc_next(struct seq_file *m, void *v, loff_t *pos)
160 else { 160 else {
161 chain = v; 161 chain = v;
162 162
163 if (*pos < nr_lock_chains) 163 if (*pos - 1 < nr_lock_chains)
164 chain = lock_chains + *pos; 164 chain = lock_chains + (*pos - 1);
165 else 165 else
166 chain = NULL; 166 chain = NULL;
167 } 167 }
@@ -174,8 +174,8 @@ static void *lc_start(struct seq_file *m, loff_t *pos)
174 if (*pos == 0) 174 if (*pos == 0)
175 return SEQ_START_TOKEN; 175 return SEQ_START_TOKEN;
176 176
177 if (*pos < nr_lock_chains) 177 if (*pos - 1 < nr_lock_chains)
178 return lock_chains + *pos; 178 return lock_chains + (*pos - 1);
179 179
180 return NULL; 180 return NULL;
181} 181}