Re: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-02-17 Thread Thomas Gleixner
On Tue, 7 Feb 2017, Stephane Eranian wrote: > > I think the design must ensure that the following usage models can be > monitored: >- the allocations in your CAT partitions >- the allocations from a task (inclusive of children tasks) >- the allocations from a group of tasks (inclusive

Re: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-02-08 Thread Stephane Eranian
Tony, On Tue, Feb 7, 2017 at 10:52 AM, Luck, Tony wrote: > On Tue, Feb 07, 2017 at 12:08:09AM -0800, Stephane Eranian wrote: >> Hi, >> >> I wanted to take a few steps back and look at the overall goals for >> cache monitoring. >> From the various threads and discussion, my understanding is as fol

Re: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-02-08 Thread Peter Zijlstra
On Fri, Jan 20, 2017 at 12:11:53PM -0800, David Carrillo-Cisneros wrote: > Implementation ideas: > > First idea is to expose one monitoring file per resource in a CTRLGRP, > so the list of CTRLGRP's files would be: schemata, tasks, cpus, > monitor_l3_0, monitor_l3_1, ... > > the monitor_ file des

Re: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-02-08 Thread Peter Zijlstra
On Fri, Jan 20, 2017 at 03:51:48PM -0800, Shivappa Vikas wrote: > I think the email thread is going very long and we should just meet f2f > probably next week to iron out the requirements and chalk out a design > proposal. The thread isn't the problem; you lot not trimming your emails is however.

Re: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-02-07 Thread Shivappa Vikas
On Tue, 7 Feb 2017, Stephane Eranian wrote: Hi, I wanted to take a few steps back and look at the overall goals for cache monitoring. From the various threads and discussion, my understanding is as follows. I think the design must ensure that the following usage models can be monitored: -

Re: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-02-07 Thread Luck, Tony
On Tue, Feb 07, 2017 at 12:08:09AM -0800, Stephane Eranian wrote: > Hi, > > I wanted to take a few steps back and look at the overall goals for > cache monitoring. > From the various threads and discussion, my understanding is as follows. > > I think the design must ensure that the following usag

Re: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-02-07 Thread Stephane Eranian
Hi, I wanted to take a few steps back and look at the overall goals for cache monitoring. >From the various threads and discussion, my understanding is as follows. I think the design must ensure that the following usage models can be monitored: - the allocations in your CAT partitions - the

Re: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-02-06 Thread David Carrillo-Cisneros
On Mon, Feb 6, 2017 at 3:27 PM, Luck, Tony wrote: >> cgroup mode gives a per-CPU breakdown of event and running time, the >> tool aggregates it into running time vs event count. Both per-cpu >> breakdown and the aggregate are useful. >> >> Piggy-backing on perf's cgroup mode would give us all the

RE: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-02-06 Thread Luck, Tony
> cgroup mode gives a per-CPU breakdown of event and running time, the > tool aggregates it into running time vs event count. Both per-cpu > breakdown and the aggregate are useful. > > Piggy-backing on perf's cgroup mode would give us all the above for free. Do you have some sample output from a p

Re: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-02-06 Thread David Carrillo-Cisneros
On Mon, Feb 6, 2017 at 1:22 PM, Luck, Tony wrote: >> 12) Whatever fs or syscall is provided instead of perf syscalls, it >> should provide total_time_enabled in the way perf does, otherwise is >> hard to interpret MBM values. > > It seems that it is hard to define what we even mean by memory bandw

Re: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-02-06 Thread David Carrillo-Cisneros
On Mon, Feb 6, 2017 at 1:36 PM, Shivappa Vikas wrote: > > > On Mon, 6 Feb 2017, Luck, Tony wrote: > >>> 12) Whatever fs or syscall is provided instead of perf syscalls, it >>> should provide total_time_enabled in the way perf does, otherwise is >>> hard to interpret MBM values. >> >> >> It seems t

