Re: [PATCH 4/5] x86/intel: Aggregate microserver naming

[Ingo Molnar]

* Peter Zijlstra <peterz@xxxxxxxxxxxxx> wrote:

> On Mon, Aug 26, 2019 at 11:27:50AM +0200, Ingo Molnar wrote:
> > 
> > * Peter Zijlstra <peterz@xxxxxxxxxxxxx> wrote:
> > 
> > > Currently big microservers have _XEON_D while small microservers have
> > > _X, Make it uniformly: _D.
> > > 
> > > for i in `git grep -l "INTEL_FAM6_.*_\(X\|XEON_D\)"`
> > > do
> > > 	sed -i -e 's/\(INTEL_FAM6_ATOM_.*\)_X/\1_D/g' \
> > >                -e 's/\(INTEL_FAM6_.*\)_XEON_D/\1_D/g' ${i}
> > > done
> > > 
> > > Signed-off-by: Peter Zijlstra (Intel) <peterz@xxxxxxxxxxxxx>
> > > Cc: x86@xxxxxxxxxx
> > > Cc: Dave Hansen <dave.hansen@xxxxxxxxx>
> > > Cc: Borislav Petkov <bp@xxxxxxxxx>
> > > Cc: Thomas Gleixner <tglx@xxxxxxxxxxxxx>
> > > Cc: Tony Luck <tony.luck@xxxxxxxxx>
> > > ---
> > >  arch/x86/events/intel/core.c          |   20 ++++++++++----------
> > >  arch/x86/events/intel/cstate.c        |   12 ++++++------
> > >  arch/x86/events/intel/pt.c            |    2 +-
> > >  arch/x86/events/intel/rapl.c          |    4 ++--
> > >  arch/x86/events/intel/uncore.c        |    4 ++--
> > >  arch/x86/events/msr.c                 |    6 +++---
> > >  arch/x86/include/asm/intel-family.h   |   10 +++++-----
> > >  arch/x86/kernel/apic/apic.c           |    2 +-
> > >  arch/x86/kernel/cpu/intel.c           |    4 ++--
> > >  arch/x86/kernel/cpu/mce/intel.c       |    2 +-
> > >  arch/x86/kernel/tsc.c                 |    2 +-
> > >  drivers/cpufreq/intel_pstate.c        |    6 +++---
> > >  drivers/edac/i10nm_base.c             |    4 ++--
> > >  drivers/edac/pnd2_edac.c              |    2 +-
> > >  tools/power/x86/turbostat/turbostat.c |   22 +++++++++++-----------
> > >  15 files changed, 51 insertions(+), 51 deletions(-)
> > 
> > I've added the additional renames below, accounting for recent changes in 
> > cpu/common.c.
> 
> Thanks, I've respun these patches (they didn't actually build), and I'll
> make sure to include that one when I repost.

I've already respun them myself and committed them to tip:x86/cpu :-)

Thanks,

	Ingo