Re: [PATCH v7 12/13] selftests/resctrl: Disable MBA and MBM tests for AMD

2019-05-14 Thread Moger, Babu
Fenghua/Andre, On 5/10/19 2:29 PM, Yu, Fenghua wrote: > [CAUTION: External Email] > >> On Friday, May 10, 2019 10:40 AM >> Andre Przywara [mailto:andre.przyw...@arm.com] wrote: >> To: Yu, Fenghua >> Cc: Thomas Gleixner ; Ingo Molnar >> ; Borislav Petkov ; H Peter Anvin >> ; Luck, Tony ;

RE: [PATCH v7 00/13] selftests/resctrl: Add resctrl selftest

2019-03-06 Thread Moger, Babu
> ; Ravi V Shankar ; > Xiaochen Shen ; Arshiya Hayatkhan Pathan > ; Sai Praneeth Prakhya > ; Moger, Babu > Cc: linux-kernel > Subject: Re: [PATCH v7 00/13] selftests/resctrl: Add resctrl selftest > > On Sat, Feb 09, 2019 at 06:50:29PM -0800, Fenghua Yu wrote: > > Wit

RE: [PATCH v7 00/13] selftests/resctrl: Add resctrl selftest

2019-02-27 Thread Moger, Babu
kar ; > Xiaochen Shen ; Arshiya Hayatkhan Pathan > ; Sai Praneeth Prakhya > ; Moger, Babu > Cc: linux-kernel ; Fenghua Yu > > Subject: [PATCH v7 00/13] selftests/resctrl: Add resctrl selftest > > With more and more resctrl features are being added by Intel, AMD > and A

[PATCH v5 01/13] selftests/resctrl: Add README for resctrl tests

2019-01-18 Thread Moger, Babu
From: Fenghua Yu resctrl tests will be implemented. README is added for the tool first. Signed-off-by: Fenghua Yu Signed-off-by: Babu Moger --- tools/testing/selftests/resctrl/README | 53 ++ 1 file changed, 53 insertions(+) create mode 100644

[PATCH v5 10/13] selftests/resctrl: Add the test in MAINTAINERS

2019-01-18 Thread Moger, Babu
From: Fenghua Yu The resctrl selftest will be maintained by RDT maintainers. Signed-off-by: Fenghua Yu Signed-off-by: Babu Moger --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 32d7..16359bc 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@

[PATCH v5 09/13] selftests/resctrl: Add Cache Allocation Technology (CAT) selftest

2019-01-18 Thread Moger, Babu
From: Arshiya Hayatkhan Pathan Cache Allocation Technology (CAT) selftest allocates a portion of last level cache and starts a benchmark to read each cache line in this portion of cache. Measure the cache misses in perf and the misses should be equal to the number of cache lines in this portion

[PATCH v5 05/13] selftests/resctrl: Add built in benchmark

2019-01-18 Thread Moger, Babu
From: Sai Praneeth Prakhya Built-in benchmark fill_buf generates stressful memory bandwidth and cache traffic. Later it will be used as a default benchmark by various resctrl tests such as MBA (Memory Bandwidth Allocation) and MBM (Memory Bandwidth Monitoring) tests. Signed-off-by: Sai

[PATCH v5 12/13] selftests/resctrl: Use cache index3 id for AMD schemata masks

2019-01-18 Thread Moger, Babu
AMD uses the cache l3 boundary for schemata masks. Update it accordigly. Signed-off-by: Babu Moger --- tools/testing/selftests/resctrl/resctrlfs.c | 9 +++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/tools/testing/selftests/resctrl/resctrlfs.c

[PATCH v5 13/13] selftests/resctrl: Disable MBA and MBM tests for AMD

2019-01-18 Thread Moger, Babu
For now, disable MBA and MBM tests for AMD. Deciding test pass/fail is not clear right now. We can enable when we have some clarity. Signed-off-by: Babu Moger --- tools/testing/selftests/resctrl/cat_test.c | 2 +- tools/testing/selftests/resctrl/resctrl_tests.c | 4 ++-- 2 files changed, 3

[PATCH v5 06/13] selftests/resctrl: Add MBM test

2019-01-18 Thread Moger, Babu
From: Arshiya Hayatkhan Pathan MBM (Memory Bandwidth Monitoring) test is the first implemented selftest. It starts a stressful memory bandwidth benchmark and assigns the bandwidth pid in a resctrl monitoring group. Read and compare perf IMC counter and MBM total bytes for the benchmark. The

[PATCH v5 07/13] selftests/resctrl: Add MBA test

2019-01-18 Thread Moger, Babu
From: Arshiya Hayatkhan Pathan MBA (Memory Bandwidth Allocation) test starts a stressful memory bandwidth benchmark and allocates memory bandwidth from 100% down to 10% for the benchmark process. For each allocation, compare perf IMC counter and mbm total bytes from resctrl. The difference

[PATCH v5 04/13] selftests/resctrl: Add callback to start a benchmark

2019-01-18 Thread Moger, Babu
From: Sai Praneeth Prakhya The callback starts a child process and puts the child pid in created resctrl group with specified memory bandwidth in schemata. The child starts running benchmark. Signed-off-by: Sai Praneeth Prakhya Signed-off-by: Arshiya Hayatkhan Pathan Signed-off-by: Fenghua Yu

[PATCH v5 11/13] selftests/resctrl: Add vendor detection mechanism

2019-01-18 Thread Moger, Babu
RESCTRL feature is supported both on Intel and AMD now. Some features are implemented differently. Add vendor detection mechanism. Use the vendor check where there are differences. Signed-off-by: Babu Moger --- tools/testing/selftests/resctrl/resctrl.h | 13 +

