<div dir="ltr">Ping... Can someone review this simple patch ?<div><br></div><div>Thanks,</div><div>Shiva</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Mar 2, 2016 at 8:39 PM, Shivaprasad G Bhat <span dir="ltr"><<a href="mailto:shivaprasadbhat@gmail.com" target="_blank">shivaprasadbhat@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">The default USB controller is not sent to destination as the older versions<br>
of libvirt(0.9.4 or earlier as I see in commit log of 409b5f54) didn't<br>
support them. For some archs where the support started much later can<br>
safely send the USB controllers without this worry. For now, changing the<br>
checks to allow PPC64 here. If we can track the historical support for<br>
other archs, the changes can be extrapolated to them too.<br>
<br>
The patch fixes a bug that, if the USB controller happens to occupy<br>
a slot after disks/interfaces and one of them is hot-unplugged, then<br>
the default USB controller added on destination takes the smallest slot<br>
number and that would lead to savestate mismatch and migration<br>
failure.<br>
<br>
Signed-off-by: Shivaprasad G Bhat <<a href="mailto:sbhat@linux.vnet.ibm.com">sbhat@linux.vnet.ibm.com</a>><br>
---<br>
 src/qemu/qemu_domain.c |    2 +-<br>
 1 file changed, 1 insertion(+), 1 deletion(-)<br>
<br>
diff --git a/src/qemu/qemu_domain.c b/src/qemu/qemu_domain.c<br>
index 9044792..6ecc0e2 100644<br>
--- a/src/qemu/qemu_domain.c<br>
+++ b/src/qemu/qemu_domain.c<br>
@@ -2226,7 +2226,7 @@ qemuDomainDefFormatBuf(virQEMUDriverPtr driver,<br>
                 usb = def->controllers[i];<br>
             }<br>
         }<br>
-        if (usb && usb->idx == 0 && usb->model == -1) {<br>
+        if (!ARCH_IS_PPC64(def->os.arch) && usb && usb->idx == 0 && usb->model == -1) {<br>
             VIR_DEBUG("Removing default USB controller from domain '%s'"<br>
                       " for migration compatibility", def->name);<br>
             toremove++;<br>
<br>
--<br>
libvir-list mailing list<br>
<a href="mailto:libvir-list@redhat.com">libvir-list@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/libvir-list" rel="noreferrer" target="_blank">https://www.redhat.com/mailman/listinfo/libvir-list</a><br>
</blockquote></div><br></div>