Re: [PATCH v2] usbip: vudc: fix: Clear already_seen flag also for ep0

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Dec 02, 2016 at 08:23:49AM -0700, Shuah Khan wrote:
> On 12/01/2016 11:14 AM, Krzysztof Opasiak wrote:
> > ep_list inside gadget structure doesn't contain ep0.
> > It is stored separately in ep0 field.
> > 
> > This causes an urb hang if gadget driver decides to
> > delay setup handling. On host side this is visible as
> > timeout error when setting configuration.
> > 
> > This bug can be reproduced using for example any gadget
> > with mass storage function.
> > 
> > Fixes: abdb29574322 ("usbip: vudc: Add vudc_transfer")
> > Signed-off-by: Krzysztof Opasiak <k.opasiak@xxxxxxxxxxx>
> 
> Greg,
> 
> Assuming you want to continue in the same mode as you pulling
> in the USB/IP patches with Ack. This patch looks good. Please
> pick this up. If you want me to maintain USB/IP tree and send you
> pull request, I can do that as well. Whatever works for you.
> 
> Acked-by: Shuah Khan <shuahkh@xxxxxxxxxxxxxxx>

A simple "acked-by" like this is all I need, no need to spin up a whole
tree just for a single driver.  I can take it from here.

thanks,

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux