diff mbox series

[1/3] Revert "usbredir: avoid queuing hello packet on snapshot restore"

Message ID 20220813011031.3744-2-j@getutm.app (mailing list archive)
State New, archived
Headers show
Series Set runstate to RUN_STATE_RESTORE_VM when started with "-loadvm" | expand

Commit Message

Joelle van Dyne Aug. 13, 2022, 1:10 a.m. UTC
Run state is also in RUN_STATE_PRELAUNCH while "-S" is used.

This reverts commit 12d182898a4866e4be418e2abac286b497cfa1b2.

Signed-off-by: Joelle van Dyne <j@getutm.app>
---
 hw/usb/redirect.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

Comments

Ján Tomko Nov. 21, 2022, 12:26 p.m. UTC | #1
On a Friday in 2022, Joelle van Dyne wrote:
>Run state is also in RUN_STATE_PRELAUNCH while "-S" is used.
>
>This reverts commit 12d182898a4866e4be418e2abac286b497cfa1b2.
>
>Signed-off-by: Joelle van Dyne <j@getutm.app>
>---
> hw/usb/redirect.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)


Reviewed-by: Ján Tomko <jtomko@redhat.com>

This fixes usb redirect on VM startup for VMs started by libvirt, which
uses -S:
https://bugzilla.redhat.com/show_bug.cgi?id=2144436

Jano
diff mbox series

Patch

diff --git a/hw/usb/redirect.c b/hw/usb/redirect.c
index 1bd30efc3e..fd7df599bc 100644
--- a/hw/usb/redirect.c
+++ b/hw/usb/redirect.c
@@ -1280,8 +1280,7 @@  static void usbredir_create_parser(USBRedirDevice *dev)
     }
 #endif
 
-    if (runstate_check(RUN_STATE_INMIGRATE) ||
-        runstate_check(RUN_STATE_PRELAUNCH)) {
+    if (runstate_check(RUN_STATE_INMIGRATE)) {
         flags |= usbredirparser_fl_no_hello;
     }
     usbredirparser_init(dev->parser, VERSION, caps, USB_REDIR_CAPS_SIZE,