[libvirt] [PATCH] Set a sensible default master start port for ehci companion controllers

Eric Blake eblake at redhat.com
Mon May 14 21:47:57 UTC 2012


On 05/14/2012 06:24 AM, Daniel P. Berrange wrote:
> From: "Daniel P. Berrange" <berrange at redhat.com>
> 
> The uhci1, uhci2, uhci3 companion controllers for ehci1 must
> have a master start port set. Since this value is predictable
> we should set it automatically if the app does not supply it
> ---
>  src/conf/domain_conf.c                             |   22 ++++++++++++++++++
>  .../qemuxml2argv-usb-ich9-ehci-addr.xml            |   24 +++++---------------
>  2 files changed, 28 insertions(+), 18 deletions(-)

ACK.  But shouldn't we also have a test in qemuxml2xmltest, by using a
file in qemuxml2xmloutdata/, that shows that we generate the correct
start port on output in response to a user omitting information that we
can determine by default?

-- 
Eric Blake   eblake at redhat.com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 620 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20120514/6096afb2/attachment-0001.sig>


More information about the libvir-list mailing list