lkml.org 
[lkml]   [2009]   [Mar]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH -tip] cpuacct: per-cgroup utime/stime statistics - v6
* Bharata B Rao <bharata@linux.vnet.ibm.com> [2009-03-25 10:09:50]:

> Here is the next version which fixes the bug pointed by Li Zefan in v5.
>
> We currently have two stats named stime and utime. In future if we
> decide to have more fine grained stats like time spent in irq, softirq
> etc, it will be difficult to name them similar to stime and utime. Hence
> would it make sense to name these stats as system and user so that we
> could easily add nice, irq, steal, softirq etc in future ?
>
> Regards,
> Bharata.
>
> cpuacct: Add stime and utime statistics
>
> Add per-cgroup cpuacct controller statistics like the system and user
> time consumed by the group of tasks.
>
> Changelog:
>
> v6
> - Fixed a bug in the error path of cpuacct_create() (pointed by Li Zefan).
>
> v5
> - In cpuacct_stats_show(), use cputime64_to_clock_t() since we are
> operating on a 64bit variable here.
>
> v4
> - Remove comments in cpuacct_update_stats() which explained why rcu_read_lock()
> was needed (as per Peter Zijlstra's review comments).
> - Don't say that percpu_counter_read() is broken in Documentation/cpuacct.txt
> as per KAMEZAWA Hiroyuki's review comments.
>
> v3
> - Fix a small race in the cpuacct hierarchy walk.
>
> v2
> - stime and utime now exported in clock_t units instead of msecs.
> - Addressed the code review comments from Balbir and Li Zefan.
> - Moved to -tip tree.
>
> v1
> - Moved the stime/utime accounting to cpuacct controller.
>
> Earlier versions
> - http://lkml.org/lkml/2009/2/25/129
>
> Signed-off-by: Bharata B Rao <bharata@linux.vnet.ibm.com>
> Signed-off-by: Balaji Rao <balajirrao@gmail.com>
> Cc: Dhaval Giani <dhaval@linux.vnet.ibm.com>
> Cc: Paul Menage <menage@google.com>
> Cc: Andrew Morton <akpm@linux-foundation.org>
> Cc: Ingo Molnar <mingo@elte.hu>
> Cc: KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>
> Reviewed-by: Li Zefan <lizf@cn.fujitsu.com>
> Acked-by: Peter Zijlstra <a.p.zijlstra@chello.nl>
> Acked-by: Balbir Singh <balbir@linux.vnet.ibm.com>
> Tested-by: Balbir Singh <balbir@linux.vnet.ibm.com>
> ---
> Documentation/cgroups/cpuacct.txt | 18 +++++++
> kernel/sched.c | 87 +++++++++++++++++++++++++++++++++++---
> 2 files changed, 99 insertions(+), 6 deletions(-)
>
> --- a/Documentation/cgroups/cpuacct.txt
> +++ b/Documentation/cgroups/cpuacct.txt
> @@ -30,3 +30,21 @@ The above steps create a new group g1 an
> process (bash) into it. CPU time consumed by this bash and its children
> can be obtained from g1/cpuacct.usage and the same is accumulated in
> /cgroups/cpuacct.usage also.
> +
> +cpuacct.stat file lists a few statistics which further divide the
> +CPU time obtained by the cgroup into user and system times. Currently
> +the following statistics are supported:
> +
> +utime: Time spent by tasks of the cgroup in user mode.
> +stime: Time spent by tasks of the cgroup in kernel mode.
> +

Bharata can we move away from stime, utime (names) to system and user with
time being an implicit unit. I would like to see softirq, hardirq, etc
times in the future being added.

--
Balbir


\
 
 \ /
  Last update: 2009-03-30 11:23    [from the cache]
©2003-2011 Jasper Spaans