RE: [RFC 3/3] arch/x86/kvm: SVM: Introduce pause loop exit logic in SVM

2018-03-15 Thread Moger, Babu
> -Original Message- > From: Radim Krčmář <rkrc...@redhat.com> > Sent: Wednesday, March 14, 2018 8:26 AM > To: Moger, Babu <babu.mo...@amd.com> > Cc: j...@8bytes.org; t...@linutronix.de; mi...@redhat.com; > h...@zytor.com; x...@kernel.org; pbonz...@redh

RE: [RFC 3/3] arch/x86/kvm: SVM: Introduce pause loop exit logic in SVM

2018-03-09 Thread Moger, Babu
Radim, Thanks for the comments. Taken care of most of the comments. I have few questions/comments. Please see inline. > -Original Message- > From: Radim Krčmář <rkrc...@redhat.com> > Sent: Friday, March 9, 2018 12:13 PM > To: Moger, Babu <babu.mo...@amd.com>

RE: [PATCH v2 3/5] KVM: VMX: Bring the common code to header file

2018-03-28 Thread Moger, Babu
> -Original Message- > From: Radim Krčmář <rkrc...@redhat.com> > Sent: Wednesday, March 28, 2018 3:27 PM > To: Moger, Babu <babu.mo...@amd.com> > Cc: j...@8bytes.org; t...@linutronix.de; mi...@redhat.com; > h...@zytor.com; x...@kernel.org; pbonz...@redh

RE: [PATCH v2 5/5] KVM: SVM: Implement pause loop exit logic in SVM

2018-03-28 Thread Moger, Babu
> -Original Message- > From: Radim Krčmář <rkrc...@redhat.com> > Sent: Wednesday, March 28, 2018 3:31 PM > To: Moger, Babu <babu.mo...@amd.com> > Cc: j...@8bytes.org; t...@linutronix.de; mi...@redhat.com; > h...@zytor.com; x...@kernel.org; pbonz...@redh

Re: [RFC PATCH 00/10] arch/x86: AMD QoS support

2018-10-05 Thread Moger, Babu
Hi James, On 10/05/2018 11:18 AM, James Morse wrote: > Hi Babu, > > (Thanks for looping me in!) > > On 28/09/18 02:57, Moger, Babu wrote: >>> On Mon, 24 Sep 2018, Moger, Babu wrote: >>> >>>> This series adds support for AMD64 architectural ex

Re: [RFC PATCH 10/10] arch/x86: Introduce QOS feature for AMD

2018-10-05 Thread Moger, Babu
Hi James, On 10/05/2018 11:20 AM, James Morse wrote: > Hi Babu, > > On 24/09/18 20:19, Moger, Babu wrote: >> Enables QOS feature on AMD. >> Following QoS sub-features are supported in AMD if the underlying >> hardware supports it. >> - L3 Cache allocation enf

RE: [PATCH v2 10/11] arch/x86: Add AMD feature bit X86_FEATURE_MBA in cpuid bits array

2018-10-07 Thread Moger, Babu
> -Original Message- > From: Borislav Petkov > Sent: Friday, October 5, 2018 4:31 PM > To: Moger, Babu > Cc: t...@linutronix.de; mi...@redhat.com; h...@zytor.com; > reinette.cha...@intel.com; fenghua...@intel.com; james.mo...@arm.com; > vikas.shiva...@li

RE: [PATCH v2 05/11] arch/x86: Introduce a new config parameter PLATFORM_QOS

2018-10-07 Thread Moger, Babu
Hi Fenghua, > -Original Message- > From: Fenghua Yu > Sent: Friday, October 5, 2018 6:39 PM > To: Moger, Babu > Cc: t...@linutronix.de; mi...@redhat.com; h...@zytor.com; > reinette.cha...@intel.com; fenghua...@intel.com; james.mo...@arm.com; > vikas.shiva...@li

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

2018-10-05 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 v2 04/11] arch/x86: Bring all the macros to rdt.h

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

