lkml.org 
[lkml]   [2013]   [Jan]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v5 11/11] sched: introduce cgroup file stat_percpu
On Wed,  9 Jan 2013 15:45:38 +0400
Glauber Costa <glommer@parallels.com> wrote:

> The file cpu.stat_percpu will show various scheduler related
> information, that are usually available to the top level through other
> files.
>
> For instance, most of the meaningful data in /proc/stat is presented
> here. Given this file, a container can easily construct a local copy of
> /proc/stat for internal consumption.
>
> The data we export is comprised of:
> * all the tick information, previously available only through cpuacct,
> like user time, system time, etc.
>
> * wait time, which can be used to construct analogous information to
> steal time in hypervisors,
>
> * nr_switches and nr_running, which are cgroup-local versions of
> their global counterparts.
>
> The file includes a header, so fields can come and go if needed.

Please update this changelog to fully describe the proposed userspace
interfaces. That means full pathnames and example output.
Understanding these interfaces is the most important part of reviewing
this patchset, so this info should be prominent.

Also, this patchset appears to alter (or remove?) existing userspace
interfaces? If so, please clearly describe those alterations and also
share your thinking on the back-compatibility issues.

Also, I'm not seeing any changes to Docmentation/ in this patchset.
How do we explain the interface to our users?


From a quick read, it appears that the output will be something along
the lines of:

user nice system irq softirq guest guest_nice wait nr_switches nr_running
cpu0 nn nn nn nn nn ...
cpu1 nn nn nn nn nn ...

which looks pretty terrible. Apart from being very hard to read, it
means that we can never remove fields. A nicer and more extensible
interface would be

cpu:0 nice:nn system:nn irq:nn ...





\
 
 \ /
  Last update: 2013-01-09 22:41    [W:0.178 / U:0.264 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site