[PATCH v5 00/13] selftests/resctrl: Add resctrl selftest

2019-01-18 Thread Moger, Babu
With more and more resctrl features are being added by Intel, AMD and ARM, a test tool is becoming more and more useful to validate that both hardware and software functionalities work as expected. We introduce resctrl selftest to cover resctrl features on both X86 and ARM architectures. It first

[PATCH v5 08/13] selftests/resctrl Add Cache QoS Monitoring (CQM) selftest

2019-01-18 Thread Moger, Babu
From: Arshiya Hayatkhan Pathan Cache QoS Monitoring (CQM) selftest starts stressful cache benchmark with specified size of memory to access the cache. Last Level cache occupancy reported by CQM should be close to the size of the memory. Signed-off-by: Arshiya Hayatkhan Pathan Signed-off-by:

[PATCH v5 02/13] selftests/resctrl: Add basic resctrl file system operations and data

2019-01-18 Thread Moger, Babu
From: Sai Praneeth Prakhya The basic resctrl file system operations and data are added for future usage by resctrl selftest tool. Signed-off-by: Sai Praneeth Prakhya Signed-off-by: Arshiya Hayatkhan Pathan Signed-off-by: Fenghua Yu Signed-off-by: Babu Moger ---

[PATCH v5 03/13] selftests/resctrl: Read memory bandwidth from perf IMC counter and from resctrl file system

2019-01-18 Thread Moger, Babu
From: Sai Praneeth Prakhya Total memory bandwidth can be monitored from perf IMC counter and from resctrl file system. Later the two will be compared to verify the total memory bandwidth read from resctrl is correct. Signed-off-by: Sai Praneeth Prakhya Signed-off-by: Arshiya Hayatkhan Pathan

Re: [PATCH v4 00/10] selftests/resctrl: Add resctrl selftest

2019-01-16 Thread Moger, Babu
On 1/16/19 1:13 PM, Fenghua Yu wrote: > On Wed, Jan 16, 2019 at 03:54:07PM +0000, Moger, Babu wrote: >> Hi Fenghua/Sai/Arshiya, >> We were thinking of updating these selftests to work on both Intel and >> AMD. What do you guys think? >> >> If that is ok, I can

Re: [PATCH v4 00/10] selftests/resctrl: Add resctrl selftest

2019-01-16 Thread Moger, Babu
Hi Fenghua/Sai/Arshiya, We were thinking of updating these selftests to work on both Intel and AMD. What do you guys think? If that is ok, I can update these patches and resubmit. If you have already updated, please post the latest series. I will use the latest version. Please let me know.

Re: [PATCH v4 09/10] selftests/resctrl: Add Cache Allocation Technology (CAT) selftest

2019-01-14 Thread Moger, Babu
Hi Fenghua/Sai/Arshiya, Few comments on this patch below. Sorry for the late comment. On 12/21/18 6:20 PM, Fenghua Yu wrote: > From: Arshiya Hayatkhan Pathan > > Cache Allocation Technology (CAT) selftest allocates a portion of > last level cache and starts a benchmark to read each cache >

RE: [PATCH] x86/cache: Rename config option to CONFIG_X86_RESCTRL

2019-01-09 Thread Moger, Babu
Looks good to me. Sanity tested the patch. > -Original Message- > From: Borislav Petkov > Sent: Tuesday, January 8, 2019 11:14 AM > To: Ingo Molnar > Cc: Linus Torvalds ; x86-ml > ; lkml ; James Morse > ; Moger, Babu ; > Fenghua Yu ; Reinette Chatre > ; Tony L

RE: [PATCH v4 10/10] selftests/resctrl: Add the test in MAINTAINERS

2019-01-02 Thread Moger, Babu
Hi Fenghua, > -Original Message- > From: Fenghua Yu > Sent: Friday, December 21, 2018 6:21 PM > To: Thomas Gleixner ; Ingo Molnar > ; H Peter Anvin ; Tony Luck > ; Peter Zijlstra ; Reinette > Chatre ; Moger, Babu > ; James Morse ; > Xiaochen Shen ; Ravi V Shank

RE: [PATCH v4 08/10] selftests/resctrl Add Cache QoS Monitoring (CQM) selftest

2019-01-02 Thread Moger, Babu
Fenghua, Sai, Couple of problems with these patches(see below). Please check again. > -Original Message- > From: Fenghua Yu > Sent: Friday, December 21, 2018 6:21 PM > To: Thomas Gleixner ; Ingo Molnar > ; H Peter Anvin ; Tony Luck > ; Peter Zijlstra ; Reinette >

RE: [PATCH v9 00/13] arch/resctrl: AMD QoS support

2018-12-26 Thread Moger, Babu
Hi Jan, > -Original Message- > From: Jan Engelhardt > Sent: Sunday, December 23, 2018 12:27 PM > To: Moger, Babu > Cc: t...@linutronix.de; mi...@redhat.com; b...@alien8.de; cor...@lwn.net; > fenghua...@intel.com; reinette.cha...@intel.com; pet...@in

[PATCH v3] x86/resctrl: Remove unnecessary check for cbm_validate

2018-11-28 Thread Moger, Babu
The static checker(smatch) reports the following error after the commit a36c5ff560fb ("x86/resctrl: Bring cbm_validate() into the resource structure"). arch/x86/kernel/cpu/resctrl//ctrlmondata.c:227 parse_cbm() error: uninitialized symbol 'cbm_val'. arch/x86/kernel/cpu/resctrl//ctrlmondata.c:236

