Re: [dmidecode] [PATCH v2] Support new byte value of Processor Upgrade field from SMBIOS 3.5.0

[Coiby Xu]

On Thu, Oct 21, 2021 at 04:02:17PM +0200, Jean Delvare wrote:
> On Thu, 21 Oct 2021 15:35:19 +0800, Coiby Xu wrote:
> > Intel(R) 4th Generation Xeon Scalable 'Sapphire Rapids' processors
> > have byte value 3Fh which means Socket LGA4677 for Processor
> > Information - Processor Upgrade field.
> >
> > Signed-off-by: Coiby Xu <coxu@redhat.com>
> > ---
> >  dmidecode.c | 5 +++--
> >  1 file changed, 3 insertions(+), 2 deletions(-)
> >
> > diff --git a/dmidecode.c b/dmidecode.c
> > index 0b44182..3147d4d 100644
> > --- a/dmidecode.c
> > +++ b/dmidecode.c
> > @@ -1394,10 +1394,11 @@ static const char *dmi_processor_upgrade(u8 code)
> >                 "Socket BGA1510",
> >                 "Socket BGA1528",
> >                 "Socket LGA4189",
> > -               "Socket LGA1200" /* 0x3E */
> > +               "Socket LGA1200",
> > +               "Socket LGA4677" /* 0x3F */
> >         };
> >
> > -       if (code >= 0x01 && code <= 0x3E)
> > +       if (code >= 0x01 && code <= 0x3F)
> >                 return upgrade[code - 0x01];
> >         return out_of_spec;
> >  }
>
> Hehe, already applied a few minutes ago :-)

Thanks! Btw, I've also updated the commit message in v2.
"Intel(R) Xeon(R) Platinum 8375C CPU @ 2.90GHz has" has been changed 
to "Intel(R) 4th Generation Xeon Scalable 'Sapphire Rapids' 
processors have". Can this issue still be fixed?

> --
> Jean Delvare
> SUSE L3 Support

--
Best regards,
Coiby