[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <1389031548.32504.29.camel@ppwaskie-mobl.amr.corp.intel.com>
Date: Mon, 6 Jan 2014 18:05:59 +0000
From: "Waskiewicz Jr, Peter P" <peter.p.waskiewicz.jr@...el.com>
To: Peter Zijlstra <peterz@...radead.org>
CC: Tejun Heo <tj@...nel.org>, Thomas Gleixner <tglx@...utronix.de>,
"Ingo Molnar" <mingo@...hat.com>, "H. Peter Anvin" <hpa@...or.com>,
Li Zefan <lizefan@...wei.com>,
"containers@...ts.linux-foundation.org"
<containers@...ts.linux-foundation.org>,
"cgroups@...r.kernel.org" <cgroups@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 0/4] x86: Add Cache QoS Monitoring (CQM) support
On Mon, 2014-01-06 at 18:53 +0100, Peter Zijlstra wrote:
> On Mon, Jan 06, 2014 at 04:47:57PM +0000, Waskiewicz Jr, Peter P wrote:
> > > Yeah that's not accurate, nor desired I think, because you get into
> > > horrible problems with hierarchies, do child groups belong to your RMID
> > > or not?
> >
> > I'd rather not support a child group of a child group. Only groups off
> > the root, and each group would be assigned an RMID when it's activated
> > for monitoring.
>
> Yeah, that's a complete non started for cgroups. Cgroups need to be
> completely hierarchical.
>
> So even the root group should represent all tasks; which if you fragment
> RMIDs on child cgroups doesn't work anymore.
The root group does represent all tasks in the current patchset on RMID
0. Then any child assigned to another group will be assigned to a
different RMID. It looks like this:
root (rmid 0)
/ \
(rmid 4) g1 g2 (rmid 16)
We could keep going down from there, but I don't see it buying anything
extra.
Cheers,
-PJ
--
PJ Waskiewicz Open Source Technology Center
peter.p.waskiewicz.jr@...el.com Intel Corp.
Powered by blists - more mailing lists