[PATCH v3] x86/resctrl: Remove unnecessary check for cbm_validate

2018-11-28 Thread Moger, Babu
The static checker(smatch) reports the following error after the commit a36c5ff560fb ("x86/resctrl: Bring cbm_validate() into the resource structure"). arch/x86/kernel/cpu/resctrl//ctrlmondata.c:227 parse_cbm() error: uninitialized symbol 'cbm_val'. arch/x86/kernel/cpu/resctrl//ctrlmondata.c:236

RE: [PATCH] x86/resctrl: fix the uninitialized variable warning

2018-11-28 Thread Moger, Babu
Boris, > -Original Message- > From: Borislav Petkov > Sent: Wednesday, November 28, 2018 2:30 PM > To: Moger, Babu > Cc: t...@linutronix.de; mi...@redhat.com; h...@zytor.com; > x...@kernel.org; fenghua...@intel.com; reinette.cha...@intel.com; > dan.carpen...@ora

RE: [PATCH] x86/resctrl: fix the uninitialized variable warning

2018-11-28 Thread Moger, Babu
Boris, > -Original Message- > From: Borislav Petkov > Sent: Wednesday, November 28, 2018 2:30 PM > To: Moger, Babu > Cc: t...@linutronix.de; mi...@redhat.com; h...@zytor.com; > x...@kernel.org; fenghua...@intel.com; reinette.cha...@intel.com; > dan.carpen...@ora

[PATCH v2] x86/resctrl: fix the uninitialized variable warning

2018-11-28 Thread Moger, Babu
Fix the following compiler warning caused by recent change. arch/x86/kernel/cpu/resctrl/ctrlmondata.c:227 parse_cbm() error: uninitialized symbol 'cbm_val' Fixes: a36c5ff560fb ("x86/resctrl: Bring cbm_validate() into the resource structure") Reported-by: Dan Carpenter Signed-off-by: Babu

[PATCH v2] x86/resctrl: fix the uninitialized variable warning

2018-11-28 Thread Moger, Babu
Fix the following compiler warning caused by recent change. arch/x86/kernel/cpu/resctrl/ctrlmondata.c:227 parse_cbm() error: uninitialized symbol 'cbm_val' Fixes: a36c5ff560fb ("x86/resctrl: Bring cbm_validate() into the resource structure") Reported-by: Dan Carpenter Signed-off-by: Babu

[PATCH] x86/resctrl: fix the uninitialized variable warning

2018-11-28 Thread Moger, Babu
Fix the compiler warning caused by a recent change. Fixes: a36c5ff560fb ("x86/resctrl: Bring cbm_validate() into the resource structure") Reported-by: Dan Carpenter Signed-off-by: Babu Moger --- arch/x86/kernel/cpu/resctrl/ctrlmondata.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

[PATCH] x86/resctrl: fix the uninitialized variable warning

2018-11-28 Thread Moger, Babu
Fix the compiler warning caused by a recent change. Fixes: a36c5ff560fb ("x86/resctrl: Bring cbm_validate() into the resource structure") Reported-by: Dan Carpenter Signed-off-by: Babu Moger --- arch/x86/kernel/cpu/resctrl/ctrlmondata.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

RE: [PATCH] KVM: VMX: re-add ple_gap module parameter

2018-11-28 Thread Moger, Babu
My bad.. Sorry about this. I think this should also go to sta...@vger.kernel.org > -Original Message- > From: Luiz Capitulino > Sent: Friday, November 23, 2018 12:27 PM > To: Liran Alon > Cc: Paolo Bonzini ; Moger, Babu > ; k...@vger.kernel.org; linux- > ker...@vger

RE: [PATCH] KVM: VMX: re-add ple_gap module parameter

2018-11-28 Thread Moger, Babu
My bad.. Sorry about this. I think this should also go to sta...@vger.kernel.org > -Original Message- > From: Luiz Capitulino > Sent: Friday, November 23, 2018 12:27 PM > To: Liran Alon > Cc: Paolo Bonzini ; Moger, Babu > ; k...@vger.kernel.org; linux- > ker...@vger

Re: [PATCH v3 3/8] selftests/resctrl: Read memory bandwidth from perf IMC counter and from resctrl file system

2018-11-12 Thread Moger, Babu
On 10/31/18 4:02 PM, Fenghua Yu wrote: > From: Sai Praneeth Prakhya > > Total memory bandwidth can be monitored from perf IMC counter and from > resctrl file system. Later the two will be compared to verify the total > memory bandwidth read from resctrl is correct. > > Signed-off-by: Sai

Re: [PATCH v3 3/8] selftests/resctrl: Read memory bandwidth from perf IMC counter and from resctrl file system

2018-11-12 Thread Moger, Babu
On 10/31/18 4:02 PM, Fenghua Yu wrote: > From: Sai Praneeth Prakhya > > Total memory bandwidth can be monitored from perf IMC counter and from > resctrl file system. Later the two will be compared to verify the total > memory bandwidth read from resctrl is correct. > > Signed-off-by: Sai

Re: [PATCH v3 4/8] selftests/resctrl: Add callback to start a benchmark

2018-11-12 Thread Moger, Babu
On 10/31/18 4:02 PM, Fenghua Yu wrote: > From: Sai Praneeth Prakhya > > The callback starts a child process and puts the child pid in created > resctrl group with specified memory bandwidth in schemata. The child > starts running benchmark. > > Signed-off-by: Sai Praneeth Prakhya >

Re: [PATCH v3 4/8] selftests/resctrl: Add callback to start a benchmark

