Re: [usb-storage] [PATCH] usb: storage: Add quirks for PNY Elite Portable SSD

Re: [usb-storage] [PATCH] usb: storage: Add quirks for PNY Elite Portable SSD

[Oliver Neukum]

On 01.05.26 15:23, Sam Burkels wrote:
> The PNY Elite Portable SSD (USB ID 154b:f009) is a sibling of the
> already-quirked PNY Pro Elite SSDs (154b:f00b and 154b:f00d). Like its
> siblings, it uses a Phison-based USB-SATA bridge that exhibits
> firmware bugs when bound to the uas driver.
> 
> Without quirks, the device fails to complete READ CAPACITY commands
> when accessed over UAS on a SuperSpeed (USB 3) port. The device
> enumerates and reports as a SCSI direct-access device, but reports
> zero logical blocks and never finishes spin-up:
> 
>      usb 2-3: new SuperSpeed USB device number 8 using xhci_hcd
>      usb 2-3: New USB device found, idVendor=154b, idProduct=f009
>      usb 2-3: Product: PNY ELITE PSSD
>      usb 2-3: Manufacturer: PNY
>      scsi host0: uas
>      scsi 0:0:0:0: Direct-Access     PNY      PNY ELITE PSSD   0
>      sd 0:0:0:0: [sda] Spinning up disk...
>      [...10+ seconds of polling, no progress...]
>      sd 0:0:0:0: [sda] Read Capacity(16) failed: hostbyte=DID_ERROR
>      sd 0:0:0:0: [sda] Read Capacity(10) failed: hostbyte=DID_ERROR
>      sd 0:0:0:0: [sda] 0 512-byte logical blocks: (0 B/0 B)
> 
> Tested each individual quirk to find the minimum that fixes this:
>    - US_FL_NO_ATA_1X alone: device hangs on spin-up
>    - US_FL_NO_REPORT_OPCODES alone: works on USB 2.0, hangs on USB 3.0
>    - US_FL_NO_ATA_1X | US_FL_NO_REPORT_OPCODES: works on both
> 
> With both quirks the device enumerates correctly while still using
> the uas driver, and delivers full UAS throughput (~281 MB/s
> sequential read on a USB 3.0 Gen 1 port).
> 
> The existing PNY Pro Elite entries (f00b, f00d) only set NO_ATA_1X,
> but this device additionally chokes on REPORT OPCODES under
> SuperSpeed.
> 
> Signed-off-by: Sam Burkels <sam@1a38.nl>
Acked-by: Oliver Neukum <oneukum@suse.com>