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