2018-11-12 Thread Moger, Babu
On 10/31/18 4:02 PM, Fenghua Yu wrote: > From: Sai Praneeth Prakhya > > The callback starts a child process and puts the child pid in created > resctrl group with specified memory bandwidth in schemata. The child > starts running benchmark. > > Signed-off-by: Sai Praneeth Prakhya >

Re: [PATCH v7 02/13] arch/x86: Rename the RDT functions and definitions

2018-11-12 Thread Moger, Babu
Boris, On 11/12/18 11:56 AM, Borislav Petkov wrote: > On Fri, Nov 09, 2018 at 08:52:27PM +0000, Moger, Babu wrote: >> As AMD is starting to support RDT(or QOS) features, rename >> the RDT functions and definitions to more generic names. >> >> Replace intel_rdt

Re: [PATCH v7 02/13] arch/x86: Rename the RDT functions and definitions

2018-11-12 Thread Moger, Babu
Boris, On 11/12/18 11:56 AM, Borislav Petkov wrote: > On Fri, Nov 09, 2018 at 08:52:27PM +0000, Moger, Babu wrote: >> As AMD is starting to support RDT(or QOS) features, rename >> the RDT functions and definitions to more generic names. >> >> Replace intel_rdt

[PATCH v7 06/13] arch/x86: Initialize the resource functions that are different

2018-11-09 Thread Moger, Babu
Initialize the resource functions that are different between the vendors. Some features are initialized differently between the vendors. Add _intel suffix to Intel specific functions. For example, MBA feature varies significantly between Intel and AMD. Separate the initialization of these

[PATCH v7 07/13] arch/x86: Bring cbm_validate function into the resource structure

2018-11-09 Thread Moger, Babu
Idea is to bring all the functions that are different between the vendors into resource structure and initialize them dynamically. Add _intel suffix to Intel specific functions. Following function is implemented separately for each vendors. cbm_validate : Cache bitmask validate function. AMD

[PATCH v7 06/13] arch/x86: Initialize the resource functions that are different

2018-11-09 Thread Moger, Babu
Initialize the resource functions that are different between the vendors. Some features are initialized differently between the vendors. Add _intel suffix to Intel specific functions. For example, MBA feature varies significantly between Intel and AMD. Separate the initialization of these

[PATCH v7 07/13] arch/x86: Bring cbm_validate function into the resource structure

2018-11-09 Thread Moger, Babu
Idea is to bring all the functions that are different between the vendors into resource structure and initialize them dynamically. Add _intel suffix to Intel specific functions. Following function is implemented separately for each vendors. cbm_validate : Cache bitmask validate function. AMD

[PATCH v6 14/14] MAINTAINERS: Update the file and documentation names in arch/x86

2018-11-06 Thread Moger, Babu
Update the MAINTAINERS to reflect the changed file(and documentation) names in arch/x86/kernel/cpu. The file names have changed from intel_rdt* to resctrl*. Signed-off-by: Babu Moger --- MAINTAINERS | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/MAINTAINERS

[PATCH v6 14/14] MAINTAINERS: Update the file and documentation names in arch/x86

2018-11-06 Thread Moger, Babu
Update the MAINTAINERS to reflect the changed file(and documentation) names in arch/x86/kernel/cpu. The file names have changed from intel_rdt* to resctrl*. Signed-off-by: Babu Moger --- MAINTAINERS | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/MAINTAINERS

[PATCH v6 12/14] arch/x86: Introduce QOS feature for AMD

2018-11-06 Thread Moger, Babu
Enables QOS feature on AMD. Following QoS sub-features are supported in AMD if the underlying hardware supports it. - L3 Cache allocation enforcement - L3 Cache occupancy monitoring - L3 Code-Data Prioritization support - Memory Bandwidth Enforcement(Allocation) The specification for this

[PATCH v6 12/14] arch/x86: Introduce QOS feature for AMD

2018-11-06 Thread Moger, Babu
Enables QOS feature on AMD. Following QoS sub-features are supported in AMD if the underlying hardware supports it. - L3 Cache allocation enforcement - L3 Cache occupancy monitoring - L3 Code-Data Prioritization support - Memory Bandwidth Enforcement(Allocation) The specification for this

RE: [PATCH v2 00/11] arch/x86: AMD QoS support

2018-11-02 Thread Moger, Babu
Jon, > -Original Message- > From: linux-kernel-ow...@vger.kernel.org ow...@vger.kernel.org> On Behalf Of Jon Masters > Sent: Friday, November 2, 2018 1:43 AM > To: Moger, Babu ; t...@linutronix.de; > mi...@redhat.com; h...@zytor.com; reinette.cha...@intel.com; >

RE: [PATCH v2 00/11] arch/x86: AMD QoS support

2018-11-02 Thread Moger, Babu
Jon, > -Original Message- > From: linux-kernel-ow...@vger.kernel.org ow...@vger.kernel.org> On Behalf Of Jon Masters > Sent: Friday, November 2, 2018 1:43 AM > To: Moger, Babu ; t...@linutronix.de; > mi...@redhat.com; h...@zytor.com; reinette.cha...@intel.com; >

RE: Fwd: [PATCH v2 2/8] selftests/resctrl: Add basic resctrl file system operations and data

2018-10-30 Thread Moger, Babu
Fenghua, > -Original Message- > From: Fenghua Yu > Sent: Tuesday, October 30, 2018 1:36 PM > To: Moger, Babu ; Fenghua Yu > > Cc: Thomas Gleixner ; Ingo Molnar > ; H Peter Anvin ; Tony Luck > ; Peter Zijlstra ; Reinette > Chatre ; James Morse > ; Ravi V Sh

