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

2016-12-01 Thread Krzysztof Opasiak 

Hi,

On 12/01/2016 06:26 PM, Sergei Shtylyov wrote:
> Hello!
> 
> On 12/01/2016 07:20 PM, 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: abdb2957 ("usbip: vudc: Add vudc_transfer")
> 
>At least 12 digits please, the format is standardized.

You are right. I forgot about this and just used a short-id. I will post
a fix in a minute.

Thank you.
-- 
Krzysztof Opasiak
Samsung R Institute Poland
Samsung Electronics
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

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

2016-12-01 Thread Sergei Shtylyov 

Hello!

On 12/01/2016 07:20 PM, 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: abdb2957 ("usbip: vudc: Add vudc_transfer")


   At least 12 digits please, the format is standardized.


Signed-off-by: Krzysztof Opasiak 

[...]

MBR, Sergei

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