[PATCH v2 06/11] arch/x86: Use new config parameter PLATFORM_QOS for compilation

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

[PATCH v2 05/11] arch/x86: Introduce a new config parameter PLATFORM_QOS

2018-10-05 Thread Moger, Babu
Introduces a new config parameter PLATFORM_QOS. This will be used as a common config parameter for both Intel and AMD. Each vendor will have their own config parameter to enable RDT feature. One for Intel(INTEL_RDT) and one for AMD(AMD_QOS). It can be enabled or disabled separately. The new

[PATCH v2 07/11] arch/x86: Initialize the resource functions that are different

2018-10-05 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 v2 10/11] arch/x86: Add AMD feature bit X86_FEATURE_MBA in cpuid bits array

2018-10-05 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: Babu Moger Signed-off-by:

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

2018-10-05 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) There are differences in the

[PATCH v2 03/11] arch/x86: Re-arrange RDT init code

2018-10-05 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/rdt.c | 29 +++-- 1 file changed, 23 insertions(+), 6 deletions(-) diff --git

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

2018-10-05 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 v2 02/11] arch/x86: Rename the RDT functions and definitions

2018-10-05 Thread Moger, Babu
As AMD is starting to support RDT(or QOS) features, rename the RDT functions and definitions to more generic names. Signed-off-by: Babu Moger --- arch/x86/include/asm/rdt_sched.h | 22 +++--- arch/x86/kernel/cpu/rdt.c | 24

[PATCH v2 09/11] arch/x86: Introduce new config parameter AMD_QOS

2018-10-05 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 PLATFORM_QOS if selected. Signed-off-by: Babu Moger --- arch/x86/Kconfig | 17

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

2018-10-05 Thread Moger, Babu
New generation of AMD processors start support RDT(or QOS) features. With more than one vendors supporting these features, it seems more appropriate to rename these files. Signed-off-by: Babu Moger --- arch/x86/include/asm/{intel_rdt_sched.h => rdt_sched.h} | 0 arch/x86/kernel/cpu/Makefile

[PATCH v3 02/11] arch/x86: Rename the RDT functions and definitions

2018-10-11 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 v3 00/11] arch/x86: AMD QoS support

2018-10-11 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 v3 07/11] arch/x86: Initialize the resource functions that are different

2018-10-11 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 v3 08/11] arch/x86: Bring few more functions into the resource structure

2018-10-11 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 v3 05/11] arch/x86: Introduce a new config parameter RESCTRL

2018-10-11 Thread Moger, Babu
Introduces a new config parameter RESCTRL. This will be used as a common config parameter for both Intel and AMD. Each vendor will have their own config parameter to enable RDT feature. One for Intel(INTEL_RDT) and one for AMD(AMD_QOS). It can be enabled or disabled separately. The new parameter

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

2018-10-11 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 v3 11/11] arch/x86: Introduce QOS feature for AMD

2018-10-11 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 v3 06/11] arch/x86: Use new config parameter RESCTRL for compilation

2018-10-11 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 v3 04/11] arch/x86: Bring all the macros to resctrl.h

2018-10-11 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 v3 09/11] arch/x86: Introduce new config parameter AMD_QOS

2018-10-11 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 v3 01/11] arch/x86: Start renaming the rdt files to more generic names

2018-10-11 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 v3 03/11] arch/x86: Re-arrange RDT init code

2018-10-11 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

Re: [PATCH v3 09/11] arch/x86: Introduce new config parameter AMD_QOS

2018-10-12 Thread Moger, Babu
Boris, On 10/11/2018 05:02 PM, Borislav Petkov wrote: > On Thu, Oct 11, 2018 at 08:33:35PM +0000, Moger, Babu wrote: >> Introduces the new config parameter AMD_QOS. This parameter will be >> used to enable cache and memory bandwidth allocation and monitoring >> featu

Re: [PATCH v3 01/11] arch/x86: Start renaming the rdt files to more generic names