RE: Fwd: [PATCH v2 2/8] selftests/resctrl: Add basic resctrl file system operations and data

2018-10-30 Thread Moger, Babu
Fenghua, > -Original Message- > From: Fenghua Yu > Sent: Tuesday, October 30, 2018 1:36 PM > To: Moger, Babu ; Fenghua Yu > > Cc: Thomas Gleixner ; Ingo Molnar > ; H Peter Anvin ; Tony Luck > ; Peter Zijlstra ; Reinette > Chatre ; James Morse > ; Ravi V Sh

RE: [PATCH v2 0/8] selftests/resctrl: Add resctrl selftest

2018-10-29 Thread Moger, Babu
Hi Fenghua, > -Original Message- > From: linux-kernel-ow...@vger.kernel.org ow...@vger.kernel.org> On Behalf Of Fenghua Yu > Sent: Thursday, October 25, 2018 6:07 PM > To: Thomas Gleixner ; Ingo Molnar > ; H Peter Anvin ; Tony Luck > ; Peter Zijlstra ; Reinette &

RE: [PATCH v2 0/8] selftests/resctrl: Add resctrl selftest

2018-10-29 Thread Moger, Babu
Hi Fenghua, > -Original Message- > From: linux-kernel-ow...@vger.kernel.org ow...@vger.kernel.org> On Behalf Of Fenghua Yu > Sent: Thursday, October 25, 2018 6:07 PM > To: Thomas Gleixner ; Ingo Molnar > ; H Peter Anvin ; Tony Luck > ; Peter Zijlstra ; Reinette &

RE: [PATCH v2 3/8] selftests/resctrl: Read memory bandwidth from perf IMC counter and from resctrl file system

2018-10-29 Thread Moger, Babu
Fenghua/Sai, > -Original Message- > From: Fenghua Yu > Sent: Thursday, October 25, 2018 6:07 PM > To: Thomas Gleixner ; Ingo Molnar > ; H Peter Anvin ; Tony Luck > ; Peter Zijlstra ; Reinette > Chatre ; Moger, Babu > ; James Morse ; Ravi V > Shankar ; Sai P

RE: [PATCH v2 3/8] selftests/resctrl: Read memory bandwidth from perf IMC counter and from resctrl file system

2018-10-29 Thread Moger, Babu
Fenghua/Sai, > -Original Message- > From: Fenghua Yu > Sent: Thursday, October 25, 2018 6:07 PM > To: Thomas Gleixner ; Ingo Molnar > ; H Peter Anvin ; Tony Luck > ; Peter Zijlstra ; Reinette > Chatre ; Moger, Babu > ; James Morse ; Ravi V > Shankar ; Sai P

RE: [PATCH v2 4/8] selftests/resctrl: Add callback to start a benchmark

2018-10-29 Thread Moger, Babu
Hi Fenghua/Sai, > -Original Message- > From: Fenghua Yu > Sent: Thursday, October 25, 2018 6:07 PM > To: Thomas Gleixner ; Ingo Molnar > ; H Peter Anvin ; Tony Luck > ; Peter Zijlstra ; Reinette > Chatre ; Moger, Babu > ; James Morse ; Ravi V > Shankar ; Sai P

RE: [PATCH v2 4/8] selftests/resctrl: Add callback to start a benchmark

2018-10-29 Thread Moger, Babu
Hi Fenghua/Sai, > -Original Message- > From: Fenghua Yu > Sent: Thursday, October 25, 2018 6:07 PM > To: Thomas Gleixner ; Ingo Molnar > ; H Peter Anvin ; Tony Luck > ; Peter Zijlstra ; Reinette > Chatre ; Moger, Babu > ; James Morse ; Ravi V > Shankar ; Sai P

RE: [PATCH v2 3/8] selftests/resctrl: Read memory bandwidth from perf IMC counter and from resctrl file system

2018-10-29 Thread Moger, Babu
Fenghua, Sai > -Original Message- > From: Fenghua Yu > Sent: Thursday, October 25, 2018 6:07 PM > To: Thomas Gleixner ; Ingo Molnar > ; H Peter Anvin ; Tony Luck > ; Peter Zijlstra ; Reinette > Chatre ; Moger, Babu > ; James Morse ; Ravi V > Shankar ; Sai P

RE: [PATCH v2 3/8] selftests/resctrl: Read memory bandwidth from perf IMC counter and from resctrl file system

2018-10-29 Thread Moger, Babu
Fenghua, Sai > -Original Message- > From: Fenghua Yu > Sent: Thursday, October 25, 2018 6:07 PM > To: Thomas Gleixner ; Ingo Molnar > ; H Peter Anvin ; Tony Luck > ; Peter Zijlstra ; Reinette > Chatre ; Moger, Babu > ; James Morse ; Ravi V > Shankar ; Sai P

RE: [PATCH v2 2/8] selftests/resctrl: Add basic resctrl file system operations and data

2018-10-29 Thread Moger, Babu
Hi Fenghua, Sai, > -Original Message- > From: Fenghua Yu > Sent: Thursday, October 25, 2018 6:07 PM > To: Thomas Gleixner ; Ingo Molnar > ; H Peter Anvin ; Tony Luck > ; Peter Zijlstra ; Reinette > Chatre ; Moger, Babu > ; James Morse ; Ravi V > Shankar ; Sai P

RE: [PATCH v2 2/8] selftests/resctrl: Add basic resctrl file system operations and data

