|
From: | Mark Wu |
Subject: | Re: [Qemu-ppc] [PATCH v2 2/2] Fix return value of vga initlization on ppc |
Date: | Mon, 10 Mar 2014 22:39:48 +0800 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.2.0 |
On 03/07/2014 05:43 PM, Paolo Bonzini wrote:
Yes, it changes the semantics of 'nodefaults'. I will fix it in v3 and let the management app add usb controller.Il 07/03/2014 10:37, Mark Wu ha scritto:Before spapr_vga_init will returned false if the vga is specified by the command '-device VGA' because vga_interface_type was evaluated to VGA_NONE. With the change in previous patch of this series, spapr_vga_init should return true if it's told that the vga will be initialized in flow of the generic devices initialization. This patch also makes two cleanups: 1. skip initialization for VGA_NONE 2. remove the useless 'break'I think that after this patch, "-nodefaults -device VGA" will get a USB controller that it didn't get before. Perhaps this in vl.c: bool usb_enabled(bool default_usb) { return qemu_opt_get_bool(qemu_get_machine_opts(), "usb", default_usb); } should be bool usb_enabled(bool default_usb) { return qemu_opt_get_bool(qemu_get_machine_opts(), "usb", !no_defaults && default_usb); }
? Thanks, PaoloSigned-off-by: Mark Wu <address@hidden> --- hw/ppc/spapr.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/hw/ppc/spapr.c b/hw/ppc/spapr.c index 93d02c1..4d0ac56 100644 --- a/hw/ppc/spapr.c +++ b/hw/ppc/spapr.c @@ -765,13 +765,15 @@ static int spapr_vga_init(PCIBus *pci_bus) { switch (vga_interface_type) { case VGA_NONE: + return false; + case VGA_DEVICE: + return true; case VGA_STD: return pci_vga_init(pci_bus) != NULL; default: fprintf(stderr, "This vga model is not supported," "currently it only supports -vga std\n"); exit(0); - break; } }
[Prev in Thread] | Current Thread | [Next in Thread] |