On 04/29/2013 10:11 AM, Viktor Mihajlovski wrote: > QEMU for S390 has presently no USB controllers nor a PCI bus. > Both are created by the default in the libvirt domain definition. > Although this can be suppressed by explicitly specifying > an USB controller with model='none', it's a real bad default. > > With Jan's recent patch we can reliably suppress the USB controller > (including it's bogus PCI address) if the source XML has no > usb controller element. Explicitly set addresses are still honored. The change in defaults only affects S390, so I think it is low enough risk to apply even after freeze. Changing any default interpretation of XML is risky, but in the case of S390, overall libvirt support is relatively new, so getting it right as soon as possible still makes sense, rather than burdening ourselves with bad defaults forever. Series ACK and will push soon. > > Viktor Mihajlovski (2): > S390: Mention changed USB behavior > S390: Do not generate a default USB controller > > docs/formatdomain.html.in | 5 +++-- > src/qemu/qemu_domain.c | 8 ++++++++ > .../qemuxml2argv-console-sclp.args | 2 +- > .../qemuxml2argv-console-virtio-ccw.args | 2 +- > .../qemuxml2argv-console-virtio-s390.args | 2 +- > .../qemuxml2argv-disk-virtio-ccw-many.args | 2 +- > .../qemuxml2argv-disk-virtio-ccw.args | 2 +- > .../qemuxml2argv-disk-virtio-s390.args | 2 +- > .../qemuxml2argv-disk-virtio-scsi-ccw.args | 2 +- > .../qemuxml2argv-net-virtio-ccw.args | 2 +- > .../qemuxml2argv-net-virtio-s390.args | 2 +- > .../qemuxml2argv-virtio-rng-ccw.args | 2 +- > 12 files changed, 21 insertions(+), 12 deletions(-) > -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org
Attachment:
signature.asc
Description: OpenPGP digital signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list