2018-10-29 Thread Moger, Babu
Hi Fenghua, Sai, > -Original Message- > From: Fenghua Yu > Sent: Thursday, October 25, 2018 6:07 PM > To: Thomas Gleixner ; Ingo Molnar > ; H Peter Anvin ; Tony Luck > ; Peter Zijlstra ; Reinette > Chatre ; Moger, Babu > ; James Morse ; Ravi V > Shankar ; Sai P

Re: [PATCH 0/7] selftests/resctrl: Add resctrl selftest

2018-10-17 Thread Moger, Babu
On 10/17/2018 12:28 PM, Prakhya, Sai Praneeth wrote: >>> No, the selftest in this patch set will not replace intel-cmt-cat or >>> vice versa. >>> >>> The selftest in this patch set has a different purpose from intel-cmt-cat: >>> the selftest is a test tool which validates resctrl functionalities

Re: [PATCH 0/7] selftests/resctrl: Add resctrl selftest

2018-10-17 Thread Moger, Babu
On 10/17/2018 12:28 PM, Prakhya, Sai Praneeth wrote: >>> No, the selftest in this patch set will not replace intel-cmt-cat or >>> vice versa. >>> >>> The selftest in this patch set has a different purpose from intel-cmt-cat: >>> the selftest is a test tool which validates resctrl functionalities

Re: [PATCH 0/7] selftests/resctrl: Add resctrl selftest

2018-10-17 Thread Moger, Babu
On 10/17/2018 01:03 PM, Moger, Babu wrote: > Hi Fenghua, > My few comments. > > On 10/17/2018 09:40 AM, Moger, Babu wrote: >> >> >> On 10/16/2018 03:32 PM, Fenghua Yu wrote: >>>> From: Moger, Babu [mailto:babu.mo...@amd.com] >>>> On

Re: [PATCH 0/7] selftests/resctrl: Add resctrl selftest

2018-10-17 Thread Moger, Babu
On 10/17/2018 01:03 PM, Moger, Babu wrote: > Hi Fenghua, > My few comments. > > On 10/17/2018 09:40 AM, Moger, Babu wrote: >> >> >> On 10/16/2018 03:32 PM, Fenghua Yu wrote: >>>> From: Moger, Babu [mailto:babu.mo...@amd.com] >>>> On

Re: [PATCH 0/7] selftests/resctrl: Add resctrl selftest

2018-10-17 Thread Moger, Babu
Hi Fenghua, My few comments. On 10/17/2018 09:40 AM, Moger, Babu wrote: > > > On 10/16/2018 03:32 PM, Fenghua Yu wrote: >>> From: Moger, Babu [mailto:babu.mo...@amd.com] >>> On 10/16/2018 11:56 AM, Fenghua Yu wrote: >>>> With more and more resctrl

Re: [PATCH 0/7] selftests/resctrl: Add resctrl selftest

2018-10-17 Thread Moger, Babu
Hi Fenghua, My few comments. On 10/17/2018 09:40 AM, Moger, Babu wrote: > > > On 10/16/2018 03:32 PM, Fenghua Yu wrote: >>> From: Moger, Babu [mailto:babu.mo...@amd.com] >>> On 10/16/2018 11:56 AM, Fenghua Yu wrote: >>>> With more and more resctrl

Re: [PATCH v4 08/13] arch/x86: Bring few more functions into the resource structure

2018-10-17 Thread Moger, Babu
Hi Fenghua, On 10/16/2018 06:45 PM, Fenghua Yu wrote: > On Mon, Oct 15, 2018 at 08:55:49PM +0000, Moger, Babu wrote: >> update_mba_bw : Feedback loop bandwidth update functionality is not >> needed for AMD. > > Will you implement update_mba_b

Re: [PATCH v4 08/13] arch/x86: Bring few more functions into the resource structure

2018-10-17 Thread Moger, Babu
Hi Fenghua, On 10/16/2018 06:45 PM, Fenghua Yu wrote: > On Mon, Oct 15, 2018 at 08:55:49PM +0000, Moger, Babu wrote: >> update_mba_bw : Feedback loop bandwidth update functionality is not >> needed for AMD. > > Will you implement update_mba_b

Re: [PATCH v4 10/13] arch/x86: Add AMD feature bit X86_FEATURE_MBA in cpuid bits array

2018-10-17 Thread Moger, Babu
On 10/16/2018 06:48 PM, Fenghua Yu wrote: > On Mon, Oct 15, 2018 at 08:55:54PM +0000, Moger, Babu wrote: >> From: Sherry Hurwitz >> >> The feature bit X86_FEATURE_MBA is detected via CPUID leaf 0x8008 >> EBX Bit 06. This bit indicates the support of AMD's MBA

Re: [PATCH v4 10/13] arch/x86: Add AMD feature bit X86_FEATURE_MBA in cpuid bits array

2018-10-17 Thread Moger, Babu
On 10/16/2018 06:48 PM, Fenghua Yu wrote: > On Mon, Oct 15, 2018 at 08:55:54PM +0000, Moger, Babu wrote: >> From: Sherry Hurwitz >> >> The feature bit X86_FEATURE_MBA is detected via CPUID leaf 0x8008 >> EBX Bit 06. This bit indicates the support of AMD's MBA

Re: [PATCH 0/7] selftests/resctrl: Add resctrl selftest

2018-10-17 Thread Moger, Babu
On 10/16/2018 03:32 PM, Fenghua Yu wrote: >> From: Moger, Babu [mailto:babu.mo...@amd.com] >> On 10/16/2018 11:56 AM, Fenghua Yu wrote: >>> With more and more resctrl features are being added by Intel, AMD and >>> ARM, a test tool is becoming more and more use

