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

From: Moger, Babu
Date: Sun Oct 07 2018 - 21:22:21 EST


Hi Fenghua,

> -----Original Message-----
> From: Fenghua Yu <fenghua.yu@xxxxxxxxx>
> Sent: Friday, October 5, 2018 6:39 PM
> To: Moger, Babu <Babu.Moger@xxxxxxx>
> Cc: tglx@xxxxxxxxxxxxx; mingo@xxxxxxxxxx; hpa@xxxxxxxxx;
> reinette.chatre@xxxxxxxxx; fenghua.yu@xxxxxxxxx; james.morse@xxxxxxx;
> vikas.shivappa@xxxxxxxxxxxxxxx; tony.luck@xxxxxxxxx; x86@xxxxxxxxxx;
> peterz@xxxxxxxxxxxxx; pombredanne@xxxxxxxx;
> gregkh@xxxxxxxxxxxxxxxxxxx; kstewart@xxxxxxxxxxxxxxxxxxx; bp@xxxxxxx;
> rafael.j.wysocki@xxxxxxxxx; ak@xxxxxxxxxxxxxxx;
> kirill.shutemov@xxxxxxxxxxxxxxx; xiaochen.shen@xxxxxxxxx;
> colin.king@xxxxxxxxxxxxx; Hurwitz, Sherry <sherry.hurwitz@xxxxxxx>;
> Lendacky, Thomas <Thomas.Lendacky@xxxxxxx>; pbonzini@xxxxxxxxxx;
> dwmw@xxxxxxxxxxxx; luto@xxxxxxxxxx; jroedel@xxxxxxx;
> jannh@xxxxxxxxxx; dima@xxxxxxxxxx; jpoimboe@xxxxxxxxxx;
> vkuznets@xxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx
> Subject: Re: [PATCH v2 05/11] arch/x86: Introduce a new config parameter
> PLATFORM_QOS
>
> On Fri, Oct 05, 2018 at 08:55:52PM +0000, 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 <babu.moger@xxxxxxx>
> > ---
> > 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