2009-01-03 20:41:25

by KOSAKI Motohiro

Subject: [PATCH] getrusage: RUSAGE_THREAD should return ru_utime and ru_stime

I believe that's bugfix. not new implement.

==
Subject: [PATCH] getrusage: RUSAGE_THREAD should return ru_utime and ru_stime
Impact: regression fix

Original getrusage(RUSAGE_THREAD) implementation can return ru_utime and ru_stime.
but unforunately commit f06febc96ba8e0af80bcc3eaec0a109e88275fac break it.

this patch restore it.


Signed-off-by: KOSAKI Motohiro <[email protected]>
CC: Roland McGrath <[email protected]>
CC: Sripathi Kodi <[email protected]>
Cc: Ingo Molnar <[email protected]>
CC: Frank Mayhar <[email protected]>
---
kernel/sys.c | 2 ++
1 file changed, 2 insertions(+)

Index: b/kernel/sys.c
===================================================================
--- a/kernel/sys.c 2009-01-04 05:01:15.000000000 +0900
+++ b/kernel/sys.c 2009-01-04 05:35:51.000000000 +0900
@@ -1551,6 +1551,8 @@ static void k_getrusage(struct task_stru
utime = stime = cputime_zero;

if (who == RUSAGE_THREAD) {
+ utime = task_utime(current);
+ stime = task_stime(current);
accumulate_thread_rusage(p, r);
goto out;
}

2009-01-03 21:37:08

by Roland McGrath

Subject: Re: [PATCH] getrusage: RUSAGE_THREAD should return ru_utime and ru_stime

I think you're right. Good catch.

Acked-by: Roland McGrath <[email protected]>


Thanks,
Roland

2009-01-04 13:04:44

by Ingo Molnar

Subject: Re: [PATCH] getrusage: RUSAGE_THREAD should return ru_utime and ru_stime

* Roland McGrath <[email protected]> wrote:

> I think you're right. Good catch.
>
> Acked-by: Roland McGrath <[email protected]>

applied to tip/sched/urgent, thanks guys!

Ingo