Re: [PATCH 0/7] selftests/resctrl: Add resctrl selftest

2018-10-17 Thread Moger, Babu
On 10/16/2018 03:32 PM, Fenghua Yu wrote: >> From: Moger, Babu [mailto:babu.mo...@amd.com] >> On 10/16/2018 11:56 AM, Fenghua Yu wrote: >>> With more and more resctrl features are being added by Intel, AMD and >>> ARM, a test tool is becoming more and more use

Re: [PATCH 0/7] selftests/resctrl: Add resctrl selftest

2018-10-16 Thread Moger, Babu
Hi Fenghua, On 10/16/2018 11:56 AM, Fenghua Yu wrote: > With more and more resctrl features are being added by Intel, AMD > and ARM, a test tool is becoming more and more useful to validate > that both hardware and software functionalities work as expected. I like the initiative here. It is

Re: [PATCH 0/7] selftests/resctrl: Add resctrl selftest

2018-10-16 Thread Moger, Babu
Hi Fenghua, On 10/16/2018 11:56 AM, Fenghua Yu wrote: > With more and more resctrl features are being added by Intel, AMD > and ARM, a test tool is becoming more and more useful to validate > that both hardware and software functionalities work as expected. I like the initiative here. It is

[PATCH v4 00/13] arch/x86: AMD QoS support

2018-10-15 Thread Moger, Babu
This series adds support for AMD64 architectural extensions for Platform Quality of Service. These extensions are intended to provide for the monitoring of the usage of certain system resources by one or more processors and for the separate allocation and enforcement of limits on the use of

[PATCH v4 00/13] arch/x86: AMD QoS support

2018-10-15 Thread Moger, Babu
This series adds support for AMD64 architectural extensions for Platform Quality of Service. These extensions are intended to provide for the monitoring of the usage of certain system resources by one or more processors and for the separate allocation and enforcement of limits on the use of

[PATCH v4 11/13] arch/x86: Introduce QOS feature for AMD

2018-10-15 Thread Moger, Babu
Enables QOS feature on AMD. Following QoS sub-features are supported in AMD if the underlying hardware supports it. - L3 Cache allocation enforcement - L3 Cache occupancy monitoring - L3 Code-Data Prioritization support - Memory Bandwidth Enforcement(Allocation) The specification for this

[PATCH v4 11/13] arch/x86: Introduce QOS feature for AMD

2018-10-15 Thread Moger, Babu
Enables QOS feature on AMD. Following QoS sub-features are supported in AMD if the underlying hardware supports it. - L3 Cache allocation enforcement - L3 Cache occupancy monitoring - L3 Code-Data Prioritization support - Memory Bandwidth Enforcement(Allocation) The specification for this

[PATCH v4 13/13] MAINTAINERS: Update the file and documentation names in arch/x86

2018-10-15 Thread Moger, Babu
Update the MAINTAINERS to reflect the changed file(and documentation) names in arch/x86/kernel/cpu. The file names have changed from intel_rdt* to resctrl*. Signed-off-by: Babu Moger --- MAINTAINERS | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/MAINTAINERS

[PATCH v4 03/13] arch/x86: Re-arrange RDT init code

2018-10-15 Thread Moger, Babu
Separate the call sequence for rdt_quirks and MBA feature. This is in preparation to handle vendor differences in these call sequences. Signed-off-by: Babu Moger --- arch/x86/kernel/cpu/resctrl.c | 29 +++-- 1 file changed, 23 insertions(+), 6 deletions(-) diff --git

[PATCH v4 10/13] arch/x86: Add AMD feature bit X86_FEATURE_MBA in cpuid bits array

2018-10-15 Thread Moger, Babu
From: Sherry Hurwitz The feature bit X86_FEATURE_MBA is detected via CPUID leaf 0x8008 EBX Bit 06. This bit indicates the support of AMD's MBA feature. This feature is supported by both Intel and AMD. But they are detected different CPUID leaves. Signed-off-by: Sherry Hurwitz

[PATCH v4 07/13] arch/x86: Initialize the resource functions that are different

2018-10-15 Thread Moger, Babu
Initialize the resource functions that are different between the vendors. Some features are initialized differently between the vendors. Add _intel suffix to Intel specific functions. For example, MBA feature varies significantly between Intel and AMD. Separate the initialization of these

[PATCH v4 13/13] MAINTAINERS: Update the file and documentation names in arch/x86

2018-10-15 Thread Moger, Babu
Update the MAINTAINERS to reflect the changed file(and documentation) names in arch/x86/kernel/cpu. The file names have changed from intel_rdt* to resctrl*. Signed-off-by: Babu Moger --- MAINTAINERS | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/MAINTAINERS

[PATCH v4 03/13] arch/x86: Re-arrange RDT init code

2018-10-15 Thread Moger, Babu
Separate the call sequence for rdt_quirks and MBA feature. This is in preparation to handle vendor differences in these call sequences. Signed-off-by: Babu Moger --- arch/x86/kernel/cpu/resctrl.c | 29 +++-- 1 file changed, 23 insertions(+), 6 deletions(-) diff --git

[PATCH v4 10/13] arch/x86: Add AMD feature bit X86_FEATURE_MBA in cpuid bits array

2018-10-15 Thread Moger, Babu
From: Sherry Hurwitz The feature bit X86_FEATURE_MBA is detected via CPUID leaf 0x8008 EBX Bit 06. This bit indicates the support of AMD's MBA feature. This feature is supported by both Intel and AMD. But they are detected different CPUID leaves. Signed-off-by: Sherry Hurwitz

