[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

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

Attachment: signature.asc
Description: OpenPGP digital signature


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]