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