[PATCH v4 07/13] arch/x86: Initialize the resource functions that are different

2018-10-15 Thread Moger, Babu
Initialize the resource functions that are different between the vendors. Some features are initialized differently between the vendors. Add _intel suffix to Intel specific functions. For example, MBA feature varies significantly between Intel and AMD. Separate the initialization of these

[PATCH v4 09/13] arch/x86: Introduce new config parameter AMD_QOS

2018-10-15 Thread Moger, Babu
Introduces the new config parameter AMD_QOS. This parameter will be used to enable cache and memory bandwidth allocation and monitoring features on AMD processors. This will enable common config parameter RESCTRL if selected. Signed-off-by: Babu Moger --- arch/x86/Kconfig | 17 -

[PATCH v4 09/13] arch/x86: Introduce new config parameter AMD_QOS

2018-10-15 Thread Moger, Babu
Introduces the new config parameter AMD_QOS. This parameter will be used to enable cache and memory bandwidth allocation and monitoring features on AMD processors. This will enable common config parameter RESCTRL if selected. Signed-off-by: Babu Moger --- arch/x86/Kconfig | 17 -

[PATCH v4 04/13] arch/x86: Bring all the macros to resctrl.h

2018-10-15 Thread Moger, Babu
Bring all the macros to resctrl.h and rename for consistency. Signed-off-by: Babu Moger --- arch/x86/kernel/cpu/resctrl.c | 3 --- arch/x86/kernel/cpu/resctrl.h | 5 + arch/x86/kernel/cpu/resctrl_monitor.c | 7 ++- 3 files changed, 7 insertions(+), 8 deletions(-) diff

[PATCH v4 12/13] Documentation/x86: Rename and update intel_rdt_ui.txt

2018-10-15 Thread Moger, Babu
Rename intel_rdt_ui.txt to generic resctrl_ui.txt and update the documentation for AMD. Signed-off-by: Babu Moger --- Documentation/x86/{intel_rdt_ui.txt => resctrl_ui.txt} | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) rename Documentation/x86/{intel_rdt_ui.txt =>

[PATCH v4 04/13] arch/x86: Bring all the macros to resctrl.h

2018-10-15 Thread Moger, Babu
Bring all the macros to resctrl.h and rename for consistency. Signed-off-by: Babu Moger --- arch/x86/kernel/cpu/resctrl.c | 3 --- arch/x86/kernel/cpu/resctrl.h | 5 + arch/x86/kernel/cpu/resctrl_monitor.c | 7 ++- 3 files changed, 7 insertions(+), 8 deletions(-) diff

[PATCH v4 12/13] Documentation/x86: Rename and update intel_rdt_ui.txt

2018-10-15 Thread Moger, Babu
Rename intel_rdt_ui.txt to generic resctrl_ui.txt and update the documentation for AMD. Signed-off-by: Babu Moger --- Documentation/x86/{intel_rdt_ui.txt => resctrl_ui.txt} | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) rename Documentation/x86/{intel_rdt_ui.txt =>

[PATCH v4 06/13] arch/x86: Use new config parameter RESCTRL for compilation

2018-10-15 Thread Moger, Babu
Use newly added config parameter RESCTRL to compile sources. This is common parameter across both Intel and AMD. Signed-off-by: Babu Moger --- arch/x86/include/asm/resctrl_sched.h | 4 ++-- arch/x86/kernel/cpu/Makefile | 4 ++-- include/linux/sched.h| 2 +- 3 files

[PATCH v4 08/13] arch/x86: Bring few more functions into the resource structure

2018-10-15 Thread Moger, Babu
Bring all resource functions that are different between the vendors into resource structure and initialize them dynamically. Add _intel suffix to Intel specific functions. Implement these functions separately for each vendors. update_mba_bw : Feedback loop bandwidth update functionality is not

[PATCH v4 01/13] arch/x86: Start renaming the rdt files to more generic names

2018-10-15 Thread Moger, Babu
New generation of AMD processors start supporting RDT(or QOS) features. With more than one vendors supporting these features, it seems more appropriate to rename these files. Changed intel_rdt to resctrl where applicable. Signed-off-by: Babu Moger --- arch/x86/include/asm/{intel_rdt_sched.h =>

[PATCH v4 02/13] arch/x86: Rename the RDT functions and definitions

2018-10-15 Thread Moger, Babu
As AMD is starting to support RDT(or QOS) features, rename the RDT functions and definitions to more generic names. Replace intel_rdt to resctrl where applicable. Signed-off-by: Babu Moger --- arch/x86/include/asm/resctrl_sched.h | 24 arch/x86/kernel/cpu/resctrl.c

[PATCH v4 06/13] arch/x86: Use new config parameter RESCTRL for compilation

2018-10-15 Thread Moger, Babu
Use newly added config parameter RESCTRL to compile sources. This is common parameter across both Intel and AMD. Signed-off-by: Babu Moger --- arch/x86/include/asm/resctrl_sched.h | 4 ++-- arch/x86/kernel/cpu/Makefile | 4 ++-- include/linux/sched.h| 2 +- 3 files

[PATCH v4 08/13] arch/x86: Bring few more functions into the resource structure

2018-10-15 Thread Moger, Babu
Bring all resource functions that are different between the vendors into resource structure and initialize them dynamically. Add _intel suffix to Intel specific functions. Implement these functions separately for each vendors. update_mba_bw : Feedback loop bandwidth update functionality is not

  1   2   3   >