Re: [Qemu-devel] [PATCH] spapr: skip adding usb keyboard/mouse in case of -nodefaults

2014-04-04 Thread Eric Blake
[adding libvir-list] On 04/04/2014 05:23 AM, Markus Armbruster wrote: > Paolo Bonzini writes: > >> Il 04/04/2014 12:58, Markus Armbruster ha scritto: >>> >>> Have you considered extending QEMUMachineInitArgs instead of adding this >>> function? > > Did not think of this optio

Re: [Qemu-devel] [PATCH] spapr: skip adding usb keyboard/mouse in case of -nodefaults

2014-04-04 Thread Andreas Färber
Am 04.04.2014 10:28, schrieb Nikunj A Dadhania: > diff --git a/vl.c b/vl.c > index 017f92d..0d6c36c 100644 > --- a/vl.c > +++ b/vl.c > @@ -4348,7 +4348,8 @@ int main(int argc, char **argv, char **envp) > .kernel_filename = kernel_filename, >

Re: [Qemu-devel] [PATCH] spapr: skip adding usb keyboard/mouse in case of -nodefaults

2014-04-04 Thread Paolo Bonzini
Il 04/04/2014 13:40, Nikunj A Dadhania ha scritto: > Sure. However, I'm saying that it's fine for spapr to make -usb mean > "OHCI, and also keyboard & mouse if there is a VGA card in the system". > > If libvirt used "-device pci-ohci" unconditionally, it would fix the bug > *and* it would ensure

Re: [Qemu-devel] [PATCH] spapr: skip adding usb keyboard/mouse in case of -nodefaults

2014-04-04 Thread Nikunj A Dadhania
Paolo Bonzini writes: > Il 04/04/2014 07:28, Nikunj A Dadhania ha scritto: >> >> And -usb is translated to adding "pci-ohci" controller for spapr >>> > >>> > Yeah, but with -nodefaults it's better to use -device directly. >> I think there is special handling for this in vl.c >> >> bool

Re: [Qemu-devel] [PATCH] spapr: skip adding usb keyboard/mouse in case of -nodefaults

2014-04-04 Thread Markus Armbruster
Paolo Bonzini writes: > Il 04/04/2014 12:58, Markus Armbruster ha scritto: >> >> Have you considered extending QEMUMachineInitArgs instead of adding this >> function? >>> > >>> > Did not think of this option earlier. You mean doing something like >>> > this? >> Yes. Looks nicer, d

Re: [Qemu-devel] [PATCH] spapr: skip adding usb keyboard/mouse in case of -nodefaults

2014-04-04 Thread Paolo Bonzini
Il 04/04/2014 07:28, Nikunj A Dadhania ha scritto: >> >> And -usb is translated to adding "pci-ohci" controller for spapr > > Yeah, but with -nodefaults it's better to use -device directly. I think there is special handling for this in vl.c bool usb_enabled(bool default_usb) { return qemu_

Re: [Qemu-devel] [PATCH] spapr: skip adding usb keyboard/mouse in case of -nodefaults

2014-04-04 Thread Paolo Bonzini
Il 04/04/2014 12:58, Markus Armbruster ha scritto: >> >> Have you considered extending QEMUMachineInitArgs instead of adding this >> function? > > Did not think of this option earlier. You mean doing something like > this? Yes. Looks nicer, doesn't it? I still think it's a libvirt bug. Mixi

Re: [Qemu-devel] [PATCH] spapr: skip adding usb keyboard/mouse in case of -nodefaults

2014-04-04 Thread Markus Armbruster
Nikunj A Dadhania writes: > Markus Armbruster writes: >> Nikunj A Dadhania writes: >> >> Have you considered extending QEMUMachineInitArgs instead of adding this >> function? > > Did not think of this option earlier. You mean doing something like > this? Yes. Looks nicer, doesn't it?

Re: [Qemu-devel] [PATCH] spapr: skip adding usb keyboard/mouse in case of -nodefaults

2014-04-04 Thread Nikunj A Dadhania
Markus Armbruster writes: > Nikunj A Dadhania writes: > > Have you considered extending QEMUMachineInitArgs instead of adding this > function? Did not think of this option earlier. You mean doing something like this? diff --git a/hw/ppc/spapr.c b/hw/ppc/spapr.c index 3a13231..936a17f 100644 ---

Re: [Qemu-devel] [PATCH] spapr: skip adding usb keyboard/mouse in case of -nodefaults

2014-04-03 Thread Markus Armbruster
Nikunj A Dadhania writes: > The following commit caused the regression in qemu-system-ppc64 > > 7effdaa3: spapr: Fix return value of vga initialization > d44229c5: Fix vga_interface_type for command line argument '-device VGA' > > Even when -nodefaults was provided, USB Keyboard and Mouse was add

