diff options
author | Pavel Emelyanov <xemul@openvz.org> | 2007-10-19 02:40:40 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2007-10-19 14:53:43 -0400 |
commit | ba25f9dcc4ea6e30839fcab5a5516f2176d5bfed (patch) | |
tree | 3123c03b25dd5c0cd24b6ab4fc16731217838157 /lib/spinlock_debug.c | |
parent | 9a2e70572e94e21e7ec4186702d045415422bda0 (diff) |
Use helpers to obtain task pid in printks
The task_struct->pid member is going to be deprecated, so start
using the helpers (task_pid_nr/task_pid_vnr/task_pid_nr_ns) in
the kernel.
The first thing to start with is the pid, printed to dmesg - in
this case we may safely use task_pid_nr(). Besides, printks produce
more (much more) than a half of all the explicit pid usage.
[akpm@linux-foundation.org: git-drm went and changed lots of stuff]
Signed-off-by: Pavel Emelyanov <xemul@openvz.org>
Cc: Dave Airlie <airlied@linux.ie>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'lib/spinlock_debug.c')
-rw-r--r-- | lib/spinlock_debug.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/lib/spinlock_debug.c b/lib/spinlock_debug.c index 479fd462eaa9..9c4b0256490b 100644 --- a/lib/spinlock_debug.c +++ b/lib/spinlock_debug.c | |||
@@ -60,12 +60,12 @@ static void spin_bug(spinlock_t *lock, const char *msg) | |||
60 | owner = lock->owner; | 60 | owner = lock->owner; |
61 | printk(KERN_EMERG "BUG: spinlock %s on CPU#%d, %s/%d\n", | 61 | printk(KERN_EMERG "BUG: spinlock %s on CPU#%d, %s/%d\n", |
62 | msg, raw_smp_processor_id(), | 62 | msg, raw_smp_processor_id(), |
63 | current->comm, current->pid); | 63 | current->comm, task_pid_nr(current)); |
64 | printk(KERN_EMERG " lock: %p, .magic: %08x, .owner: %s/%d, " | 64 | printk(KERN_EMERG " lock: %p, .magic: %08x, .owner: %s/%d, " |
65 | ".owner_cpu: %d\n", | 65 | ".owner_cpu: %d\n", |
66 | lock, lock->magic, | 66 | lock, lock->magic, |
67 | owner ? owner->comm : "<none>", | 67 | owner ? owner->comm : "<none>", |
68 | owner ? owner->pid : -1, | 68 | owner ? task_pid_nr(owner) : -1, |
69 | lock->owner_cpu); | 69 | lock->owner_cpu); |
70 | dump_stack(); | 70 | dump_stack(); |
71 | } | 71 | } |
@@ -116,7 +116,7 @@ static void __spin_lock_debug(spinlock_t *lock) | |||
116 | printk(KERN_EMERG "BUG: spinlock lockup on CPU#%d, " | 116 | printk(KERN_EMERG "BUG: spinlock lockup on CPU#%d, " |
117 | "%s/%d, %p\n", | 117 | "%s/%d, %p\n", |
118 | raw_smp_processor_id(), current->comm, | 118 | raw_smp_processor_id(), current->comm, |
119 | current->pid, lock); | 119 | task_pid_nr(current), lock); |
120 | dump_stack(); | 120 | dump_stack(); |
121 | #ifdef CONFIG_SMP | 121 | #ifdef CONFIG_SMP |
122 | trigger_all_cpu_backtrace(); | 122 | trigger_all_cpu_backtrace(); |
@@ -161,7 +161,7 @@ static void rwlock_bug(rwlock_t *lock, const char *msg) | |||
161 | 161 | ||
162 | printk(KERN_EMERG "BUG: rwlock %s on CPU#%d, %s/%d, %p\n", | 162 | printk(KERN_EMERG "BUG: rwlock %s on CPU#%d, %s/%d, %p\n", |
163 | msg, raw_smp_processor_id(), current->comm, | 163 | msg, raw_smp_processor_id(), current->comm, |
164 | current->pid, lock); | 164 | task_pid_nr(current), lock); |
165 | dump_stack(); | 165 | dump_stack(); |
166 | } | 166 | } |
167 | 167 | ||