Re: [PATCH 2/6] Use -machine option for PPC64

[Li Zhang <zhlcindy@xxxxxxxxx>]

On Fri, Dec 21, 2012 at 10:21 AM, Li Zhang <zhlcindy@xxxxxxxxx> wrote:

On Thu, Dec 20, 2012 at 6:28 PM, Daniel P. Berrange <berrange@xxxxxxxxxx> wrote:

On Thu, Dec 20, 2012 at 02:02:23PM +0800, Li Zhang wrote:
> From: Li Zhang <zhlcindy@xxxxxxxxxxxxxxxxxx>
>
> On PPC64 platform, it sets USB option in -machine option. It is
> set as the following:
>   -machine pseries,usb=off
>
> This will disable USB, which avoids creating one USB controller
> implicitly in the platform's initialization. Libvirt will add one
> USB controller if there is no definition in XML file. Users also can
> add one specific USB controller in the XML file as the following:
>
>     <controller type='usb' index='0' model='pci-ohci'>
>     </controller>

NACK, current behaviour matches libvirt on x86, where we also
add a USB controller by default.

Yes, I saw the code in Libvirt which adds a USB controller by default.

My problem is that:

In  QEMU for pseries, default VGA is enabled, one USB controller is created and USB keyboard and mouse are created.

If libvirt also creates USB controller with -device xxxx, it will produce the error:

qemu-system-ppc64: -device pci-ohci,id=usb,bus=pci,addr=0x1.0x2: PCI: single function device can't be populated in function 1.2
qemu-system-ppc64: -device pci-ohci,id=usb,bus=pci,addr=0x1.0x2: Device 'pci-ohci' could not be initialized
2012-12-20 15:16:54.985+0000: shutting down

To avoid the conflict, we need to use -machine name,usb=off to disable USB option.

My patch doesn't create the default USB controller, it only help disable USB option.

Hi Daniel,

I think this is one bug for USB option.
USB option in machine options can't be used with -device xxxx  at the same time

Actually, some platforms would like to enable USB option by default.

So we need to pass usb=off, then use -device xxx to create a USB controller in libvirt.

We discussed this problem in QEMU mailing list before.
Here is the link,
http://lists.nongnu.org/archive/html/qemu-devel/2012-06/msg01008.html

Any idea?

Thanks. :)

 

Thanks. :)

 

The mechanism for disabling the default USB controlle ris
to use

     <controller type='usb' index='0' model='none'>

PPC should follow this same behaviour


Daniel
--
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|

--

Best Regards

-Li

--

Best Regards

-Li

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list