Re: [Qemu-devel] [PATCH] spapr: skip adding usb keyboard/mouse in case of -nodefaults

2014-04-03 Thread Nikunj A Dadhania
Paolo Bonzini writes: > Il 03/04/2014 21:24, Nikunj A Dadhania ha scritto: >>> > Does libvirt use "-nodefaults -machine usb=true"? It should create the >>> > OHCI controller separately instead of using "-machine". >> I see it creating: >> >> -nodefaults -usb -device usb-kbd,id=input0 -device usb

Re: [Qemu-devel] [PATCH] spapr: skip adding usb keyboard/mouse in case of -nodefaults

2014-04-03 Thread Paolo Bonzini
Il 03/04/2014 21:24, Nikunj A Dadhania ha scritto: > Does libvirt use "-nodefaults -machine usb=true"? It should create the > OHCI controller separately instead of using "-machine". I see it creating: -nodefaults -usb -device usb-kbd,id=input0 -device usb-mouse,id=input1 And -usb is translate

Re: [Qemu-devel] [PATCH] spapr: skip adding usb keyboard/mouse in case of -nodefaults

2014-04-03 Thread Nikunj A Dadhania
Paolo Bonzini writes: > Il 03/04/2014 18:56, Nikunj A Dadhania ha scritto: >> The following commit caused the regression in qemu-system-ppc64 >> >> 7effdaa3: spapr: Fix return value of vga initialization >> d44229c5: Fix vga_interface_type for command line argument '-device VGA' >> >> Even when -

Re: [Qemu-devel] [PATCH] spapr: skip adding usb keyboard/mouse in case of -nodefaults

2014-04-03 Thread Eric Blake
On 04/03/2014 12:01 PM, Paolo Bonzini wrote: > Il 03/04/2014 18:56, Nikunj A Dadhania ha scritto: >> The following commit caused the regression in qemu-system-ppc64 >> >> 7effdaa3: spapr: Fix return value of vga initialization >> d44229c5: Fix vga_interface_type for command line argument '-device V

Re: [Qemu-devel] [PATCH] spapr: skip adding usb keyboard/mouse in case of -nodefaults

2014-04-03 Thread Paolo Bonzini
Il 03/04/2014 18:56, Nikunj A Dadhania ha scritto: The following commit caused the regression in qemu-system-ppc64 7effdaa3: spapr: Fix return value of vga initialization d44229c5: Fix vga_interface_type for command line argument '-device VGA' Even when -nodefaults was provided, USB Keyboard an

Re: [Qemu-devel] [PATCH] spapr: skip adding usb keyboard/mouse in case of -nodefaults

2014-04-03 Thread Andreas Färber
Am 03.04.2014 19:06, schrieb Nikunj A Dadhania: > > The following commit caused the regression in qemu-system-ppc64 > > 7effdaa3: spapr: Fix return value of vga initialization > d44229c5: Fix vga_interface_type for command line argument '-device VGA' > > Even when -nodefaults was provided, USB K

[Qemu-devel] [PATCH] spapr: skip adding usb keyboard/mouse in case of -nodefaults

2014-04-03 Thread Nikunj A Dadhania
The following commit caused the regression in qemu-system-ppc64 7effdaa3: spapr: Fix return value of vga initialization d44229c5: Fix vga_interface_type for command line argument '-device VGA' Even when -nodefaults was provided, USB Keyboard and Mouse was added to the machine. This breaks libvirt

Re: [Qemu-devel] [PATCH] spapr: skip adding usb keyboard/mouse in case of -nodefaults

2014-04-03 Thread Nikunj A Dadhania
Andreas Färber writes: > Am 03.04.2014 18:56, schrieb Nikunj A Dadhania: >> The following commit caused the regression in qemu-system-ppc64 >> >> 7effdaa3: spapr: Fix return value of vga initialization >> d44229c5: Fix vga_interface_type for command line argument '-device VGA' >> >> Even when -

Re: [Qemu-devel] [PATCH] spapr: skip adding usb keyboard/mouse in case of -nodefaults

2014-04-03 Thread Nikunj A Dadhania
The following commit caused the regression in qemu-system-ppc64 7effdaa3: spapr: Fix return value of vga initialization d44229c5: Fix vga_interface_type for command line argument '-device VGA' Even when -nodefaults was provided, USB Keyboard and Mouse was added to the machine. This breaks libvir

Re: [Qemu-devel] [PATCH] spapr: skip adding usb keyboard/mouse in case of -nodefaults

2014-04-03 Thread Andreas Färber
Am 03.04.2014 18:56, schrieb Nikunj A Dadhania: > The following commit caused the regression in qemu-system-ppc64 > > 7effdaa3: spapr: Fix return value of vga initialization > d44229c5: Fix vga_interface_type for command line argument '-device VGA' > > Even when -nodefaults was provided, USB Keyb