Re: [PATCH 1/1] x86/cqm: Cqm requirements

2017-03-13 Thread Thomas Gleixner
On Mon, 13 Mar 2017, David Carrillo-Cisneros wrote: > >> I am ok removing the perf-like CPU filtering from the requirements. > > > > So if I'm not missing something then ALL remaining requirements can be > > solved with the RDT integrated monitoring mechanics, right? > > > > Right. Excellent.

Re: [PATCH 1/1] x86/cqm: Cqm requirements

2017-03-13 Thread David Carrillo-Cisneros
>> I am ok removing the perf-like CPU filtering from the requirements. > > So if I'm not missing something then ALL remaining requirements can be > solved with the RDT integrated monitoring mechanics, right? > Right.

Re: [PATCH 1/1] x86/cqm: Cqm requirements

2017-03-13 Thread Thomas Gleixner
On Fri, 10 Mar 2017, David Carrillo-Cisneros wrote: > > Fine. So we need this for ONE particular use case. And if that is not well > > documented including the underlying mechanics to analyze the data then this > > will be a nice source of confusion for Joe User. > > > > I still think that this can

Re: [PATCH 1/1] x86/cqm: Cqm requirements

2017-03-10 Thread David Carrillo-Cisneros
> > Fine. So we need this for ONE particular use case. And if that is not well > documented including the underlying mechanics to analyze the data then this > will be a nice source of confusion for Joe User. > > I still think that this can be done differently while keeping the overhead > small. > >

Re: [PATCH 1/1] x86/cqm: Cqm requirements

2017-03-10 Thread Thomas Gleixner
On Thu, 9 Mar 2017, David Carrillo-Cisneros wrote: > On Thu, Mar 9, 2017 at 3:01 AM, Thomas Gleixner wrote: > > On Wed, 8 Mar 2017, David Carrillo-Cisneros wrote: > >> On Wed, Mar 8, 2017 at 12:30 AM, Thomas Gleixner > >> wrote: > >> > Same applies for per CPU measurements. > >> > >> For CPU mea

Re: [PATCH 1/1] x86/cqm: Cqm requirements

2017-03-09 Thread David Carrillo-Cisneros
On Thu, Mar 9, 2017 at 3:01 AM, Thomas Gleixner wrote: > On Wed, 8 Mar 2017, David Carrillo-Cisneros wrote: >> On Wed, Mar 8, 2017 at 12:30 AM, Thomas Gleixner wrote: >> > Same applies for per CPU measurements. >> >> For CPU measurements. We need perf-like CPU filtering to support tools >> that p

Re: [PATCH 1/1] x86/cqm: Cqm requirements

2017-03-09 Thread Thomas Gleixner
On Wed, 8 Mar 2017, David Carrillo-Cisneros wrote: > On Wed, Mar 8, 2017 at 12:30 AM, Thomas Gleixner wrote: > > Same applies for per CPU measurements. > > For CPU measurements. We need perf-like CPU filtering to support tools > that perform low overhead monitoring by polling CPU events. These >

Re: [PATCH 1/1] x86/cqm: Cqm requirements

2017-03-08 Thread David Carrillo-Cisneros
On Wed, Mar 8, 2017 at 12:30 AM, Thomas Gleixner wrote: > Stephane, > > On Tue, 7 Mar 2017, Stephane Eranian wrote: >> On Tue, Mar 7, 2017 at 12:04 PM, Luck, Tony wrote: >> >> That's all nice and good, but I still have no coherent explanation why >> >> measuring across allocation domains makes se

Re: [PATCH 1/1] x86/cqm: Cqm requirements

2017-03-08 Thread Thomas Gleixner
Stephane, On Tue, 7 Mar 2017, Stephane Eranian wrote: > On Tue, Mar 7, 2017 at 12:04 PM, Luck, Tony wrote: > >> That's all nice and good, but I still have no coherent explanation why > >> measuring across allocation domains makes sense. > > > > Is this in reaction to this one? > > > >>> 5) P

Re: [PATCH 1/1] x86/cqm: Cqm requirements

2017-03-07 Thread Shivappa Vikas
On Tue, 7 Mar 2017, Stephane Eranian wrote: Hi, On Tue, Mar 7, 2017 at 12:04 PM, Luck, Tony wrote: That's all nice and good, but I still have no coherent explanation why measuring across allocation domains makes sense. Is this in reaction to this one? 5) Put multiple threads into a

Re: [PATCH 1/1] x86/cqm: Cqm requirements

2017-03-07 Thread Stephane Eranian
Hi, On Tue, Mar 7, 2017 at 12:04 PM, Luck, Tony wrote: >> That's all nice and good, but I still have no coherent explanation why >> measuring across allocation domains makes sense. > > Is this in reaction to this one? > >>> 5) Put multiple threads into a single measurement group > > If we fi

[PATCH 1/1] x86/cqm: Cqm requirements

2017-03-07 Thread Vikas Shivappa
Sending the cqm requirements as per Thomas comments in the previous verson of cqm patch series - https://marc.info/?l=linux-kernel&m=148374167726502 This is modified version of requirements sent by Tony in the same thread with inputs from David and Stephan. Reviewed-by: Tony Luck Reviewed-by: Dav

RE: [PATCH 1/1] x86/cqm: Cqm requirements

2017-03-07 Thread Thomas Gleixner
On Tue, 7 Mar 2017, Luck, Tony wrote: > > That's all nice and good, but I still have no coherent explanation why > > measuring across allocation domains makes sense. > > Is this in reaction to this one? > > >> 5) Put multiple threads into a single measurement group > > If we fix it to say

Re: [PATCH 1/1] x86/cqm: Cqm requirements

2017-03-07 Thread Thomas Gleixner
On Tue, 7 Mar 2017, Vikas Shivappa wrote: > Sending the cqm requirements as per Thomas comments in the previous > verson of cqm patch series - > https://marc.info/?l=linux-kernel&m=148374167726502 > > This is modified version of requirements sent by Tony in the same > thread with inputs from Davi

RE: [PATCH 1/1] x86/cqm: Cqm requirements

2017-03-07 Thread Luck, Tony
> That's all nice and good, but I still have no coherent explanation why > measuring across allocation domains makes sense. Is this in reaction to this one? >> 5) Put multiple threads into a single measurement group If we fix it to say "threads from the same CAT group" does it fix things?