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 /mm | |
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 'mm')
-rw-r--r-- | mm/oom_kill.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/mm/oom_kill.c b/mm/oom_kill.c index e3778f1215c0..824cade07827 100644 --- a/mm/oom_kill.c +++ b/mm/oom_kill.c | |||
@@ -278,7 +278,8 @@ static void __oom_kill_task(struct task_struct *p, int verbose) | |||
278 | } | 278 | } |
279 | 279 | ||
280 | if (verbose) | 280 | if (verbose) |
281 | printk(KERN_ERR "Killed process %d (%s)\n", p->pid, p->comm); | 281 | printk(KERN_ERR "Killed process %d (%s)\n", |
282 | task_pid_nr(p), p->comm); | ||
282 | 283 | ||
283 | /* | 284 | /* |
284 | * We give our sacrificial lamb high priority and access to | 285 | * We give our sacrificial lamb high priority and access to |
@@ -356,7 +357,7 @@ static int oom_kill_process(struct task_struct *p, gfp_t gfp_mask, int order, | |||
356 | } | 357 | } |
357 | 358 | ||
358 | printk(KERN_ERR "%s: kill process %d (%s) score %li or a child\n", | 359 | printk(KERN_ERR "%s: kill process %d (%s) score %li or a child\n", |
359 | message, p->pid, p->comm, points); | 360 | message, task_pid_nr(p), p->comm, points); |
360 | 361 | ||
361 | /* Try to kill a child first */ | 362 | /* Try to kill a child first */ |
362 | list_for_each_entry(c, &p->children, sibling) { | 363 | list_for_each_entry(c, &p->children, sibling) { |