Message ID | 1406191971-22404-1-git-send-email-agraf@suse.de |
---|---|
State | New |
Headers | show |
Il 24/07/2014 10:52, Alexander Graf ha scritto: > On PPC we have 2 different styles of KVM: PR and HV. HV can only virtualize > sPAPR guests while PR can virtualize everything that's reasonably close to > the host hardware platform. > > As long as only one kernel module (PR or HV) is loaded, the "default" kvm type > is the module that's loaded. So if your hardware only supports PR mode you can > easily spawn a Mac VM. > > However, if both HV and PR are loaded we default to HV mode. And in that case > the Mac machines have to explicitly ask for PR mode to get a working VM. > > Fix this up by explicitly having the Mac machines ask for PR style KVM. This > fixes bootup of Mac VMs on systems where bot HV and PR kvm modules are loaded > for me. > > Signed-off-by: Alexander Graf <agraf@suse.de> > --- > hw/ppc/mac_newworld.c | 7 +++++++ > hw/ppc/mac_oldworld.c | 7 +++++++ > 2 files changed, 14 insertions(+) > > diff --git a/hw/ppc/mac_newworld.c b/hw/ppc/mac_newworld.c > index f5bccd2..2e66a35 100644 > --- a/hw/ppc/mac_newworld.c > +++ b/hw/ppc/mac_newworld.c > @@ -477,12 +477,19 @@ static void ppc_core99_init(MachineState *machine) > qemu_register_boot_set(fw_cfg_boot_set, fw_cfg); > } > > +static int core99_kvm_type(const char *arg) > +{ > + /* Always force PR KVM */ > + return 2; > +} > + > static QEMUMachine core99_machine = { > .name = "mac99", > .desc = "Mac99 based PowerMAC", > .init = ppc_core99_init, > .max_cpus = MAX_CPUS, > .default_boot_order = "cd", > + .kvm_type = core99_kvm_type, > }; > > static void core99_machine_init(void) > diff --git a/hw/ppc/mac_oldworld.c b/hw/ppc/mac_oldworld.c > index cd9bdbc..ec7ed38 100644 > --- a/hw/ppc/mac_oldworld.c > +++ b/hw/ppc/mac_oldworld.c > @@ -346,6 +346,12 @@ static void ppc_heathrow_init(MachineState *machine) > qemu_register_boot_set(fw_cfg_boot_set, fw_cfg); > } > > +static int heathrow_kvm_type(const char *arg) > +{ > + /* Always force PR KVM */ > + return 2; > +} > + > static QEMUMachine heathrow_machine = { > .name = "g3beige", > .desc = "Heathrow based PowerMAC", > @@ -355,6 +361,7 @@ static QEMUMachine heathrow_machine = { > .is_default = 1, > #endif > .default_boot_order = "cd", /* TOFIX "cad" when Mac floppy is implemented */ > + .kvm_type = heathrow_kvm_type, > }; > > static void heathrow_machine_init(void) > Why should anything except pseries ever use HV KVM? Paolo
On 24.07.14 15:44, Paolo Bonzini wrote: > Il 24/07/2014 10:52, Alexander Graf ha scritto: >> On PPC we have 2 different styles of KVM: PR and HV. HV can only virtualize >> sPAPR guests while PR can virtualize everything that's reasonably close to >> the host hardware platform. >> >> As long as only one kernel module (PR or HV) is loaded, the "default" kvm type >> is the module that's loaded. So if your hardware only supports PR mode you can >> easily spawn a Mac VM. >> >> However, if both HV and PR are loaded we default to HV mode. And in that case >> the Mac machines have to explicitly ask for PR mode to get a working VM. >> >> Fix this up by explicitly having the Mac machines ask for PR style KVM. This >> fixes bootup of Mac VMs on systems where bot HV and PR kvm modules are loaded >> for me. >> >> Signed-off-by: Alexander Graf <agraf@suse.de> >> --- >> hw/ppc/mac_newworld.c | 7 +++++++ >> hw/ppc/mac_oldworld.c | 7 +++++++ >> 2 files changed, 14 insertions(+) >> >> diff --git a/hw/ppc/mac_newworld.c b/hw/ppc/mac_newworld.c >> index f5bccd2..2e66a35 100644 >> --- a/hw/ppc/mac_newworld.c >> +++ b/hw/ppc/mac_newworld.c >> @@ -477,12 +477,19 @@ static void ppc_core99_init(MachineState *machine) >> qemu_register_boot_set(fw_cfg_boot_set, fw_cfg); >> } >> >> +static int core99_kvm_type(const char *arg) >> +{ >> + /* Always force PR KVM */ >> + return 2; >> +} >> + >> static QEMUMachine core99_machine = { >> .name = "mac99", >> .desc = "Mac99 based PowerMAC", >> .init = ppc_core99_init, >> .max_cpus = MAX_CPUS, >> .default_boot_order = "cd", >> + .kvm_type = core99_kvm_type, >> }; >> >> static void core99_machine_init(void) >> diff --git a/hw/ppc/mac_oldworld.c b/hw/ppc/mac_oldworld.c >> index cd9bdbc..ec7ed38 100644 >> --- a/hw/ppc/mac_oldworld.c >> +++ b/hw/ppc/mac_oldworld.c >> @@ -346,6 +346,12 @@ static void ppc_heathrow_init(MachineState *machine) >> qemu_register_boot_set(fw_cfg_boot_set, fw_cfg); >> } >> >> +static int heathrow_kvm_type(const char *arg) >> +{ >> + /* Always force PR KVM */ >> + return 2; >> +} >> + >> static QEMUMachine heathrow_machine = { >> .name = "g3beige", >> .desc = "Heathrow based PowerMAC", >> @@ -355,6 +361,7 @@ static QEMUMachine heathrow_machine = { >> .is_default = 1, >> #endif >> .default_boot_order = "cd", /* TOFIX "cad" when Mac floppy is implemented */ >> + .kvm_type = heathrow_kvm_type, >> }; >> >> static void heathrow_machine_init(void) >> > Why should anything except pseries ever use HV KVM? Because there are no other Book3S machines :). And for BookE we don't implement kvm_type != 0 because we only support either PR or HV depending on the host platform. Alex
Il 24/07/2014 15:54, Alexander Graf ha scritto: >>> >> Why should anything except pseries ever use HV KVM? > > Because there are no other Book3S machines :). And for BookE we don't > implement kvm_type != 0 because we only support either PR or HV > depending on the host platform. Yeah, what I meant was (in a rather Socratic way): why should the default for kvm_type == NULL be anything but PR? All 2.2 stuff, of course. Paolo
On 24.07.14 16:11, Paolo Bonzini wrote: > Il 24/07/2014 15:54, Alexander Graf ha scritto: >>> Why should anything except pseries ever use HV KVM? >> Because there are no other Book3S machines :). And for BookE we don't >> implement kvm_type != 0 because we only support either PR or HV >> depending on the host platform. > Yeah, what I meant was (in a rather Socratic way): why should the > default for kvm_type == NULL be anything but PR? I would prefer to keep the BookE machines on kvm_type = 0. Saying "give me whatever you can" makes a lot more sense there. Alex
Il 24/07/2014 16:13, Alexander Graf ha scritto: > > On 24.07.14 16:11, Paolo Bonzini wrote: >> Il 24/07/2014 15:54, Alexander Graf ha scritto: >>>> Why should anything except pseries ever use HV KVM? >>> Because there are no other Book3S machines :). And for BookE we don't >>> implement kvm_type != 0 because we only support either PR or HV >>> depending on the host platform. >> Yeah, what I meant was (in a rather Socratic way): why should the >> default for kvm_type == NULL be anything but PR? > > I would prefer to keep the BookE machines on kvm_type = 0. Saying "give > me whatever you can" makes a lot more sense there. Got the answer on IRC: (16:18:18) bonzini: agraf: do g3beige and mac99 run on booke PR kvm? (16:33:51) agraf: bonzini: nope - booke only virtualizes booke Paolo
diff --git a/hw/ppc/mac_newworld.c b/hw/ppc/mac_newworld.c index f5bccd2..2e66a35 100644 --- a/hw/ppc/mac_newworld.c +++ b/hw/ppc/mac_newworld.c @@ -477,12 +477,19 @@ static void ppc_core99_init(MachineState *machine) qemu_register_boot_set(fw_cfg_boot_set, fw_cfg); } +static int core99_kvm_type(const char *arg) +{ + /* Always force PR KVM */ + return 2; +} + static QEMUMachine core99_machine = { .name = "mac99", .desc = "Mac99 based PowerMAC", .init = ppc_core99_init, .max_cpus = MAX_CPUS, .default_boot_order = "cd", + .kvm_type = core99_kvm_type, }; static void core99_machine_init(void) diff --git a/hw/ppc/mac_oldworld.c b/hw/ppc/mac_oldworld.c index cd9bdbc..ec7ed38 100644 --- a/hw/ppc/mac_oldworld.c +++ b/hw/ppc/mac_oldworld.c @@ -346,6 +346,12 @@ static void ppc_heathrow_init(MachineState *machine) qemu_register_boot_set(fw_cfg_boot_set, fw_cfg); } +static int heathrow_kvm_type(const char *arg) +{ + /* Always force PR KVM */ + return 2; +} + static QEMUMachine heathrow_machine = { .name = "g3beige", .desc = "Heathrow based PowerMAC", @@ -355,6 +361,7 @@ static QEMUMachine heathrow_machine = { .is_default = 1, #endif .default_boot_order = "cd", /* TOFIX "cad" when Mac floppy is implemented */ + .kvm_type = heathrow_kvm_type, }; static void heathrow_machine_init(void)
On PPC we have 2 different styles of KVM: PR and HV. HV can only virtualize sPAPR guests while PR can virtualize everything that's reasonably close to the host hardware platform. As long as only one kernel module (PR or HV) is loaded, the "default" kvm type is the module that's loaded. So if your hardware only supports PR mode you can easily spawn a Mac VM. However, if both HV and PR are loaded we default to HV mode. And in that case the Mac machines have to explicitly ask for PR mode to get a working VM. Fix this up by explicitly having the Mac machines ask for PR style KVM. This fixes bootup of Mac VMs on systems where bot HV and PR kvm modules are loaded for me. Signed-off-by: Alexander Graf <agraf@suse.de> --- hw/ppc/mac_newworld.c | 7 +++++++ hw/ppc/mac_oldworld.c | 7 +++++++ 2 files changed, 14 insertions(+)