[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [BUGFIX][PATCH for 2.2 1/1] hw/i386/pc_piix.c: Also pas
From: |
Eduardo Habkost |
Subject: |
Re: [Qemu-devel] [BUGFIX][PATCH for 2.2 1/1] hw/i386/pc_piix.c: Also pass vmport=off for xenfv machine |
Date: |
Thu, 20 Nov 2014 13:13:46 -0200 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
On Wed, Nov 19, 2014 at 02:08:08PM -0500, Don Slutz wrote:
>
> On 11/19/14 13:08, Paolo Bonzini wrote:
> >
> >On 19/11/2014 19:07, Don Slutz wrote:
> >>>"-M pc -machine accel=xen" should work and, if that's what you want,
> >>>disable the vmport device. I think this patch is wrong.
> >>>
> >>>Paolo
> >>Well, I also want "-M pc -machine accel=xen,vmport=on" to work.
> >Right. So let's start by deciding what the desired semantics are for
> >all six cases: -M pc/xenfv, -machine vmport=on/off/absent.
> >
> >Paolo
>
> I get 12 cases (PCMachineState *pcms = PC_MACHINE(obj)):
We have more cases, if we consider "-M pc-2.1" too.
With this first patch (the one changing default_machine_opts), I expect
to get the following results:
-M pc
pcms->vmport is true
-M pc -machine vmport=on
pcms->vmport is true
-M pc -machine vmport=off
pcms->vmport is false
-M pc-2.1
pcms->vmport is true
-M pc-2.1 -machine vmport=on
pcms->vmport is true
(but it doesn't matter, QEMU 2.1 didn't have the vmport option)
-M pc-2.1 -machine vmport=off
pcms->vmport is false
(but it doesn't matter, QEMU 2.1 didn't have the vmport option)
-M xenfv
pcms->vmport is false
-M xenfv -machine vmport=on
pcms->vmport is true
-M xenfv -machine vmport=off
pcms->vmport is false
-M pc -machine accel=xen
pcms->vmport is true
-M pc -machine vmport=on,accel=xen
pcms->vmport is true
-M pc -machine vmport=off,accel=xen
pcms->vmport is false
-M pc-2.1 -machine accel=xen
pcms->vmport is true **
-M pc-2.1 -machine vmport=on,accel=xen
pcms->vmport is true
(but it doesn't matter, QEMU 2.1 didn't have the vmport option)
-M pc-2.1 -machine vmport=off,accel=xen
pcms->vmport is false
(but it doesn't matter, QEMU 2.1 didn't have the vmport option)
-M xenfv -machine accel=xen
pcms->vmport is false
-M xenfv -machine vmport=on,accel=xen
pcms->vmport is true
-M xenfv -machine vmport=off,accel=xen
pcms->vmport is false
I believe there's no consensus yet about the one marked with "**" above. It
boils to the question: do we need to keep guest ABI stability when using
"-M pc-2.1 -machine accel=xen"?
--
Eduardo
- Re: [Qemu-devel] [BUGFIX][PATCH for 2.2 1/1] hw/i386/pc_piix.c: Also pass vmport=off for xenfv machine, (continued)
- Re: [Qemu-devel] [BUGFIX][PATCH for 2.2 1/1] hw/i386/pc_piix.c: Also pass vmport=off for xenfv machine, Don Slutz, 2014/11/19
- Re: [Qemu-devel] [BUGFIX][PATCH for 2.2 1/1] hw/i386/pc_piix.c: Also pass vmport=off for xenfv machine, Paolo Bonzini, 2014/11/19
- Re: [Qemu-devel] [BUGFIX][PATCH for 2.2 1/1] hw/i386/pc_piix.c: Also pass vmport=off for xenfv machine, Don Slutz, 2014/11/19
- Re: [Qemu-devel] [BUGFIX][PATCH for 2.2 1/1] hw/i386/pc_piix.c: Also pass vmport=off for xenfv machine, Don Slutz, 2014/11/19
- Re: [Qemu-devel] [BUGFIX][PATCH for 2.2 1/1] hw/i386/pc_piix.c: Also pass vmport=off for xenfv machine, Eduardo Habkost, 2014/11/19
- Re: [Qemu-devel] [BUGFIX][PATCH for 2.2 1/1] hw/i386/pc_piix.c: Also pass vmport=off for xenfv machine, Don Slutz, 2014/11/19
- Re: [Qemu-devel] [BUGFIX][PATCH for 2.2 1/1] hw/i386/pc_piix.c: Also pass vmport=off for xenfv machine, Eduardo Habkost, 2014/11/19
- Re: [Qemu-devel] [BUGFIX][PATCH for 2.2 1/1] hw/i386/pc_piix.c: Also pass vmport=off for xenfv machine, Don Slutz, 2014/11/19
- Re: [Qemu-devel] [BUGFIX][PATCH for 2.2 1/1] hw/i386/pc_piix.c: Also pass vmport=off for xenfv machine, Paolo Bonzini, 2014/11/20
- Re: [Qemu-devel] [BUGFIX][PATCH for 2.2 1/1] hw/i386/pc_piix.c: Also pass vmport=off for xenfv machine, Don Slutz, 2014/11/20
- Re: [Qemu-devel] [BUGFIX][PATCH for 2.2 1/1] hw/i386/pc_piix.c: Also pass vmport=off for xenfv machine,
Eduardo Habkost <=
- Re: [Qemu-devel] [BUGFIX][PATCH for 2.2 1/1] hw/i386/pc_piix.c: Also pass vmport=off for xenfv machine, Don Slutz, 2014/11/20
- Re: [Qemu-devel] [BUGFIX][PATCH for 2.2 1/1] hw/i386/pc_piix.c: Also pass vmport=off for xenfv machine, Eduardo Habkost, 2014/11/20