Skip to content
This repository has been archived by the owner on Mar 19, 2024. It is now read-only.

Disable UAS for JMicron JMS578 and JMS579 #95

Merged
merged 1 commit into from
Aug 15, 2020

Conversation

mayankchhabra
Copy link
Member

JMicron's JMS578 and JMS579 are one of the most popular chipsets used in USB-to-SATA adapters, but they're still blacklisted by the kernel USB subsystem and require explicit quirk to disable UAS.

This PR adds an explicit quirk to disable UAS for both the chipsets.

Net effect:
SSD read speed before the quirk: 50-100kbps
SSD read speed after the quirk: ~200mbps (depending upon the SSD)

Further reading:

@AaronDewes
Copy link
Contributor

but they're still blacklisted by the kernel USB subsystem and require explicit quirk to disable UAS.

According to https://www.raspberrypi.org/forums/viewtopic.php?t=245931, if I understand it correctly, they are not blacklisted but should be.
UAS is enabled except if the device is in the blacklist.

@mayankchhabra
Copy link
Member Author

I'm not sure I understood you fully there... but I can confirm that JMS578 chipset is indeed blacklisted as that's what I use. And adding a usb-storage quirk disabled UAS for it and now I can successfully use it.

@AaronDewes
Copy link
Contributor

As far as I understand this, blacklisted devices work, the quirk adds them to the blacklist. The blacklist contains devices which don't support UAS, not devices that should use UAS

@mayankchhabra
Copy link
Member Author

blacklisted devices work, the quirk adds them to the blacklist

Oh no, the quirk disables UAS for the device that is added.

The blacklist contains devices which don't support UAS, not devices that should use UAS

This is true. That's why we add a quirk here, to switch off UAS for these specific chipsets since they're blacklisted by default.

@mayankchhabra mayankchhabra deleted the patch/quirks branch August 21, 2020 19:30
tmm1 added a commit to fancybits/operating-system that referenced this pull request Oct 9, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants