Re: [PATCH 1/1] usb: misc: usbtest: add fix for driver hang

2016-08-10 Thread Lu Baolu
Hi, On 08/09/2016 10:18 PM, Alan Stern wrote: > On Tue, 9 Aug 2016, Felipe Balbi wrote: > >> Hi, >> >> Lu Baolu writes: >>> In sg_timeout(), req->status is set to "-ETIMEDOUT" before calling >>> into usb_sg_cancel(). usb_sg_cancel() will do nothing and return >>>

Re: [PATCH 1/1] usb: misc: usbtest: add fix for driver hang

2016-08-10 Thread Lu Baolu
Hi, On 08/09/2016 10:18 PM, Alan Stern wrote: > On Tue, 9 Aug 2016, Felipe Balbi wrote: > >> Hi, >> >> Lu Baolu writes: >>> In sg_timeout(), req->status is set to "-ETIMEDOUT" before calling >>> into usb_sg_cancel(). usb_sg_cancel() will do nothing and return >>> directly if req->status has been

Re: [PATCH 1/1] usb: misc: usbtest: add fix for driver hang

2016-08-09 Thread Alan Stern
On Tue, 9 Aug 2016, Felipe Balbi wrote: > > Hi, > > Lu Baolu writes: > > In sg_timeout(), req->status is set to "-ETIMEDOUT" before calling > > into usb_sg_cancel(). usb_sg_cancel() will do nothing and return > > directly if req->status has been set to a non-zero

Re: [PATCH 1/1] usb: misc: usbtest: add fix for driver hang

2016-08-09 Thread Alan Stern
On Tue, 9 Aug 2016, Felipe Balbi wrote: > > Hi, > > Lu Baolu writes: > > In sg_timeout(), req->status is set to "-ETIMEDOUT" before calling > > into usb_sg_cancel(). usb_sg_cancel() will do nothing and return > > directly if req->status has been set to a non-zero value. This will > > cause

Re: [PATCH 1/1] usb: misc: usbtest: add fix for driver hang

2016-08-09 Thread Felipe Balbi
Hi, Lu Baolu writes: > In sg_timeout(), req->status is set to "-ETIMEDOUT" before calling > into usb_sg_cancel(). usb_sg_cancel() will do nothing and return > directly if req->status has been set to a non-zero value. This will > cause driver hang as soon as transfer

Re: [PATCH 1/1] usb: misc: usbtest: add fix for driver hang

2016-08-09 Thread Felipe Balbi
Hi, Lu Baolu writes: > In sg_timeout(), req->status is set to "-ETIMEDOUT" before calling > into usb_sg_cancel(). usb_sg_cancel() will do nothing and return > directly if req->status has been set to a non-zero value. This will > cause driver hang as soon as transfer time out is triggered. > >

[PATCH 1/1] usb: misc: usbtest: add fix for driver hang

2016-08-08 Thread Lu Baolu
In sg_timeout(), req->status is set to "-ETIMEDOUT" before calling into usb_sg_cancel(). usb_sg_cancel() will do nothing and return directly if req->status has been set to a non-zero value. This will cause driver hang as soon as transfer time out is triggered. In my test case, below system log

[PATCH 1/1] usb: misc: usbtest: add fix for driver hang

2016-08-08 Thread Lu Baolu
In sg_timeout(), req->status is set to "-ETIMEDOUT" before calling into usb_sg_cancel(). usb_sg_cancel() will do nothing and return directly if req->status has been set to a non-zero value. This will cause driver hang as soon as transfer time out is triggered. In my test case, below system log