Hi Reinette, On 10/12/2018 02:07 PM, Reinette Chatre wrote: > Hi Babu, > > On 10/11/2018 1:33 PM, Moger, Babu wrote: >> New generation of AMD processors start supporting RDT(or QOS) features. >> With more than one vendors supporting these features, it seems more >> appropriate to rename these files. >> >> Changed intel_rdt to resctrl where applicable. >> >> Signed-off-by: Babu Moger <[email protected]> >> --- >> arch/x86/include/asm/{intel_rdt_sched.h => resctrl_sched.h} | 0 >> arch/x86/kernel/cpu/Makefile | 6 +++--- >> arch/x86/kernel/cpu/{intel_rdt.c => resctrl.c} | 4 ++-- >> arch/x86/kernel/cpu/{intel_rdt.h => resctrl.h} | 6 +++--- >> .../cpu/{intel_rdt_ctrlmondata.c => resctrl_ctrlmondata.c} | 2 +- >> .../kernel/cpu/{intel_rdt_monitor.c => resctrl_monitor.c} | 2 +- >> .../cpu/{intel_rdt_pseudo_lock.c => resctrl_pseudo_lock.c} | 6 +++--- >> ..._rdt_pseudo_lock_event.h => resctrl_pseudo_lock_event.h} | 2 +- >> .../kernel/cpu/{intel_rdt_rdtgroup.c => resctrl_rdtgroup.c} | 4 ++-- >> arch/x86/kernel/process_32.c | 2 +- >> arch/x86/kernel/process_64.c | 2 +- >> 11 files changed, 18 insertions(+), 18 deletions(-) >> rename arch/x86/include/asm/{intel_rdt_sched.h => resctrl_sched.h} (100%) >> rename arch/x86/kernel/cpu/{intel_rdt.c => resctrl.c} (99%) >> rename arch/x86/kernel/cpu/{intel_rdt.h => resctrl.h} (99%) >> rename arch/x86/kernel/cpu/{intel_rdt_ctrlmondata.c => >> resctrl_ctrlmondata.c} (99%) >> rename arch/x86/kernel/cpu/{intel_rdt_monitor.c => resctrl_monitor.c} (99%) >> rename arch/x86/kernel/cpu/{intel_rdt_pseudo_lock.c => >> resctrl_pseudo_lock.c} (99%) >> rename arch/x86/kernel/cpu/{intel_rdt_pseudo_lock_event.h => >> resctrl_pseudo_lock_event.h} (95%) >> rename arch/x86/kernel/cpu/{intel_rdt_rdtgroup.c => resctrl_rdtgroup.c} >> (99%) >> > > I missed this earlier ... this change creates a need to change the > MAINTAINERS file. There were some other things that a run of
Yes. I see that now. We need to update MAINTAINERS file. The file name Documentation/x86/intel_rdt_ui.txt needs rename. Also contents of the file needs some minor changes. Will take care of it in next revision. > "checkpatch.pl --strict" picked up in the other patches that you could > consider. Yes. With --strict I see some more warnings. Will try to fix most of it. Sorry, I did not run --strict before. There are couple of "line over 80 characters" warnings which are difficult to handle. I will skip those. > > Reinette >