RE: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-02-06 Thread Shivappa Vikas
On Mon, 6 Feb 2017, Luck, Tony wrote: 12) Whatever fs or syscall is provided instead of perf syscalls, it should provide total_time_enabled in the way perf does, otherwise is hard to interpret MBM values. It seems that it is hard to define what we even mean by memory bandwidth. If you are m

RE: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-02-06 Thread Luck, Tony
> 12) Whatever fs or syscall is provided instead of perf syscalls, it > should provide total_time_enabled in the way perf does, otherwise is > hard to interpret MBM values. It seems that it is hard to define what we even mean by memory bandwidth. If you are measuring just one task and you find th

RE: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-02-06 Thread Luck, Tony
Digging through the e-mails from last week to generate a new version of the requirements I looked harder at this: > 12) Whatever fs or syscall is provided instead of perf syscalls, it > should provide total_time_enabled in the way perf does, otherwise is > hard to interpret MBM values. This looks

Re: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-02-03 Thread Luck, Tony
On Fri, Feb 03, 2017 at 01:08:05PM -0800, David Carrillo-Cisneros wrote: > On Fri, Feb 3, 2017 at 9:52 AM, Luck, Tony wrote: > > On Thu, Feb 02, 2017 at 06:14:05PM -0800, David Carrillo-Cisneros wrote: > >> If we tie allocation groups and monitoring groups, we are tying the > >> meaning of CPUs an

Re: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-02-03 Thread David Carrillo-Cisneros
On Fri, Feb 3, 2017 at 9:52 AM, Luck, Tony wrote: > On Thu, Feb 02, 2017 at 06:14:05PM -0800, David Carrillo-Cisneros wrote: >> If we tie allocation groups and monitoring groups, we are tying the >> meaning of CPUs and we'll have to choose between the CAT meaning or >> the perf meaning. >> >> Let'

Re: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-02-03 Thread Luck, Tony
On Thu, Feb 02, 2017 at 06:14:05PM -0800, David Carrillo-Cisneros wrote: > If we tie allocation groups and monitoring groups, we are tying the > meaning of CPUs and we'll have to choose between the CAT meaning or > the perf meaning. > > Let's allow semantics that will allow perf like monitoring to

Re: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-02-02 Thread David Carrillo-Cisneros
Something to be aware is that CAT cpus don't work the way CPU filtering works in perf: If I have the following CAT groups: - default group with task TD - group GC1 with CPU0 and CLOSID 1 - group GT1 with no CPUs and task T1 and CLOSID2 - TD and T1 run on CPU0. Then T1 will use CLOSID2 and TD

Re: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-02-02 Thread David Carrillo-Cisneros
On Thu, Feb 2, 2017 at 3:41 PM, Luck, Tony wrote: > On Thu, Feb 02, 2017 at 12:22:42PM -0800, David Carrillo-Cisneros wrote: >> There is no need to change perf(1) to support >> # perf stat -I 1000 -e intel_cqm/llc_occupancy {command} >> >> the PMU can work with resctrl to provide the support thro

Re: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-02-02 Thread Luck, Tony
On Thu, Feb 02, 2017 at 12:22:42PM -0800, David Carrillo-Cisneros wrote: > There is no need to change perf(1) to support > # perf stat -I 1000 -e intel_cqm/llc_occupancy {command} > > the PMU can work with resctrl to provide the support through > perf_event_open, with the advantage that tools oth

