Re: [PATCH v6 4/4] x86: Tune 'Saltwell' microarch the same was a 'Bonnell'

[Noah Goldstein <goldstein.w.n@gmail.com>]

On Wed, May 10, 2023 at 5:04 PM H.J. Lu <hjl.tools@gmail.com> wrote:
>
> On Tue, May 9, 2023 at 5:34 PM Noah Goldstein <goldstein.w.n@gmail.com> wrote:
> >
> > Saltwell is just a die shrink of Bonnell, so the same
> > micro-architectural optimization preferences apply.
> > ---
> >  sysdeps/x86/cpu-features.c | 5 +++--
> >  1 file changed, 3 insertions(+), 2 deletions(-)
> >
> > diff --git a/sysdeps/x86/cpu-features.c b/sysdeps/x86/cpu-features.c
> > index 4cc1cd9fed..517b1be34c 100644
> > --- a/sysdeps/x86/cpu-features.c
> > +++ b/sysdeps/x86/cpu-features.c
> > @@ -678,7 +678,9 @@ init_cpu_features (struct cpu_features *cpu_features)
> >               break;
> >
> >               /* Unaligned load versions are faster than SSSE3
> > -                    on Airmont, Silvermont, Goldmont, and Goldmont Plus.  */
> > +                    on Saltwell, Airmont, Silvermont, Goldmont, and Goldmont
> > +             Plus.  */
> > +           case INTEL_ATOM_SALTWELL:
>
> It should be grouped with Bonnell.
>
Done.
> >             case INTEL_ATOM_AIRMONT:
> >             case INTEL_ATOM_SILVERMONT:
> >             case INTEL_ATOM_GOLDMONT:
> > @@ -710,7 +712,6 @@ init_cpu_features (struct cpu_features *cpu_features)
> >               /* Default tuned atom microarch.  */
> >             case INTEL_ATOM_SIERRAFOREST:
> >             case INTEL_ATOM_GRANDRIDGE:
> > -           case INTEL_ATOM_SALTWELL:
> >               goto default_tuning;
> >
> >               /* Bigcore Tuning.  */
> > --
> > 2.34.1
> >
>
>
> --
> H.J.