Re: [libvirt PATCH] cpu_map: Add EPYC-Rome model
- First Post
- Replies
- Stats
-
Go to
- ----- 2024 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2023 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2022 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2021 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2020 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2019 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2018 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2017 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2016 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2015 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2014 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2013 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2012 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2011 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2010 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2009 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2008 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2007 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2006 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2005 -----
- December
Okay, that did not go as expected. Anyway
This patch add the qemu EPYC-Rome model and includes cpuid data from an
EPYC 7502 system as well as updated cpuid information from a Ryzen 9
3900 system.
Since the EPYC 7502 and the Ryzen 9 are pretty much identical, the
latter could be dropped as there is no corresponding qemu model.
Markus
Attachments:
- signature.asc (application/pgp-signature — 488 bytes)
Wednesday, 30 September
Wed, 30 Sep
4:23 a.m.
New subject: [libvirt PATCH] cpu_map: Add EPYC-Rome model
On Mon, Sep 28, 2020 at 13:20:06 +0200, Markus Schade wrote:
Okay, that did not go as expected. Anyway
I'm not sure what did not go as expected...
This patch add the qemu EPYC-Rome model and includes cpuid data from
an
EPYC 7502 system
Great.
as well as updated cpuid information from a Ryzen 9 3900 system.
These changes should be dropped. We already have this model covered and
regenerating the data files for existing CPUs just causes unnecessary
and often unrelated changes.
Since the EPYC 7502 and the Ryzen 9 are pretty much identical, the
latter could be dropped as there is no corresponding qemu model.
While we don't have a special model Ryzen 9, we still want to have it
covered in the test suit to check we assign the appropriate model to it.
Anyway, more on this in my reply to the patch itself.
Thanks for the work so far.
Jirka
On 30.09.20 at 11:23 Jiri Denemark wrote:
On Mon, Sep 28, 2020 at 13:20:06 +0200, Markus Schade wrote:
> Okay, that did not go as expected. Anyway
I'm not sure what did not go as expected...
I had meant to sent the cover letter with the patch ;-)
> as well as updated cpuid information from a Ryzen 9 3900 system.
These changes should be dropped. We already have this model covered and
regenerating the data files for existing CPUs just causes unnecessary
and often unrelated changes.
These updated data files are required because libvirt will now try to
use the better matching EPYC-Rome model for Ryzen 9 tests.
So leaving them out, will break tests. Just changing the model to
EPYC-Rome in the Ryzen 9 tests files, will also breaks tests. Hence I
re-ran the gather/parse on a Ryzen 9 system to get the updated cpuid
information.
> Since the EPYC 7502 and the Ryzen 9 are pretty much identical,
the
> latter could be dropped as there is no corresponding qemu model.
While we don't have a special model Ryzen 9, we still want to have it
covered in the test suit to check we assign the appropriate model to it.
Got it.
Markus
1544
days inactive
1546
days old
2 comments
2 participants
tags (0)
participants (2)
-
Jiri Denemark -
Markus Schade