Re: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-02-02 Thread David Carrillo-Cisneros
On Thu, Feb 2, 2017 at 11:33 AM, Luck, Tony wrote: >>> Nice to have: >>> 1) Readout using "perf(1)" [subset of modes that make sense ... tying >>> monitoring >>> to resctrl file system will make most command line usage of perf(1) close >>> to impossible. >> >> >> We discussed this offline a

RE: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-02-02 Thread Shivappa Vikas
Hello Peterz/Andi, On Thu, 2 Feb 2017, Luck, Tony wrote: Nice to have: 1) Readout using "perf(1)" [subset of modes that make sense ... tying monitoring to resctrl file system will make most command line usage of perf(1) close to impossible. Vikas is pushing for "-R rdtgroup" ... thou

RE: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-02-02 Thread Luck, Tony
>> Nice to have: >> 1) Readout using "perf(1)" [subset of modes that make sense ... tying >> monitoring >> to resctrl file system will make most command line usage of perf(1) close to >> impossible. > > > We discussed this offline and I still disagree that it is close to > impossible to use

RE: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-02-02 Thread Shivappa Vikas
On Wed, 1 Feb 2017, Yu, Fenghua wrote: From: Andi Kleen [mailto:a...@firstfloor.org] "Luck, Tony" writes: 9) Measure per logical CPU (pick active RMID in same precedence for task/cpu as CAT picks CLOSID) 10) Put multiple CPUs into a group I'm not sure this is a real requirement.

RE: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-02-02 Thread Luck, Tony
>> 7) Must be able to measure based on existing resctrl CAT group >> 8) Can get measurements for subsets of tasks in a CAT group (to find >> the guys hogging the resources) >> 9) Measure per logical CPU (pick active RMID in same precedence for >> task/cpu as CAT picks CLOSID) > > I

RE: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-02-01 Thread Yu, Fenghua
> From: Andi Kleen [mailto:a...@firstfloor.org] > "Luck, Tony" writes: > > 9) Measure per logical CPU (pick active RMID in same precedence for > task/cpu as CAT picks CLOSID) > > 10) Put multiple CPUs into a group > > I'm not sure this is a real requirement. It's just an optimization, right? If

Re: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-02-01 Thread Andi Kleen
> > I'm not sure this is a real requirement. It's just an optimization, > > right? If you can assign policies to threads, you can implicitly set it > > per CPU through affinity (or the other way around). > > That's difficult when distinct users/systems do monitoring and system > management. What

Re: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-02-01 Thread David Carrillo-Cisneros
On Wed, Feb 1, 2017 at 4:35 PM, Andi Kleen wrote: > "Luck, Tony" writes: >> 9)Measure per logical CPU (pick active RMID in same precedence for >> task/cpu as CAT picks CLOSID) >> 10) Put multiple CPUs into a group > > I'm not sure this is a real requirement. It's just an optimization, > ri

Re: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-02-01 Thread Andi Kleen
"Luck, Tony" writes: > 9)Measure per logical CPU (pick active RMID in same precedence for > task/cpu as CAT picks CLOSID) > 10) Put multiple CPUs into a group I'm not sure this is a real requirement. It's just an optimization, right? If you can assign policies to threads, you can implicitl

Re: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-02-01 Thread David Carrillo-Cisneros
On Wed, Feb 1, 2017 at 12:08 PM Luck, Tony wrote: > > > I was asking for requirements, not a design proposal. In order to make a > > design you need a requirements specification. > > Here's what I came up with ... not a fully baked list, but should allow for > some useful > discussion on whether

RE: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-02-01 Thread Luck, Tony
> I was asking for requirements, not a design proposal. In order to make a > design you need a requirements specification. Here's what I came up with ... not a fully baked list, but should allow for some useful discussion on whether any of these are not really needed, or if there is a glaring ho

Re: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-01-23 Thread Peter Zijlstra
On Mon, Jan 23, 2017 at 10:47:44AM +0100, Thomas Gleixner wrote: > So again: > > Can please everyone involved write up their specific requirements > for CQM and stop spamming us with half baken design proposals? > > And I mean abstract requirements and not again something which is > refe

Re: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-01-23 Thread Thomas Gleixner
On Fri, 20 Jan 2017, David Carrillo-Cisneros wrote: > On Fri, Jan 20, 2017 at 5:29 AM Thomas Gleixner wrote: > > Can you please write up in a abstract way what the design requirements are > > that you need. So far we are talking about implementation details and > > unspecfied wishlists, but what w