2018-10-12 Thread Moger, Babu
Hi Reinette, On 10/12/2018 02:07 PM, Reinette Chatre wrote: > Hi Babu, > > On 10/11/2018 1:33 PM, Moger, Babu wrote: >> New generation of AMD processors start supporting RDT(or QOS) features. >> With more than one vendors supporting these features, it seems more >

Re: [PATCH v3 09/11] arch/x86: Introduce new config parameter AMD_QOS

2018-10-12 Thread Moger, Babu
On 10/12/2018 09:43 AM, Borislav Petkov wrote: > On Fri, Oct 12, 2018 at 02:40:50PM +0000, Moger, Babu wrote: >> That is correct. CPU_SUP_AMD implicitly means x86. >> To be more specific, I will change it to >> "depends on X86_64 && CPU_SUP_AMD" as this fe

Re: [PATCH v3 02/11] arch/x86: Rename the RDT functions and definitions

2018-10-12 Thread Moger, Babu
On 10/12/2018 02:40 PM, Reinette Chatre wrote: > Hi Babu, > > On 10/11/2018 1:33 PM, Moger, Babu wrote: >> @@ -883,20 +883,20 @@ static int __init intel_rdt_late_init(void) >> rdt_online = state; >> >> for_each_alloc_capable_rdt_resource(r) >

[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 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 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 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 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 05/13] arch/x86: Introduce a new config parameter RESCTRL

2018-10-15 Thread Moger, Babu
Introduces a new config parameter RESCTRL. This will be used as a common config parameter for both Intel and AMD. Each vendor will have their own config parameter to enable RDT feature. One for Intel(INTEL_RDT) and one for AMD(AMD_QOS). It can be enabled or disabled separately. The new parameter

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
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 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-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-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 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 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 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 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 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: 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 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; >

