Re: [libvirt] [PATCH 0/2] Get rid of default USB controller on S390
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
Attachments:
- signature.asc (application/pgp-signature — 621 bytes)