linux-kernel - Re: [PATCH -tip] cpuacct: per-cgroup utime/stime statistics

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20090331043317.GB4093@in.ibm.com>
Date:	Tue, 31 Mar 2009 10:03:17 +0530
From:	Bharata B Rao <bharata@...ux.vnet.ibm.com>
To:	Balbir Singh <balbir@...ux.vnet.ibm.com>
Cc:	linux-kernel@...r.kernel.org, Balaji Rao <balajirrao@...il.com>,
	Dhaval Giani <dhaval@...ux.vnet.ibm.com>,
	Li Zefan <lizf@...fujitsu.com>,
	Paul Menage <menage@...gle.com>,
	Andrew Morton <akpm@...ux-foundation.org>,
	Ingo Molnar <mingo@...e.hu>,
	Peter Zijlstra <a.p.zijlstra@...llo.nl>,
	KAMEZAWA Hiroyuki <kamezawa.hiroyu@...fujitsu.com>
Subject: Re: [PATCH -tip] cpuacct: per-cgroup utime/stime statistics - v6

On Mon, Mar 30, 2009 at 02:50:02PM +0530, Balbir Singh wrote:
> * Bharata B Rao <bharata@...ux.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@...ux.vnet.ibm.com>
> > Signed-off-by: Balaji Rao <balajirrao@...il.com>
> > Cc: Dhaval Giani <dhaval@...ux.vnet.ibm.com>
> > Cc: Paul Menage <menage@...gle.com>
> > Cc: Andrew Morton <akpm@...ux-foundation.org>
> > Cc: Ingo Molnar <mingo@...e.hu>
> > Cc: KAMEZAWA Hiroyuki <kamezawa.hiroyu@...fujitsu.com>
> > Reviewed-by: Li Zefan <lizf@...fujitsu.com>
> > Acked-by: Peter Zijlstra <a.p.zijlstra@...llo.nl>
> > Acked-by: Balbir Singh <balbir@...ux.vnet.ibm.com>
> > Tested-by: Balbir Singh <balbir@...ux.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.

Done. Posted next version with this change.

Regards,
Bharata.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/