Re: [PATCH] sparc64: recognize and support Sonoma CPU type

[Khalid Aziz]

On 04/20/2016 09:54 AM, Allen Pais wrote:
> > Signed-off-by: Khalid Aziz <khalid.aziz@xxxxxxxxxx <mailto:khalid.aziz@xxxxxxxxxx>>
> > ---
> >   arch/sparc/include/asm/spitfire.h | 1 +
> >   arch/sparc/kernel/cpu.c           | 6 ++++++
> >   arch/sparc/kernel/cpumap.c        | 1 +
> >   arch/sparc/kernel/head_64.S       | 8 ++++++++
> >   arch/sparc/kernel/setup_64.c      | 7 ++++++-
> >   arch/sparc/mm/init_64.c           | 3 +++
> >   6 files changed, 25 insertions(+), 1 deletion(-)
> >
> > diff --git a/arch/sparc/include/asm/spitfire.h b/arch/sparc/include/asm/spitfire.h
> > index 56f9338..1d8321c 100644
> > --- a/arch/sparc/include/asm/spitfire.h
> > +++ b/arch/sparc/include/asm/spitfire.h
> > @@ -48,6 +48,7 @@
> >   #define SUN4V_CHIP_SPARC_M6    0x06
> >   #define SUN4V_CHIP_SPARC_M7    0x07
> >   #define SUN4V_CHIP_SPARC64X    0x8a
> > +#define SUN4V_CHIP_SPARC_SN    0x8b
> >   #define SUN4V_CHIP_UNKNOWN     0xff
>
> Looks good to me. Do you think adding these also this patch makes sense or
> it could be a separate patch.
>
>          }
> -       if (!strcmp(sparc_pmu_type, "sparc-m7")) {
> +       if (!strcmp(sparc_pmu_type, "sparc-m7") ||
> +           !strcmp(sparc_pmu_type, "sparc-sn")) {
>                  sparc_pmu = &sparc_m7_pmu;
>                  return true;
>
> Acked-by: Allen Pais <allen.pais@xxxxxxxxxx>
>
> - Allen

I would recommend that to be a separate patch. My goal with this patch 
is to address the core capabilities of sonoma and enable kernel to get 
optimized performance from the processor. Monitoring the performance can 
be a separate patch.

--
Khalid