RE: [PATCH V2 1/6] perf/x86/intel: Add Grand Ridge and Sierra Forest

From: Luck, Tony
Date: Thu Jun 08 2023 - 12:20:34 EST


> Then I'm hoping their take-away is that random gibberish names don't
> help anybody. The whole Intel naming scheme is impenetrable crap.

> > +#define INTEL_FAM6_ATOM_CRESTMONT_X 0xAF /* Sierra Forest */
> > +#define INTEL_FAM6_ATOM_CRESTMONT 0xB6 /* Grand Ridge */

This just adds another layer of confusion. Sure, these two models are based
on the same core. But giving the illusion that they are somehow the same will
lead to tears before bedtime:

1) They each took a snapshot of that core design on different dates, so there
are logic differences.
2) Feature fuses will be different
3) Microcode will be different
4) BIOS will be different
5) "uncore" is different, so anything implemented outside of the core
will be different.

-Tony