[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

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 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 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

[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 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

Re: [RFC PATCH 10/10] arch/x86: Introduce QOS feature for AMD

2018-10-03 Thread Moger, Babu
On 10/02/2018 01:27 PM, Fenghua Yu wrote: > On Mon, Sep 24, 2018 at 07:19:16PM +0000, Moger, Babu wrote: >> int parse_bw(void *_buf, struct rdt_resource *r, struct rdt_domain *d); >> +int parse_bw_amd(void *_buf, struct rdt_resource *r, struct rdt_domain *d); > > Please

Re: [RFC PATCH 07/10] arch/x86: Bring few more functions into the resource structure

2018-10-03 Thread Moger, Babu
On 10/02/2018 05:07 PM, Reinette Chatre wrote: > Hi Babu, > > On 9/24/2018 12:19 PM, Moger, Babu wrote: >> Bring all resource functions that are different between the vendors >> into resource structure and initialize them dynamically. >> >> Implement these func

Re: [RFC PATCH 06/10] arch/x86: Initialize the resource functions that are different

2018-10-03 Thread Moger, Babu
On 10/02/2018 05:06 PM, Reinette Chatre wrote: > Hi Babu, > > On 9/24/2018 12:19 PM, Moger, Babu wrote: >> Initialize the resource functions that are different between the >> vendors. Some features are initialized differently between the vendors. >> >>

Re: [RFC PATCH 10/10] arch/x86: Introduce QOS feature for AMD

2018-10-03 Thread Moger, Babu
Hi Reinette, On 10/02/2018 05:13 PM, Reinette Chatre wrote: > Hi Babu, > > On 9/24/2018 12:19 PM, Moger, Babu wrote: >> +/* >> + * Check whether a cache bit mask is valid. AMD allows >> + * non-contiguous masks. >> + */ >> +bool cbm_validate_amd(char

Re: [RFC PATCH 03/10] arch/x86: Re-arrange RDT init code

2018-10-03 Thread Moger, Babu
On 10/03/2018 01:54 PM, Reinette Chatre wrote: > Hi Babu, > > On 10/2/2018 4:41 PM, Moger, Babu wrote: >> On 10/02/2018 02:21 PM, Reinette Chatre wrote: >>> On 9/24/2018 12:19 PM, Moger, Babu wrote: >>>> static enum cpuhp_state rdt_online; >>

Re: [RFC PATCH 00/10] arch/x86: AMD QoS support

2018-10-03 Thread Moger, Babu
On 10/03/2018 01:52 PM, Fenghua Yu wrote: > On Tue, Oct 02, 2018 at 07:16:23PM +0000, Moger, Babu wrote: >> >> >> On 10/02/2018 01:46 PM, Fenghua Yu wrote: >>> On Tue, Oct 02, 2018 at 05:44:47PM +, Moger, Babu wrote: >>>> Hi Fenghua, >>>>

RE: [RFC PATCH 00/10] arch/x86: AMD QoS support

2018-10-02 Thread Moger, Babu
Hi Fenghua, > -Original Message- > From: Fenghua Yu > Sent: Tuesday, October 2, 2018 12:07 PM > To: Moger, Babu > Cc: t...@linutronix.de; mi...@redhat.com; h...@zytor.com; > fenghua...@intel.com; reinette.cha...@intel.com; > vikas.shiva...@linux.intel.com; t

Re: [RFC PATCH 00/10] arch/x86: AMD QoS support

2018-10-02 Thread Moger, Babu
On 10/02/2018 01:46 PM, Fenghua Yu wrote: > On Tue, Oct 02, 2018 at 05:44:47PM +0000, Moger, Babu wrote: >> Hi Fenghua, >> >>> -Original Message- >>> From: Fenghua Yu >>> Sent: Tuesday, October 2, 2018 12:07 PM >>> On Mon,

Re: [RFC PATCH 03/10] arch/x86: Re-arrange RDT init code

2018-10-02 Thread Moger, Babu
Hi Reinette, Thanks for the review. My response below. On 10/02/2018 02:21 PM, Reinette Chatre wrote: > Hi Babu, > > On 9/24/2018 12:19 PM, Moger, Babu wrote: >> Re-organize the RDT init code. Separate the call sequence for each >> feature. That way, it is easy to ca

RE: [RFC PATCH 00/10] arch/x86: AMD QoS support

2018-09-27 Thread Moger, Babu
Thomas, > -Original Message- > From: Thomas Gleixner > Sent: Thursday, September 27, 2018 3:14 PM > To: Moger, Babu > Cc: mi...@redhat.com; h...@zytor.com; fenghua...@intel.com; > reinette.cha...@intel.com; vikas.shiva...@linux.intel.com; > tony.l...@intel.com;

Re: [PATCH v2 01/11] arch/x86: Start renaming the rdt files to more generic names

2018-10-10 Thread Moger, Babu
Thomas, On 10/10/2018 09:16 AM, Thomas Gleixner wrote: > Babu, > > On Wed, 10 Oct 2018, Moger, Babu wrote: >> On 10/09/2018 05:01 PM, Reinette Chatre wrote: >>> As far as all the code you touch is concerned it may be easier and cause >>> less confusion for now

Re: [PATCH v2 01/11] arch/x86: Start renaming the rdt files to more generic names

2018-10-10 Thread Moger, Babu
Hi Reinette, On 10/09/2018 05:01 PM, Reinette Chatre wrote: > Hi Babu, > > On 10/9/2018 2:17 PM, Moger, Babu wrote: >> On 10/09/2018 11:39 AM, Reinette Chatre wrote: >>> Hi Babu, >>> >>> On 10/5/2018 1:55 PM, Moger, Babu wrote: >>>> New

Re: [PATCH v2 01/11] arch/x86: Start renaming the rdt files to more generic names

2018-10-10 Thread Moger, Babu
On 10/10/2018 12:53 PM, Reinette Chatre wrote: > Hi Babu, > > On 10/10/2018 7:11 AM, Moger, Babu wrote: >> On 10/09/2018 05:01 PM, Reinette Chatre wrote: >>> On 10/9/2018 2:17 PM, Moger, Babu wrote: >>>> On 10/09/2018 11:39 AM, Reinette Chatre wrote: >

Re: [PATCH v2 07/11] arch/x86: Initialize the resource functions that are different

2018-10-09 Thread Moger, Babu
On 10/09/2018 12:18 PM, Reinette Chatre wrote: > Hi Babu, > > On 10/5/2018 1:55 PM, Moger, Babu wrote: >> diff --git a/arch/x86/kernel/cpu/rdt_ctrlmondata.c >> b/arch/x86/kernel/cpu/rdt_ctrlmondata.c >> index 812cc5c5e39e..ee3e8389d8d2 100644 >> --- a/arch

Re: [PATCH v2 01/11] arch/x86: Start renaming the rdt files to more generic names

2018-10-09 Thread Moger, Babu
Hi Reinette, On 10/09/2018 11:39 AM, Reinette Chatre wrote: > Hi Babu, > > On 10/5/2018 1:55 PM, Moger, Babu wrote: >> New generation of AMD processors start support RDT(or QOS) features. >> With more than one vendors supporting these features, it seems more >> approp

[RFC PATCH 05/10] arch/x86: Use new config parameter PLATFORM_QOS for compilation

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

[RFC PATCH 06/10] arch/x86: Initialize the resource functions that are different

2018-09-24 Thread Moger, Babu
Initialize the resource functions that are different between the vendors. Some features are initialized differently between the vendors. For example, MBA feature varies significantly between Intel and AMD. Separate the initialization of these resource functions. That way we can easily add AMD's

[RFC PATCH 02/10] arch/x86: Rename the RDT functions and definitions

2018-09-24 Thread Moger, Babu
As AMD is starting to support RDT(or QOS) features, rename the RDT functions and definitions to more generic names. Signed-off-by: Babu Moger --- arch/x86/include/asm/rdt_sched.h | 22 +++--- arch/x86/kernel/cpu/rdt.c | 24

[RFC PATCH 07/10] arch/x86: Bring few more functions into the resource structure

2018-09-24 Thread Moger, Babu
Bring all resource functions that are different between the vendors into resource structure and initialize them dynamically. Implement these functions separately for each vendors. update_mba_bw : Feedback loop bandwidth update functionality is not needed for AMD. cbm_validate :

[RFC PATCH 08/10] arch/x86: Introduce new config parameter AMD_QOS

2018-09-24 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 PLATFORM_QOS if selected. Signed-off-by: Babu Moger --- arch/x86/Kconfig | 17

[RFC PATCH 03/10] arch/x86: Re-arrange RDT init code

2018-09-24 Thread Moger, Babu
Re-organize the RDT init code. Separate the call sequence for each feature. That way, it is easy to call quirks or features separately for each vendor if there are differences. Signed-off-by: Babu Moger --- arch/x86/kernel/cpu/rdt.c | 44 --- 1 file changed,

[RFC PATCH 10/10] arch/x86: Introduce QOS feature for AMD

2018-09-24 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) There are differences in the

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

2018-09-24 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. Signed-off-by: Babu Moger Signed-off-by: Sherry Hurwitz --- arch/x86/kernel/cpu/scattered.c | 1 + 1 file changed, 1 insertion(+) diff

[RFC PATCH 00/10] arch/x86: AMD QoS support

2018-09-24 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

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

2018-09-24 Thread Moger, Babu
New generation of AMD processors start support RDT(or QOS) features. With more than one vendors supporting these features, it seems more appropriate to rename these files. Signed-off-by: Babu Moger --- arch/x86/include/asm/{intel_rdt_sched.h => rdt_sched.h} | 0 arch/x86/kernel/cpu/Makefile

[RFC PATCH 04/10] arch/x86: Introduce a new config parameter PLATFORM_QOS

2018-09-24 Thread Moger, Babu
Introduces a new config parameter PLATFORM_QOS. This will be used as a common config parameter for both Intel and AMD. Each vendor will have their own config parameter to enable RDT feature. One for Intel(INTEL_RDT) and one for AMD(AMD_QOS). It can be enabled or disabled separately. The new

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

[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 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

  1   2   3   >