| Message ID | 1661930031-21908-1-git-send-email-zhongling0719@126.com (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
| Series | uas: add no-uas quirk for Thinkplus and Hiksemi usb-storage | expand |
On Wed, Aug 31, 2022 at 03:13:51PM +0800, zenghongling wrote: > The UAS mode of Thinkplus and Hiksemi is reported to fail to work on > several platforms with the following error message: > > [ 39.702439] xhci_hcd 0000:0c:00.3: ERROR Transfer event for disabled endpoint or incorrect stream ring > [ 39.702442] xhci_hcd 0000:0c:00.3: @000000026c61f810 00000000 00000000 1b000000 05038000 > > And when running iozone will disconnect from the USB controller, then after re-connecting the device > will be offlined and not working at all. Plese wrap your changelog at the properly length here (the warning message is fine.) > > Signed-off-by: zenghongling <zhongling0719@126.com> > --- > drivers/usb/storage/unusual_uas.h | 18 ++++++++++++++++++ > 1 file changed, 18 insertions(+) > > diff --git a/drivers/usb/storage/unusual_uas.h b/drivers/usb/storage/unusual_uas.h > index cdff7dc..bb0d31d 100644 > --- a/drivers/usb/storage/unusual_uas.h > +++ b/drivers/usb/storage/unusual_uas.h > @@ -215,6 +215,24 @@ UNUSUAL_DEV(0x4971, 0x1012, 0x0000, 0x9999, > USB_SC_DEVICE, USB_PR_DEVICE, NULL, > US_FL_IGNORE_UAS), > > +UNUSUAL_DEV(0x17ef, 0x3899, 0x0000, 0x9999, > + "Thinkplus", > + "External HDD", > + USB_SC_DEVICE, USB_PR_DEVICE, NULL, > + US_FL_IGNORE_UAS), > + > +UNUSUAL_DEV(0x090c, 0x2000, 0x0000, 0x9999, > + "Hiksemi", > + "External HDD", > + USB_SC_DEVICE, USB_PR_DEVICE, NULL, > + US_FL_IGNORE_UAS), > + > +UNUSUAL_DEV(0x0bda, 0x9210, 0x0000, 0x9999, > + "Hiksemi", > + "External HDD", > + USB_SC_DEVICE, USB_PR_DEVICE, NULL, > + US_FL_IGNORE_UAS), These need to be in sorted order by vendor and product id in the file. And all of these do not support UAS? How slow does this change then make this device? You are defaulting back to the old-style usb-storage protocol which is very very slow. Why are these devices exposing a UAS device if it does not work at all? thanks, greg k-h
A: http://en.wikipedia.org/wiki/Top_post
Q: Were do I find info about this thing called top-posting?
A: Because it messes up the order in which people normally read text.
Q: Why is top-posting such a bad thing?
A: Top-posting.
Q: What is the most annoying thing in e-mail?
A: No.
Q: Should I include quotations after my reply?
http://daringfireball.net/2007/07/on_top
On Wed, Aug 31, 2022 at 04:47:43PM +0800, zhongling0719 wrote:
> Hi all:
<snip>
Please do not send html email as it is rejected by the mailing lists.
Please fix up your email client as the documentation describes, and
resend.
thanks,
greg k-h
On Wed, Aug 31, 2022 at 03:13:51PM +0800, zenghongling wrote: > The UAS mode of Thinkplus and Hiksemi is reported to fail to work on > several platforms with the following error message: > > [ 39.702439] xhci_hcd 0000:0c:00.3: ERROR Transfer event for disabled endpoint or incorrect stream ring > [ 39.702442] xhci_hcd 0000:0c:00.3: @000000026c61f810 00000000 00000000 1b000000 05038000 > > And when running iozone will disconnect from the USB controller, then after re-connecting the device > will be offlined and not working at all. Are you sure these failures are caused by bugs in the devices and not by bugs in the xHCI controller or its driver? What happens if you plug one of these drives into a computer running Windows or Mac OSX? Do those operating systems avoid using the UAS protocol with the drive? Alan Stern
diff --git a/drivers/usb/storage/unusual_uas.h b/drivers/usb/storage/unusual_uas.h index cdff7dc..bb0d31d 100644 --- a/drivers/usb/storage/unusual_uas.h +++ b/drivers/usb/storage/unusual_uas.h @@ -215,6 +215,24 @@ UNUSUAL_DEV(0x4971, 0x1012, 0x0000, 0x9999, USB_SC_DEVICE, USB_PR_DEVICE, NULL, US_FL_IGNORE_UAS), +UNUSUAL_DEV(0x17ef, 0x3899, 0x0000, 0x9999, + "Thinkplus", + "External HDD", + USB_SC_DEVICE, USB_PR_DEVICE, NULL, + US_FL_IGNORE_UAS), + +UNUSUAL_DEV(0x090c, 0x2000, 0x0000, 0x9999, + "Hiksemi", + "External HDD", + USB_SC_DEVICE, USB_PR_DEVICE, NULL, + US_FL_IGNORE_UAS), + +UNUSUAL_DEV(0x0bda, 0x9210, 0x0000, 0x9999, + "Hiksemi", + "External HDD", + USB_SC_DEVICE, USB_PR_DEVICE, NULL, + US_FL_IGNORE_UAS), + /* Reported-by: Richard Henderson <rth@redhat.com> */ UNUSUAL_DEV(0x4971, 0x8017, 0x0000, 0x9999, "SimpleTech",
The UAS mode of Thinkplus and Hiksemi is reported to fail to work on several platforms with the following error message: [ 39.702439] xhci_hcd 0000:0c:00.3: ERROR Transfer event for disabled endpoint or incorrect stream ring [ 39.702442] xhci_hcd 0000:0c:00.3: @000000026c61f810 00000000 00000000 1b000000 05038000 And when running iozone will disconnect from the USB controller, then after re-connecting the device will be offlined and not working at all. Signed-off-by: zenghongling <zhongling0719@126.com> --- drivers/usb/storage/unusual_uas.h | 18 ++++++++++++++++++ 1 file changed, 18 insertions(+)