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

2016-08-10 Thread Lu Baolu
Hi, On 08/10/2016 10:16 PM, Alan Stern wrote: > On Wed, 10 Aug 2016, Lu Baolu wrote: > >> 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

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

2016-08-10 Thread Lu Baolu
Hi, On 08/10/2016 10:16 PM, Alan Stern wrote: > On Wed, 10 Aug 2016, Lu Baolu wrote: > >> 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

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

2016-08-10 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 whenever transfer time out is triggered. This patch fixes this issue. It could

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

2016-08-10 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 whenever transfer time out is triggered. This patch fixes this issue. It could

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

2016-08-10 Thread Alan Stern
On Wed, 10 Aug 2016, Lu Baolu wrote: > 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 whenever transfer time out is

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

2016-08-10 Thread Alan Stern
On Wed, 10 Aug 2016, Lu Baolu wrote: > 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 whenever transfer time out is