RE: [PATCH v8 05/13] arch/resctrl: Rename config parameter INTEL_RDT to RESCTRL

[Moger, Babu]

Boris,

> -----Original Message-----
> From: Borislav Petkov <bp@xxxxxxxxx>
> Sent: Tuesday, November 20, 2018 3:42 AM
> To: Moger, Babu <Babu.Moger@xxxxxxx>
> Cc: tglx@xxxxxxxxxxxxx; mingo@xxxxxxxxxx; corbet@xxxxxxx;
> fenghua.yu@xxxxxxxxx; reinette.chatre@xxxxxxxxx; peterz@xxxxxxxxxxxxx;
> gregkh@xxxxxxxxxxxxxxxxxxx; davem@xxxxxxxxxxxxx; akpm@linux-
> foundation.org; hpa@xxxxxxxxx; x86@xxxxxxxxxx;
> mchehab+samsung@xxxxxxxxxx; arnd@xxxxxxxx;
> kstewart@xxxxxxxxxxxxxxxxxxx; pombredanne@xxxxxxxx;
> rafael@xxxxxxxxxx; kirill.shutemov@xxxxxxxxxxxxxxx; tony.luck@xxxxxxxxx;
> qianyue.zj@xxxxxxxxxxxxxxx; xiaochen.shen@xxxxxxxxx;
> pbonzini@xxxxxxxxxx; Singh, Brijesh <brijesh.singh@xxxxxxx>; Hurwitz,
> Sherry <sherry.hurwitz@xxxxxxx>; dwmw2@xxxxxxxxxxxxx; Lendacky,
> Thomas <Thomas.Lendacky@xxxxxxx>; luto@xxxxxxxxxx; joro@xxxxxxxxxx;
> jannh@xxxxxxxxxx; vkuznets@xxxxxxxxxx; rian@xxxxxxxxxxxx;
> jpoimboe@xxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; linux-
> doc@xxxxxxxxxxxxxxx
> Subject: Re: [PATCH v8 05/13] arch/resctrl: Rename config parameter
> INTEL_RDT to RESCTRL
> 
> On Fri, Nov 16, 2018 at 08:54:30PM +0000, Moger, Babu wrote:
> > Now Resource control feature is supported by both Intel and AMD.
> > Rename the INTEL_RDT to vendor neutral RESCTRL.
> >
> > Signed-off-by: Babu Moger <babu.moger@xxxxxxx>
> > ---
> >  arch/x86/Kconfig                     | 4 ++--
> >  arch/x86/include/asm/resctrl_sched.h | 4 ++--
> >  arch/x86/kernel/cpu/Makefile         | 4 ++--
> >  include/linux/sched.h                | 2 +-
> >  4 files changed, 7 insertions(+), 7 deletions(-)
> >
> > diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig
> > index 1a0be022f91d..4aae7aba4d61 100644
> > --- a/arch/x86/Kconfig
> > +++ b/arch/x86/Kconfig
> > @@ -445,8 +445,8 @@ config RETPOLINE
> >  	  code are eliminated. Since this includes the syscall entry path,
> >  	  it is not entirely pointless.
> >
> > -config INTEL_RDT
> > -	bool "Intel Resource Director Technology support"
> > +config RESCTRL
> > +	bool "Resource Control feature support"
> 
> s/feature // - they're all features. :)

Ok sure
> 
> >  	default n
> >  	depends on X86 && CPU_SUP_INTEL
> 
> Patch 9/13 is begging to be merged with this one.

Ok. I will merge these changes in patch 9/13.

> 
> --
> Regards/Gruss,
>     Boris.
> 
> Good mailing practices for 400: avoid top-posting and trim the reply.