Re: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-01-20 Thread Shivappa Vikas
On Fri, 20 Jan 2017, David Carrillo-Cisneros wrote: On Fri, Jan 20, 2017 at 1:08 PM, Shivappa Vikas wrote: On Fri, 20 Jan 2017, David Carrillo-Cisneros wrote: On Fri, Jan 20, 2017 at 5:29 AM Thomas Gleixner wrote: On Thu, 19 Jan 2017, David Carrillo-Cisneros wrote: If resctrl grou

Re: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-01-20 Thread David Carrillo-Cisneros
On Fri, Jan 20, 2017 at 1:08 PM, Shivappa Vikas wrote: > > > On Fri, 20 Jan 2017, David Carrillo-Cisneros wrote: > >> On Fri, Jan 20, 2017 at 5:29 AM Thomas Gleixner >> wrote: >>> >>> >>> On Thu, 19 Jan 2017, David Carrillo-Cisneros wrote: If resctrl groups could lift the restricti

Re: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-01-20 Thread Shivappa Vikas
On Fri, 20 Jan 2017, David Carrillo-Cisneros wrote: On Fri, Jan 20, 2017 at 5:29 AM Thomas Gleixner wrote: On Thu, 19 Jan 2017, David Carrillo-Cisneros wrote: If resctrl groups could lift the restriction of one resctl per CLOSID, then the user can create many resctrl in the way perf cgrou

Re: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-01-20 Thread Shivappa Vikas
On Thu, 19 Jan 2017, David Carrillo-Cisneros wrote: On Thu, Jan 19, 2017 at 6:32 PM, Vikas Shivappa wrote: Resending including Thomas , also with some changes. Sorry for the spam Based on Thomas and Peterz feedback Can think of two design variants which target: -Support monitoring and allo

Re: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-01-20 Thread David Carrillo-Cisneros
On Fri, Jan 20, 2017 at 12:30 AM, Thomas Gleixner wrote: > On Thu, 19 Jan 2017, David Carrillo-Cisneros wrote: >> On Thu, Jan 19, 2017 at 9:41 AM, Thomas Gleixner wrote: >> > Above you are talking about the same CLOSID and different RMIDS and not >> > about changing both. >> >> The scenario I tal

Re: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-01-20 Thread David Carrillo-Cisneros
On Fri, Jan 20, 2017 at 5:29 AM Thomas Gleixner wrote: > > On Thu, 19 Jan 2017, David Carrillo-Cisneros wrote: > > > > If resctrl groups could lift the restriction of one resctl per CLOSID, > > then the user can create many resctrl in the way perf cgroups are > > created now. The advantage is that

Re: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-01-20 Thread Stephane Eranian
On Thu, Jan 19, 2017 at 6:32 PM, Vikas Shivappa wrote: > > Resending including Thomas , also with some changes. Sorry for the spam > > Based on Thomas and Peterz feedback Can think of two design > variants which target: > > -Support monitoring and allocating using the same resctrl group. > user ca

Re: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-01-20 Thread Thomas Gleixner
On Thu, 19 Jan 2017, David Carrillo-Cisneros wrote: > > If resctrl groups could lift the restriction of one resctl per CLOSID, > then the user can create many resctrl in the way perf cgroups are > created now. The advantage is that there wont be cgroup hierarchy! > making things much simpler. Also

Re: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-01-20 Thread Thomas Gleixner
On Thu, 19 Jan 2017, David Carrillo-Cisneros wrote: > On Thu, Jan 19, 2017 at 9:41 AM, Thomas Gleixner wrote: > > Above you are talking about the same CLOSID and different RMIDS and not > > about changing both. > > The scenario I talked about implies changing CLOSID without affecting > monitoring

