* [PATCH v6 2/3] usb-storage: Add Hiksemi USB3-FW to IGNORE_UAS
@ 2022-09-15 2:49 Hongling Zeng
2022-09-15 19:32 ` Alan Stern
2022-10-31 6:17 ` Juhyung Park
0 siblings, 2 replies; 12+ messages in thread
From: Hongling Zeng @ 2022-09-15 2:49 UTC (permalink / raw)
To: stern, gregkh; +Cc: linux-usb, usb-storage, zhongling0719, Hongling Zeng
The UAS mode of Hiksemi USB_HDD is reported to fail to work on several
platforms with the following error message, then after re-connecting the
device will be offlined and not working at all.
[ 592.518442][ 2] sd 8:0:0:0: [sda] tag#17 uas_eh_abort_handler 0 uas-tag 18
inflight: CMD
[ 592.527575][ 2] sd 8:0:0:0: [sda] tag#17 CDB: Write(10) 2a 00 03 6f 88 00 00
04 00 00
[ 592.536330][ 2] sd 8:0:0:0: [sda] tag#0 uas_eh_abort_handler 0 uas-tag 1
inflight: CMD
[ 592.545266][ 2] sd 8:0:0:0: [sda] tag#0 CDB: Write(10) 2a 00 07 44 1a 88 00
00 08 00
These disks have a broken uas implementation, the tag field of the status
iu-s is not set properly,so we need to fall-back to usb-storage.
Signed-off-by: Hongling Zeng <zenghongling@kylinos.cn>
---
Change for v1
- Change the email real name and the code worng place.
Change for v2
-Change spelling error.
Change for v3
-Add acked-by
Change for v4
-Fix version error
Change for v5
-change version
Change for v6
-Change the git message for patch 3
---
drivers/usb/storage/unusual_uas.h | 7 +++++++
1 file changed, 7 insertions(+)
diff --git a/drivers/usb/storage/unusual_uas.h b/drivers/usb/storage/unusual_uas.h
index a6bf87a..8a18d58 100644
--- a/drivers/usb/storage/unusual_uas.h
+++ b/drivers/usb/storage/unusual_uas.h
@@ -149,6 +149,13 @@ UNUSUAL_DEV(0x0bc2, 0xab2a, 0x0000, 0x9999,
USB_SC_DEVICE, USB_PR_DEVICE, NULL,
US_FL_NO_ATA_1X),
+/* Reported-by: Hongling Zeng <zenghongling@kylinos.cn> */
+UNUSUAL_DEV(0x0bda, 0x9210, 0x0000, 0x9999,
+ "Hiksemi",
+ "External HDD",
+ USB_SC_DEVICE, USB_PR_DEVICE, NULL,
+ US_FL_IGNORE_UAS),
+
/* Reported-by: Benjamin Tissoires <benjamin.tissoires@redhat.com> */
UNUSUAL_DEV(0x13fd, 0x3940, 0x0000, 0x9999,
"Initio Corporation",
--
2.1.0
^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [PATCH v6 2/3] usb-storage: Add Hiksemi USB3-FW to IGNORE_UAS
2022-09-15 2:49 [PATCH v6 2/3] usb-storage: Add Hiksemi USB3-FW to IGNORE_UAS Hongling Zeng
@ 2022-09-15 19:32 ` Alan Stern
[not found] ` <2fcdca5a-2deb-66ca-fd1d-c3ba719b19af@126.com>
2022-10-31 6:17 ` Juhyung Park
1 sibling, 1 reply; 12+ messages in thread
From: Alan Stern @ 2022-09-15 19:32 UTC (permalink / raw)
To: Hongling Zeng; +Cc: gregkh, linux-usb, usb-storage, zhongling0719
On Thu, Sep 15, 2022 at 10:49:48AM +0800, Hongling Zeng wrote:
> The UAS mode of Hiksemi USB_HDD is reported to fail to work on several
> platforms with the following error message, then after re-connecting the
> device will be offlined and not working at all.
>
> [ 592.518442][ 2] sd 8:0:0:0: [sda] tag#17 uas_eh_abort_handler 0 uas-tag 18
> inflight: CMD
> [ 592.527575][ 2] sd 8:0:0:0: [sda] tag#17 CDB: Write(10) 2a 00 03 6f 88 00 00
> 04 00 00
> [ 592.536330][ 2] sd 8:0:0:0: [sda] tag#0 uas_eh_abort_handler 0 uas-tag 1
> inflight: CMD
> [ 592.545266][ 2] sd 8:0:0:0: [sda] tag#0 CDB: Write(10) 2a 00 07 44 1a 88 00
> 00 08 00
>
> These disks have a broken uas implementation, the tag field of the status
> iu-s is not set properly,so we need to fall-back to usb-storage.
>
> Signed-off-by: Hongling Zeng <zenghongling@kylinos.cn>
> ---
> Change for v1
> - Change the email real name and the code worng place.
>
> Change for v2
> -Change spelling error.
>
> Change for v3
> -Add acked-by
>
> Change for v4
> -Fix version error
>
> Change for v5
> -change version
>
> Change for v6
> -Change the git message for patch 3
> ---
I already sent you an Acked-by: in v3 of this patch, and the patch
hasn't changed significantly since then so you can keep the Acked-by: in
this version.
Alan Stern
> drivers/usb/storage/unusual_uas.h | 7 +++++++
> 1 file changed, 7 insertions(+)
>
> diff --git a/drivers/usb/storage/unusual_uas.h b/drivers/usb/storage/unusual_uas.h
> index a6bf87a..8a18d58 100644
> --- a/drivers/usb/storage/unusual_uas.h
> +++ b/drivers/usb/storage/unusual_uas.h
> @@ -149,6 +149,13 @@ UNUSUAL_DEV(0x0bc2, 0xab2a, 0x0000, 0x9999,
> USB_SC_DEVICE, USB_PR_DEVICE, NULL,
> US_FL_NO_ATA_1X),
>
> +/* Reported-by: Hongling Zeng <zenghongling@kylinos.cn> */
> +UNUSUAL_DEV(0x0bda, 0x9210, 0x0000, 0x9999,
> + "Hiksemi",
> + "External HDD",
> + USB_SC_DEVICE, USB_PR_DEVICE, NULL,
> + US_FL_IGNORE_UAS),
> +
> /* Reported-by: Benjamin Tissoires <benjamin.tissoires@redhat.com> */
> UNUSUAL_DEV(0x13fd, 0x3940, 0x0000, 0x9999,
> "Initio Corporation",
> --
> 2.1.0
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH v6 2/3] usb-storage: Add Hiksemi USB3-FW to IGNORE_UAS
2022-09-15 2:49 [PATCH v6 2/3] usb-storage: Add Hiksemi USB3-FW to IGNORE_UAS Hongling Zeng
2022-09-15 19:32 ` Alan Stern
@ 2022-10-31 6:17 ` Juhyung Park
2022-11-09 10:40 ` Greg KH
1 sibling, 1 reply; 12+ messages in thread
From: Juhyung Park @ 2022-10-31 6:17 UTC (permalink / raw)
To: Hongling Zeng, stern; +Cc: linux-usb, usb-storage, zhongling0719, gregkh
Hi,
I'm speaking from my own experience but RTL9210 is arguably the most
reliable NVMe-to-USB converter available.
Compared to solutions from JMicron (multiple revisions) and ASMedia,
RTL9210 gave the lowest power consumption (from implementing proper
power management commands) and the least headache.
I personally own multiple enclosures and not one gave a single UAS error
from multiple platforms (Android, Intel, AMD) for years, but with this
commit now, it effectively disables UAS for all RTL9210 enclosures.
Would it be possible to apply this quirk only to a specific firmware
(range)? RTL9210 have a lot of possible firmware combinations:
https://www.station-drivers.com/index.php/en/component/remository/Drivers/Realtek/NVMe-USB-3.1/lang,en-gb/
RTL9210 was available since 2019 and the fact that this quirk came up
this late leads me to believe that this is not a widespread issue and
it'll be a shame if all of RTL9210s are blacklisted from UAS with Linux
from now on :(
If any additional information is required, please let me know.
Thanks,
On 9/15/22 11:49, Hongling Zeng wrote:
> The UAS mode of Hiksemi USB_HDD is reported to fail to work on several
> platforms with the following error message, then after re-connecting the
> device will be offlined and not working at all.
>
> [ 592.518442][ 2] sd 8:0:0:0: [sda] tag#17 uas_eh_abort_handler 0 uas-tag 18
> inflight: CMD
> [ 592.527575][ 2] sd 8:0:0:0: [sda] tag#17 CDB: Write(10) 2a 00 03 6f 88 00 00
> 04 00 00
> [ 592.536330][ 2] sd 8:0:0:0: [sda] tag#0 uas_eh_abort_handler 0 uas-tag 1
> inflight: CMD
> [ 592.545266][ 2] sd 8:0:0:0: [sda] tag#0 CDB: Write(10) 2a 00 07 44 1a 88 00
> 00 08 00
>
> These disks have a broken uas implementation, the tag field of the status
> iu-s is not set properly,so we need to fall-back to usb-storage.
>
> Signed-off-by: Hongling Zeng <zenghongling@kylinos.cn>
> ---
> Change for v1
> - Change the email real name and the code worng place.
>
> Change for v2
> -Change spelling error.
>
> Change for v3
> -Add acked-by
>
> Change for v4
> -Fix version error
>
> Change for v5
> -change version
>
> Change for v6
> -Change the git message for patch 3
> ---
> drivers/usb/storage/unusual_uas.h | 7 +++++++
> 1 file changed, 7 insertions(+)
>
> diff --git a/drivers/usb/storage/unusual_uas.h b/drivers/usb/storage/unusual_uas.h
> index a6bf87a..8a18d58 100644
> --- a/drivers/usb/storage/unusual_uas.h
> +++ b/drivers/usb/storage/unusual_uas.h
> @@ -149,6 +149,13 @@ UNUSUAL_DEV(0x0bc2, 0xab2a, 0x0000, 0x9999,
> USB_SC_DEVICE, USB_PR_DEVICE, NULL,
> US_FL_NO_ATA_1X),
>
> +/* Reported-by: Hongling Zeng <zenghongling@kylinos.cn> */
> +UNUSUAL_DEV(0x0bda, 0x9210, 0x0000, 0x9999,
> + "Hiksemi",
> + "External HDD",
> + USB_SC_DEVICE, USB_PR_DEVICE, NULL,
> + US_FL_IGNORE_UAS),
> +
> /* Reported-by: Benjamin Tissoires <benjamin.tissoires@redhat.com> */
> UNUSUAL_DEV(0x13fd, 0x3940, 0x0000, 0x9999,
> "Initio Corporation",
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH v6 2/3] usb-storage: Add Hiksemi USB3-FW to IGNORE_UAS
2022-10-31 6:17 ` Juhyung Park
@ 2022-11-09 10:40 ` Greg KH
2022-11-12 7:45 ` zhongling0719
0 siblings, 1 reply; 12+ messages in thread
From: Greg KH @ 2022-11-09 10:40 UTC (permalink / raw)
To: Juhyung Park; +Cc: Hongling Zeng, stern, linux-usb, usb-storage, zhongling0719
On Mon, Oct 31, 2022 at 03:17:48PM +0900, Juhyung Park wrote:
> Hi,
>
> I'm speaking from my own experience but RTL9210 is arguably the most
> reliable NVMe-to-USB converter available.
>
> Compared to solutions from JMicron (multiple revisions) and ASMedia, RTL9210
> gave the lowest power consumption (from implementing proper power management
> commands) and the least headache.
>
> I personally own multiple enclosures and not one gave a single UAS error
> from multiple platforms (Android, Intel, AMD) for years, but with this
> commit now, it effectively disables UAS for all RTL9210 enclosures.
>
> Would it be possible to apply this quirk only to a specific firmware
> (range)? RTL9210 have a lot of possible firmware combinations: https://www.station-drivers.com/index.php/en/component/remository/Drivers/Realtek/NVMe-USB-3.1/lang,en-gb/
>
> RTL9210 was available since 2019 and the fact that this quirk came up this
> late leads me to believe that this is not a widespread issue and it'll be a
> shame if all of RTL9210s are blacklisted from UAS with Linux from now on :(
>
> If any additional information is required, please let me know.
Can you send a revert of this commit so that we can fix this up?
thanks,
greg k-h
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: Re: [PATCH v6 2/3] usb-storage: Add Hiksemi USB3-FW to IGNORE_UAS
2022-11-09 10:40 ` Greg KH
@ 2022-11-12 7:45 ` zhongling0719
2022-11-12 7:56 ` Greg KH
2023-01-09 11:58 ` Juhyung Park
0 siblings, 2 replies; 12+ messages in thread
From: zhongling0719 @ 2022-11-12 7:45 UTC (permalink / raw)
To: Greg KH, Juhyung Park; +Cc: stern, linux-usb, usb-storage
Hi:
This error not happend on all RTL9210,The uas blacklist only for on sale device(0x0bda, 0x9210).
Thanks!
zhongling0719@126.com
From: Greg KH
Date: 2022-11-09 18:40
To: Juhyung Park
CC: Hongling Zeng; stern; linux-usb; usb-storage; zhongling0719
Subject: Re: [PATCH v6 2/3] usb-storage: Add Hiksemi USB3-FW to IGNORE_UAS
On Mon, Oct 31, 2022 at 03:17:48PM +0900, Juhyung Park wrote:
> Hi,
>
> I'm speaking from my own experience but RTL9210 is arguably the most
> reliable NVMe-to-USB converter available.
>
> Compared to solutions from JMicron (multiple revisions) and ASMedia, RTL9210
> gave the lowest power consumption (from implementing proper power management
> commands) and the least headache.
>
> I personally own multiple enclosures and not one gave a single UAS error
> from multiple platforms (Android, Intel, AMD) for years, but with this
> commit now, it effectively disables UAS for all RTL9210 enclosures.
>
> Would it be possible to apply this quirk only to a specific firmware
> (range)? RTL9210 have a lot of possible firmware combinations: https://www.station-drivers.com/index.php/en/component/remository/Drivers/Realtek/NVMe-USB-3.1/lang,en-gb/
>
> RTL9210 was available since 2019 and the fact that this quirk came up this
> late leads me to believe that this is not a widespread issue and it'll be a
> shame if all of RTL9210s are blacklisted from UAS with Linux from now on :(
>
> If any additional information is required, please let me know.
Can you send a revert of this commit so that we can fix this up?
thanks,
greg k-h
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: Re: [PATCH v6 2/3] usb-storage: Add Hiksemi USB3-FW to IGNORE_UAS
2022-11-12 7:45 ` zhongling0719
@ 2022-11-12 7:56 ` Greg KH
2023-01-09 12:00 ` Juhyung Park
2023-01-09 11:58 ` Juhyung Park
1 sibling, 1 reply; 12+ messages in thread
From: Greg KH @ 2022-11-12 7:56 UTC (permalink / raw)
To: zhongling0719; +Cc: Juhyung Park, stern, linux-usb, usb-storage
On Sat, Nov 12, 2022 at 03:45:36PM +0800, zhongling0719@126.com wrote:
> Hi:
> This error not happend on all RTL9210,The uas blacklist only for on sale device(0x0bda, 0x9210).
What about using the version of the device as the only one to blacklist?
Any more specific information you have about that device?
thanks,
greg k-h
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: Re: [PATCH v6 2/3] usb-storage: Add Hiksemi USB3-FW to IGNORE_UAS
2022-11-12 7:56 ` Greg KH
@ 2023-01-09 12:00 ` Juhyung Park
0 siblings, 0 replies; 12+ messages in thread
From: Juhyung Park @ 2023-01-09 12:00 UTC (permalink / raw)
To: Greg KH; +Cc: zhongling0719, stern, linux-usb, usb-storage
Hi Greg,
I've posted a revert commit as there were no attempts to fix this to
apply just to the said product (Hiksemi USB3-FW):
https://lore.kernel.org/all/20230109115550.71688-1-qkrwngud825@gmail.com/T/#u
Thanks. Regards
On Sat, Nov 12, 2022 at 4:56 PM Greg KH <gregkh@linuxfoundation.org> wrote:
>
> On Sat, Nov 12, 2022 at 03:45:36PM +0800, zhongling0719@126.com wrote:
> > Hi:
> > This error not happend on all RTL9210,The uas blacklist only for on sale device(0x0bda, 0x9210).
>
> What about using the version of the device as the only one to blacklist?
> Any more specific information you have about that device?
>
> thanks,
>
> greg k-h
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: Re: [PATCH v6 2/3] usb-storage: Add Hiksemi USB3-FW to IGNORE_UAS
2022-11-12 7:45 ` zhongling0719
2022-11-12 7:56 ` Greg KH
@ 2023-01-09 11:58 ` Juhyung Park
1 sibling, 0 replies; 12+ messages in thread
From: Juhyung Park @ 2023-01-09 11:58 UTC (permalink / raw)
To: zhongling0719; +Cc: Greg KH, stern, linux-usb, usb-storage
On Sat, Nov 12, 2022 at 4:46 PM zhongling0719@126.com
<zhongling0719@126.com> wrote:
>
> Hi:
> This error not happend on all RTL9210,The uas blacklist only for on sale device(0x0bda, 0x9210).
0x0bda is VendorId for Realtek,
0x9210 is ProductId for the entire RTL9210 enclosures.
This patch blacklists UAS for all RTL9210 enclosures.
>
> Thanks!
>
> zhongling0719@126.com
>
> From: Greg KH
> Date: 2022-11-09 18:40
> To: Juhyung Park
> CC: Hongling Zeng; stern; linux-usb; usb-storage; zhongling0719
> Subject: Re: [PATCH v6 2/3] usb-storage: Add Hiksemi USB3-FW to IGNORE_UAS
> On Mon, Oct 31, 2022 at 03:17:48PM +0900, Juhyung Park wrote:
> > Hi,
> >
> > I'm speaking from my own experience but RTL9210 is arguably the most
> > reliable NVMe-to-USB converter available.
> >
> > Compared to solutions from JMicron (multiple revisions) and ASMedia, RTL9210
> > gave the lowest power consumption (from implementing proper power management
> > commands) and the least headache.
> >
> > I personally own multiple enclosures and not one gave a single UAS error
> > from multiple platforms (Android, Intel, AMD) for years, but with this
> > commit now, it effectively disables UAS for all RTL9210 enclosures.
> >
> > Would it be possible to apply this quirk only to a specific firmware
> > (range)? RTL9210 have a lot of possible firmware combinations: https://www.station-drivers.com/index.php/en/component/remository/Drivers/Realtek/NVMe-USB-3.1/lang,en-gb/
> >
> > RTL9210 was available since 2019 and the fact that this quirk came up this
> > late leads me to believe that this is not a widespread issue and it'll be a
> > shame if all of RTL9210s are blacklisted from UAS with Linux from now on :(
> >
> > If any additional information is required, please let me know.
>
> Can you send a revert of this commit so that we can fix this up?
>
> thanks,
>
> greg k-h
^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2023-01-09 12:01 UTC | newest]
Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-09-15 2:49 [PATCH v6 2/3] usb-storage: Add Hiksemi USB3-FW to IGNORE_UAS Hongling Zeng
2022-09-15 19:32 ` Alan Stern
[not found] ` <2fcdca5a-2deb-66ca-fd1d-c3ba719b19af@126.com>
2022-09-16 17:21 ` Alan Stern
2022-09-18 12:25 ` zhongling0719
2022-09-22 8:17 ` nana
2022-09-22 15:16 ` Alan Stern
2022-10-31 6:17 ` Juhyung Park
2022-11-09 10:40 ` Greg KH
2022-11-12 7:45 ` zhongling0719
2022-11-12 7:56 ` Greg KH
2023-01-09 12:00 ` Juhyung Park
2023-01-09 11:58 ` Juhyung Park
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.