RE: [PATCH v8 01/13] arch/resctrl: Start renaming the rdt files to more generic names
From: Moger, Babu
Date: Mon Nov 19 2018 - 15:11:45 EST
Boris/Thomas,
> -----Original Message-----
> From: Thomas Gleixner <tglx@xxxxxxxxxxxxx>
> Sent: Monday, November 19, 2018 1:14 PM
> To: Borislav Petkov <bp@xxxxxxxxx>
> Cc: Moger, Babu <Babu.Moger@xxxxxxx>; mingo@xxxxxxxxxx;
> corbet@xxxxxxx; fenghua.yu@xxxxxxxxx; reinette.chatre@xxxxxxxxx;
> peterz@xxxxxxxxxxxxx; gregkh@xxxxxxxxxxxxxxxxxxx;
> davem@xxxxxxxxxxxxx; akpm@xxxxxxxxxxxxxxxxxxxx; 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 01/13] arch/resctrl: Start renaming the rdt files to
> more generic names
>
> On Sun, 18 Nov 2018, Borislav Petkov wrote:
> >
> > Now, it looks to me like the whole resctrl thing begs for its own dir:
> >
> > arch/x86/kernel/cpu/resctrl/core.c
> > arch/x86/kernel/cpu/resctrl/ctrlmondata.c
> > arch/x86/kernel/cpu/resctrl/internal.h
> > arch/x86/kernel/cpu/resctrl/monitor.c
> > arch/x86/kernel/cpu/resctrl/pseudo_lock.c
> > arch/x86/kernel/cpu/resctrl/pseudo_lock_event.h
> > arch/x86/kernel/cpu/resctrl/rdtgroup.c
> >
> > This way, the whole RDT machinery will be confined to its own directory.
>
> Fine with that.
Ok. Sure. Will change it. How does this look like?
arch/x86/kernel/cpu/resctrl/res.c
arch/x86/kernel/cpu/resctrl/ctrlmondata.c
arch/x86/kernel/cpu/resctrl/res.h
arch/x86/kernel/cpu/resctrl/monitor.c
arch/x86/kernel/cpu/resctrl/pseudo_lock.c
arch/x86/kernel/cpu/resctrl/pseudo_lock_event.h
arch/x86/kernel/cpu/resctrl/rdtgroup.c
Changed core.c and internel.h to res.c and res.h respectively. Both of
these files are about resource definitions and initialization.
>
> Thanks,
>
> tglx