Paolo Bonzini <pbonz...@redhat.com> 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 usb_enabled(bool default_usb) >> { >> return qemu_opt_get_bool(qemu_get_machine_opts(), "usb", >> has_defaults && default_usb); >> } >> >> And spapr.c uses this: >> >> if (usb_enabled(spapr->has_graphics)) { >> pci_create_simple(phb->bus, -1, "pci-ohci"); > > 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 that the PCI slot of pci-ohci does not change due > to some other unrelated reason. So I would rather have the fix in libvirt.
Sure, let me check this. Thanks, Nikunj