On Tue, 20 Jun 2023, Cédric Le Goater wrote:
> The 'pegasos2' machine never supported KVM. This piece of code was
> inherited from another model.
Why? These machines are supposed to run with KVM they were just not tested
yet. I think pegasos2 should work on a G4 host with KVM-PR but you need to
use the firmware rom for -bios now because VOF uses sc 1 which will not
work with KVM on G4, it's only supported with TCG. (I plan to solve this
eventually when needed by replacing that with another hypercall method
such as OSI used by MOL but that would need two vof binaries or carry
some unneeded code if using single binary.)
So please drop this patch.
Regards,
BALATON Zoltan
> Cc: BALATON Zoltan <balaton@eik.bme.hu>
> Signed-off-by: Cédric Le Goater <clg@kaod.org>
> ---
> hw/ppc/pegasos2.c | 7 ++++++-
> 1 file changed, 6 insertions(+), 1 deletion(-)
>
> diff --git a/hw/ppc/pegasos2.c b/hw/ppc/pegasos2.c
> index af5489de26ed..830323cc7849 100644
> --- a/hw/ppc/pegasos2.c
> +++ b/hw/ppc/pegasos2.c
> @@ -29,7 +29,6 @@
> #include "qemu/log.h"
> #include "qemu/error-report.h"
> #include "sysemu/kvm.h"
> -#include "kvm_ppc.h"
> #include "exec/address-spaces.h"
> #include "qom/qom-qobject.h"
> #include "qapi/qmp/qdict.h"
> @@ -120,6 +119,12 @@ static void pegasos2_init(MachineState *machine)
> int i, sz;
> uint8_t *spd_data;
>
> + if (kvm_enabled()) {
> + error_report("machine %s does not support the KVM accelerator",
> + MACHINE_GET_CLASS(machine)->name);
> + exit(EXIT_FAILURE);
> + }
> +
> /* init CPU */
> pm->cpu = POWERPC_CPU(cpu_create(machine->cpu_type));
> env = &pm->cpu->env;
>