Re: [pve-devel] [PATCH qemu-server] cpuconfig: add missing qemu 8.0 cpu models

["DERUMIER, Alexandre" <alexandre.derumier@groupe-cyllene.com>]

Le lundi 19 juin 2023 à 10:46 +0200, Fiona Ebner a écrit :
> Am 09.06.23 um 12:27 schrieb Alexandre Derumier:
> > Signed-off-by: Alexandre Derumier <aderumier@odiso.com>
> > ---
> >  PVE/QemuServer/CPUConfig.pm | 14 ++++++++++++++
> >  1 file changed, 14 insertions(+)
> > 
> > diff --git a/PVE/QemuServer/CPUConfig.pm
> > b/PVE/QemuServer/CPUConfig.pm
> > index fb0861b..77fc682 100644
> > --- a/PVE/QemuServer/CPUConfig.pm
> > +++ b/PVE/QemuServer/CPUConfig.pm
> > @@ -66,16 +66,28 @@ my $cpu_vendor_list = {
> >      'Skylake-Client' => 'GenuineIntel',
> >      'Skylake-Client-IBRS' => 'GenuineIntel',
> >      'Skylake-Client-noTSX-IBRS' => 'GenuineIntel',
> > +    'Skylake-Client-v4' => 'GenuineIntel',
> >      'Skylake-Server' => 'GenuineIntel',
> >      'Skylake-Server-IBRS' => 'GenuineIntel',
> >      'Skylake-Server-noTSX-IBRS' => 'GenuineIntel',
> > +    'Skylake-Server-v4' => 'GenuineIntel',
> > +    'Skylake-Server-v5' => 'GenuineIntel',
> >      'Cascadelake-Server' => 'GenuineIntel',
> >      'Cascadelake-Server-noTSX' => 'GenuineIntel',
> 
> The above is an alias for Cascadelake-Server-v3, but -v2 is still
> missing here AFAICT.
ok I'll add them

> 
> > +    'Cascadelake-Server-v4' => 'GenuineIntel',
> > +    'Cascadelake-Server-v5' => 'GenuineIntel',
> > +    'CooperLake' => 'GenuineIntel',
> > +    'CooperLake-v2' => 'GenuineIntel',
> 
> It's Cooperlake{-v2} with a small l.
> 
oops, thanks,will fix it.

> >      KnightsMill => 'GenuineIntel',
> >      'Icelake-Client' => 'GenuineIntel', # depreacated, removed
> > with QEMU 7.1
> >      'Icelake-Client-noTSX' => 'GenuineIntel', # depreacated,
> > removed with QEMU 7.1
> >      'Icelake-Server' => 'GenuineIntel',
> >      'Icelake-Server-noTSX' => 'GenuineIntel',
> > +    'Icelake-Server-v3' => 'GenuineIntel',
> > +    'Icelake-Server-v4' => 'GenuineIntel',
> > +    'Icelake-Server-v5' => 'GenuineIntel',
> > +    'Icelake-Server-v6' => 'GenuineIntel',
> > +    'SapphireRapids' => 'GenuineIntel',
> >  
> 

> I also noticed that
> Denverton{-v2,-v3}
> Snowridge{-v2,-v3,-v4}

Both are atom cpu. I don't known if we want to support them ? (they
seem to support vtx, so it should work with kvm)

> Dhyana{-v2}
This is a chinese fork of epyc (without aes), I don't known if we need
to support it ?
> 
> n270
> are also missing from the list. Should we add them (maybe not n270
> ;))?
>