RE: [PATCH v2 05/11] arch/x86: Introduce a new config parameter PLATFORM_QOS
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...@linux.intel.com; tony.l...@intel.com; x...@kernel.org; > pet...@infradead.org; pombreda...@nexb.com; > gre...@linuxfoundation.org; kstew...@linuxfoundation.org; b...@suse.de; > rafael.j.wyso...@intel.com; a...@linux.intel.com; > kirill.shute...@linux.intel.com; xiaochen.s...@intel.com; > colin.k...@canonical.com; Hurwitz, Sherry ; > Lendacky, Thomas ; pbonz...@redhat.com; > d...@amazon.co.uk; l...@kernel.org; jroe...@suse.de; > ja...@google.com; d...@arista.com; jpoim...@redhat.com; > vkuzn...@redhat.com; linux-kernel@vger.kernel.org > Subject: Re: [PATCH v2 05/11] arch/x86: Introduce a new config parameter > PLATFORM_QOS > > On Fri, Oct 05, 2018 at 08:55:52PM +, Moger, Babu wrote: > > 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 parameter PLATFORM_QOS will be > dependent > > on INTEL_RDT or AMD_QOS. > > > > Signed-off-by: Babu Moger > > --- > > arch/x86/Kconfig | 4 > > 1 file changed, 4 insertions(+) > > > > diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig > > index 1a0be022f91d..7f2da780a327 100644 > > --- a/arch/x86/Kconfig > > +++ b/arch/x86/Kconfig > > @@ -458,6 +458,10 @@ config INTEL_RDT > > > > Say N if unsure. > > > > +config PLATFORM_QOS > > + def_bool y > > + depends on X86 && INTEL_RDT > > + > > Can change "PLATFORM_QOS" to a more neutral name "RESCTRL"? Yes. Will change it. > > Thanks. > > -Fenghua
RE: [PATCH v2 05/11] arch/x86: Introduce a new config parameter PLATFORM_QOS
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...@linux.intel.com; tony.l...@intel.com; x...@kernel.org; > pet...@infradead.org; pombreda...@nexb.com; > gre...@linuxfoundation.org; kstew...@linuxfoundation.org; b...@suse.de; > rafael.j.wyso...@intel.com; a...@linux.intel.com; > kirill.shute...@linux.intel.com; xiaochen.s...@intel.com; > colin.k...@canonical.com; Hurwitz, Sherry ; > Lendacky, Thomas ; pbonz...@redhat.com; > d...@amazon.co.uk; l...@kernel.org; jroe...@suse.de; > ja...@google.com; d...@arista.com; jpoim...@redhat.com; > vkuzn...@redhat.com; linux-kernel@vger.kernel.org > Subject: Re: [PATCH v2 05/11] arch/x86: Introduce a new config parameter > PLATFORM_QOS > > On Fri, Oct 05, 2018 at 08:55:52PM +, Moger, Babu wrote: > > 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 parameter PLATFORM_QOS will be > dependent > > on INTEL_RDT or AMD_QOS. > > > > Signed-off-by: Babu Moger > > --- > > arch/x86/Kconfig | 4 > > 1 file changed, 4 insertions(+) > > > > diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig > > index 1a0be022f91d..7f2da780a327 100644 > > --- a/arch/x86/Kconfig > > +++ b/arch/x86/Kconfig > > @@ -458,6 +458,10 @@ config INTEL_RDT > > > > Say N if unsure. > > > > +config PLATFORM_QOS > > + def_bool y > > + depends on X86 && INTEL_RDT > > + > > Can change "PLATFORM_QOS" to a more neutral name "RESCTRL"? Yes. Will change it. > > Thanks. > > -Fenghua
Re: [PATCH v2 05/11] arch/x86: Introduce a new config parameter PLATFORM_QOS
On Fri, Oct 05, 2018 at 08:55:52PM +, Moger, Babu wrote: > 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 parameter PLATFORM_QOS will be dependent > on INTEL_RDT or AMD_QOS. > > Signed-off-by: Babu Moger > --- > arch/x86/Kconfig | 4 > 1 file changed, 4 insertions(+) > > diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig > index 1a0be022f91d..7f2da780a327 100644 > --- a/arch/x86/Kconfig > +++ b/arch/x86/Kconfig > @@ -458,6 +458,10 @@ config INTEL_RDT > > Say N if unsure. > > +config PLATFORM_QOS > + def_bool y > + depends on X86 && INTEL_RDT > + Can change "PLATFORM_QOS" to a more neutral name "RESCTRL"? Thanks. -Fenghua
Re: [PATCH v2 05/11] arch/x86: Introduce a new config parameter PLATFORM_QOS
On Fri, Oct 05, 2018 at 08:55:52PM +, Moger, Babu wrote: > 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 parameter PLATFORM_QOS will be dependent > on INTEL_RDT or AMD_QOS. > > Signed-off-by: Babu Moger > --- > arch/x86/Kconfig | 4 > 1 file changed, 4 insertions(+) > > diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig > index 1a0be022f91d..7f2da780a327 100644 > --- a/arch/x86/Kconfig > +++ b/arch/x86/Kconfig > @@ -458,6 +458,10 @@ config INTEL_RDT > > Say N if unsure. > > +config PLATFORM_QOS > + def_bool y > + depends on X86 && INTEL_RDT > + Can change "PLATFORM_QOS" to a more neutral name "RESCTRL"? Thanks. -Fenghua
[PATCH v2 05/11] arch/x86: Introduce a new config parameter PLATFORM_QOS
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 parameter PLATFORM_QOS will be dependent on INTEL_RDT or AMD_QOS. Signed-off-by: Babu Moger --- arch/x86/Kconfig | 4 1 file changed, 4 insertions(+) diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig index 1a0be022f91d..7f2da780a327 100644 --- a/arch/x86/Kconfig +++ b/arch/x86/Kconfig @@ -458,6 +458,10 @@ config INTEL_RDT Say N if unsure. +config PLATFORM_QOS + def_bool y + depends on X86 && INTEL_RDT + if X86_32 config X86_BIGSMP bool "Support for big SMP systems with more than 8 CPUs" -- 2.17.1
[PATCH v2 05/11] arch/x86: Introduce a new config parameter PLATFORM_QOS
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 parameter PLATFORM_QOS will be dependent on INTEL_RDT or AMD_QOS. Signed-off-by: Babu Moger --- arch/x86/Kconfig | 4 1 file changed, 4 insertions(+) diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig index 1a0be022f91d..7f2da780a327 100644 --- a/arch/x86/Kconfig +++ b/arch/x86/Kconfig @@ -458,6 +458,10 @@ config INTEL_RDT Say N if unsure. +config PLATFORM_QOS + def_bool y + depends on X86 && INTEL_RDT + if X86_32 config X86_BIGSMP bool "Support for big SMP systems with more than 8 CPUs" -- 2.17.1