Re: RFC: /proc/<pid>/sched should contain cumulative data for allthreads in process

From: Peter Zijlstra
Date: Tue Feb 08 2011 - 08:44:42 EST


On Tue, 2011-02-08 at 13:11 +0100, Ingo Molnar wrote:
> * Peter Zijlstra <peterz@xxxxxxxxxxxxx> wrote:
>
> > On Mon, 2011-02-07 at 16:29 -0600, Chris Friesen wrote:
> > > Hi,
> > >
> > > We've got a tool that gathers lots of scheduling data for each process
> > > (not task/thread) on the system.
> > >
> > > For /proc/<pid>/{stat,io} this is straightforward, as the per-thread
> > > values are summed together for the process as a whole.
> > >
> > > However, /proc/<pid>/sched only shows the data for the individual thread
> > > with the same tid as the pid. To get a per-process view we need to
> > > manually scan all the threads and sum them--and this can get expensive
> > > due to all the extra file operations, parsing, etc.
> > >
> > > Was this a concious design decision, or just an oversight? Would a
> > > patch converting it to whole-process values be accepted or is it enough
> > > of a standard interface that we can't break existing apps that expect
> > > the current behaviour?
> >
> > I'd as soon remove all that stuff than extend it, its an abi liability,
> > esp since you're talking about tools parsing this stuff.
>
> So assuming a tool would want to capture such stats of the system, what would be its
> options? Could we do all this via system-wide counters and perf stat alike cheap,
> transparent gathering without having to patch/rebuild the kernel?

Very much depends on what is wanted, but most of the stuff inside those
files is very specific to the implementation and pinning any of that to
an ABI is like silly.

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