diff mbox series

usb: storage: do not use UAS with Logitec LGB-4BNHUC

Message ID 20211212115506.180629-1-gnurou@gmail.com
State New
Headers show
Series usb: storage: do not use UAS with Logitec LGB-4BNHUC | expand

Commit Message

Alexandre Courbot Dec. 12, 2021, 11:55 a.m. UTC
This 4-drives USB 3.1 bay gets unhappy when used with UAS:

	sd 10:0:0:0: [sdb] tag#18 uas_eh_abort_handler 0 uas-tag 5 inflight: CMD IN
	sd 10:0:0:0: [sdb] tag#18 CDB: Read(16) 88 00 00 00 00 00 00 9d 42 30 00 00 04 00 00 00

It works perfectly fine as a regular mass-storage device, so prevent UAS
from being used with it.

Signed-off-by: Alexandre Courbot <gnurou@gmail.com>
---
 drivers/usb/storage/unusual_uas.h | 6 ++++++
 1 file changed, 6 insertions(+)

Comments

Alexandre Courbot Dec. 15, 2021, 11:41 a.m. UTC | #1
Hi Oliver,

On Tue, Dec 14, 2021 at 4:34 AM Oliver Neukum <oneukum@suse.com> wrote:
>
>
> On 12.12.21 12:55, Alexandre Courbot wrote:
> > This 4-drives USB 3.1 bay gets unhappy when used with UAS:
> >
> >       sd 10:0:0:0: [sdb] tag#18 uas_eh_abort_handler 0 uas-tag 5 inflight: CMD IN
> >       sd 10:0:0:0: [sdb] tag#18 CDB: Read(16) 88 00 00 00 00 00 00 9d 42 30 00 00 04 00 00 00
> >
> > It works perfectly fine as a regular mass-storage device, so prevent UAS
> > from being used with it.
> >
> Hi,
>
> this is a big drastic a fix. s it always that command?
> Have you tried less drastic quirks?

Unfortunately I am not familiar with the storage subsystem so I'm not
sure which quirks would be good candidates to try, would you have
suggestions? As for the command, reproduction is rather random and I
did not keep traces of all instances.

I just noticed that users with similar issues reported that disabling
UAS fixed the problem and sure enough it did in my case as well. The
bay is used with a NAS so I'd rather not mess too much with it, but I
am willing to experiment a bit (after double-checking my backups) if
you have good leads to suggest.

Cheers,
Alex.
Oliver Neukum Dec. 15, 2021, 2:03 p.m. UTC | #2
On 15.12.21 12:41, Alexandre Courbot wrote:
> Unfortunately I am not familiar with the storage subsystem so I'm not
> sure which quirks would be good candidates to try, would you have
> suggestions? As for the command, reproduction is rather random and I
> did not keep traces of all instances.

Hi,


could you try "fgkm" ?

    Regards
        Oliver
Alexandre Courbot Dec. 20, 2021, 3:28 a.m. UTC | #3
On Wed, Dec 15, 2021 at 11:03 PM Oliver Neukum <oneukum@suse.com> wrote:
>
>
> On 15.12.21 12:41, Alexandre Courbot wrote:
> > Unfortunately I am not familiar with the storage subsystem so I'm not
> > sure which quirks would be good candidates to try, would you have
> > suggestions? As for the command, reproduction is rather random and I
> > did not keep traces of all instances.
>
> Hi,
>
>
> could you try "fgkm" ?

Trying with this and will narrow down if it looks stable. Please
expect a few weeks before the (hopefully more refined) patch.

Cheers,
Alex.
diff mbox series

Patch

diff --git a/drivers/usb/storage/unusual_uas.h b/drivers/usb/storage/unusual_uas.h
index b34b858d82a6..9ef09cabef50 100644
--- a/drivers/usb/storage/unusual_uas.h
+++ b/drivers/usb/storage/unusual_uas.h
@@ -52,6 +52,12 @@  UNUSUAL_DEV(0x059f, 0x1061, 0x0000, 0x9999,
 		USB_SC_DEVICE, USB_PR_DEVICE, NULL,
 		US_FL_NO_REPORT_OPCODES | US_FL_NO_SAME),
 
+UNUSUAL_DEV(0x0789, 0x0296, 0x0000, 0x9999,
+		"Logitec",
+		"LGB-4BNHUC",
+		USB_SC_DEVICE, USB_PR_DEVICE, NULL,
+		US_FL_IGNORE_UAS),
+
 /*
  * Apricorn USB3 dongle sometimes returns "USBSUSBSUSBS" in response to SCSI
  * commands in UAS mode.  Observed with the 1.28 firmware; are there others?