On 11/25/22 11:52, Thomas Huth wrote: > When running "virsh domcapabilities" on a s390x host, all the CPU > models show up with vendor='unknown' - which sounds kind of weird > since the vendor of these mainframe CPUs is well known: IBM. > All CPUs starting with either "z" or "gen" match a real mainframe > CPU by IBM, so let's return the string "IBM" for those now. > The only remaining ones are now the artifical "qemu" and "max" > models from QEMU itself, so it should be OK to get an "unknown" > vendor for those two. > > Reviewed-by: Jiri Denemark <jdenemar@xxxxxxxxxx> > Signed-off-by: Boris Fiuczynski<fiuczy@xxxxxxxxxxxxx> > Signed-off-by: Thomas Huth <thuth@xxxxxxxxxx> > --- > v3: Use STRPREFIX() for the checks (thanks to Jiri for the hint!) > > src/cpu/cpu_s390.c | 11 ++ > tests/domaincapsdata/qemu_4.2.0.s390x.xml | 144 +++++++++++----------- > tests/domaincapsdata/qemu_5.2.0.s390x.xml | 144 +++++++++++----------- > tests/domaincapsdata/qemu_6.0.0.s390x.xml | 144 +++++++++++----------- > 4 files changed, 227 insertions(+), 216 deletions(-) > Oops. Sorry for the delay. Pushed now. Michal