Re: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-01-19 Thread David Carrillo-Cisneros
On Thu, Jan 19, 2017 at 6:32 PM, Vikas Shivappa wrote: > Resending including Thomas , also with some changes. Sorry for the spam > > Based on Thomas and Peterz feedback Can think of two design > variants which target: > > -Support monitoring and allocating using the same resctrl group. > user can

Re: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-01-19 Thread David Carrillo-Cisneros
On Thu, Jan 19, 2017 at 9:41 AM, Thomas Gleixner wrote: > On Wed, 18 Jan 2017, David Carrillo-Cisneros wrote: >> On Wed, Jan 18, 2017 at 12:53 AM, Thomas Gleixner wrote: >> There are use cases where the RMID to CLOSID mapping is not that simple. >> Some of them are: >> >> 1. Fine-tuning of cache

Re: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-01-19 Thread Vikas Shivappa
Resending including Thomas , also with some changes. Sorry for the spam Based on Thomas and Peterz feedback Can think of two design variants which target: -Support monitoring and allocating using the same resctrl group. user can use a resctrl group to allocate resources and also monitor them (wi

Re: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-01-19 Thread Shivappa Vikas
Hello Peterz, On Wed, 18 Jan 2017, Peter Zijlstra wrote: On Wed, Jan 18, 2017 at 09:53:02AM +0100, Thomas Gleixner wrote: The whole approach you and David have taken is to whack some desired cgroup functionality and whatever into CQM without rethinking the overall design. And that's fundament

Re: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-01-19 Thread Thomas Gleixner
On Thu, 19 Jan 2017, David Carrillo-Cisneros wrote: > A 1:1 mapping between CLOSID/"Resource group" to RMID, as Fenghua suggested > is very problematic because the number of CLOSIDs is much much smaller than > the > number of RMIDs, and, as Stephane mentioned it's a common use case to want to > in

Re: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-01-19 Thread Thomas Gleixner
On Wed, 18 Jan 2017, Stephane Eranian wrote: > On Wed, Jan 18, 2017 at 12:53 AM, Thomas Gleixner wrote: > > > Your use case is specific to HPC and not Web workloads we run. Jobs run > in cgroups which may span all the CPUs of the machine. CAT may be used > to partition the cache. Cgroups would

Re: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-01-19 Thread Thomas Gleixner
On Wed, 18 Jan 2017, David Carrillo-Cisneros wrote: > On Wed, Jan 18, 2017 at 12:53 AM, Thomas Gleixner wrote: > There are use cases where the RMID to CLOSID mapping is not that simple. > Some of them are: > > 1. Fine-tuning of cache allocation. We may want to have a CLOSID for a thread > during p

Re: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-01-19 Thread David Carrillo-Cisneros
On Wed, Jan 18, 2017 at 6:09 PM, David Carrillo-Cisneros wrote: > On Wed, Jan 18, 2017 at 12:53 AM, Thomas Gleixner wrote: >> On Tue, 17 Jan 2017, Shivappa Vikas wrote: >>> On Tue, 17 Jan 2017, Thomas Gleixner wrote: >>> > On Fri, 6 Jan 2017, Vikas Shivappa wrote: >>> > > - Issue(1): Inaccurate d

Re: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-01-18 Thread Stephane Eranian
On Wed, Jan 18, 2017 at 12:53 AM, Thomas Gleixner wrote: > On Tue, 17 Jan 2017, Shivappa Vikas wrote: >> On Tue, 17 Jan 2017, Thomas Gleixner wrote: >> > On Fri, 6 Jan 2017, Vikas Shivappa wrote: >> > > - Issue(1): Inaccurate data for per package data, systemwide. Just prints >> > > zeros or arbit

Re: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-01-18 Thread Vikas Shivappa
Based on Thomas and Peterz feedback Can think of two variants which target: -Support monitoring and allocating using the same resctrl group. user can use a resctrl group to allocate resources and also monitor them (with respect to tasks or cpu) -allows 'task only' monitoring outside of resctrl.

Re: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-01-18 Thread David Carrillo-Cisneros
On Wed, Jan 18, 2017 at 12:53 AM, Thomas Gleixner wrote: > On Tue, 17 Jan 2017, Shivappa Vikas wrote: >> On Tue, 17 Jan 2017, Thomas Gleixner wrote: >> > On Fri, 6 Jan 2017, Vikas Shivappa wrote: >> > > - Issue(1): Inaccurate data for per package data, systemwide. Just prints >> > > zeros or arbit

RE: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-01-18 Thread Yu, Fenghua
> From: Thomas Gleixner [mailto:t...@linutronix.de] > On Tue, 17 Jan 2017, Shivappa Vikas wrote: > > On Tue, 17 Jan 2017, Thomas Gleixner wrote: > > > On Fri, 6 Jan 2017, Vikas Shivappa wrote: > > > > - Issue(1): Inaccurate data for per package data, systemwide. Just > > > > prints zeros or arbitra

Re: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-01-18 Thread David Carrillo-Cisneros
On Wed, Jan 18, 2017 at 12:53 AM, Thomas Gleixner wrote: > On Tue, 17 Jan 2017, Shivappa Vikas wrote: >> On Tue, 17 Jan 2017, Thomas Gleixner wrote: >> > On Fri, 6 Jan 2017, Vikas Shivappa wrote: >> > > - Issue(1): Inaccurate data for per package data, systemwide. Just prints >> > > zeros or arbit

Re: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-01-18 Thread Shivappa Vikas
On Wed, 18 Jan 2017, Thomas Gleixner wrote: On Tue, 17 Jan 2017, Shivappa Vikas wrote: On Tue, 17 Jan 2017, Thomas Gleixner wrote: On Fri, 6 Jan 2017, Vikas Shivappa wrote: - Issue(1): Inaccurate data for per package data, systemwide. Just prints zeros or arbitrary numbers. Fix: Patches fi

Re: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-01-18 Thread Peter Zijlstra
On Wed, Jan 18, 2017 at 09:53:02AM +0100, Thomas Gleixner wrote: > The whole approach you and David have taken is to whack some desired cgroup > functionality and whatever into CQM without rethinking the overall > design. And that's fundamentaly broken because it does not take cache (and > memory b

Re: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-01-18 Thread Thomas Gleixner
On Tue, 17 Jan 2017, Shivappa Vikas wrote: > On Tue, 17 Jan 2017, Thomas Gleixner wrote: > > On Fri, 6 Jan 2017, Vikas Shivappa wrote: > > > - Issue(1): Inaccurate data for per package data, systemwide. Just prints > > > zeros or arbitrary numbers. > > > > > > Fix: Patches fix this by just throwin

Re: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-01-17 Thread Shivappa Vikas
On Tue, 17 Jan 2017, Thomas Gleixner wrote: On Fri, 6 Jan 2017, Vikas Shivappa wrote: Cqm(cache quality monitoring) is part of Intel RDT(resource director technology) which enables monitoring and controlling of processor shared resources via MSR interface. We know that already. No need for

Re: [PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-01-17 Thread Thomas Gleixner
On Fri, 6 Jan 2017, Vikas Shivappa wrote: > Cqm(cache quality monitoring) is part of Intel RDT(resource director > technology) which enables monitoring and controlling of processor shared > resources via MSR interface. We know that already. No need for advertising this over and over. > Below are

[PATCH 00/12] Cqm2: Intel Cache quality monitoring fixes

2017-01-06 Thread Vikas Shivappa
Resending version 5 with updated send list. Sorry for the spam. Cqm(cache quality monitoring) is part of Intel RDT(resource director technology) which enables monitoring and controlling of processor shared resources via MSR interface. The current upstream cqm(Cache monitoring) has major issues wh