Re: [Qemu-devel] [PATCH 1/1] redirect: change the state after attach usb 3.0 devices to xhci

[P J P <ppandit@redhat.com>]

  Hello Qixiong,

+-- On Fri, 29 Jan 2016, Qixiong Su wrote --+
| QEMU crashes when redirect a usb 3.0 device which attached to a xHCI 
| controller. The assertion of "dev->state == 3" in "usb_handle_packet" has 
| failed.

  You need to follow git commit message syntax. ie. One line commit summary, 
leave second line blank and then commit log message stating why you are doing 
the proposed change.

Please see:
  -> http://qemu-project.org/Contribute/SubmitAPatch

 
| diff --git a/hw/usb/redirect.c b/hw/usb/redirect.c
| index 44522d9..cd29776 100644
| --- a/hw/usb/redirect.c
| +++ b/hw/usb/redirect.c
| +static void usbredir_handle_attach(USBDevice *udev)
| +{
| +    assert(udev != NULL);
| +    assert(udev->attached);
| +    assert(udev->state == USB_STATE_ATTACHED);

   This is done in usb_attach() routine, may not be required here.

| +    if (udev->speed == USB_SPEED_SUPER && (udev->port->speedmask & USB_SPEED_MASK_SUPER)) {
| +        udev->state = USB_STATE_DEFAULT;

   Why conditionally set it to default?

--
 - P J P
47AF CE69 3A90 54AA 9045 1053 DD13 3D32 FE5B 041F