Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

usb_storage.quirks: string doesn't fit in 127 chars #3308

Closed
jens-maus opened this issue Apr 15, 2024 · 4 comments · Fixed by #3311
Closed

usb_storage.quirks: string doesn't fit in 127 chars #3308

jens-maus opened this issue Apr 15, 2024 · 4 comments · Fixed by #3311
Labels
board/raspberrypi Raspberry Pi Boards bug usb-ssd USB SSD related issue

Comments

@jens-maus
Copy link
Contributor

Describe the issue you are experiencing

With the latest version of HAos on RaspberryPi the usb_storage.quirks kernel parameter does not seem to fit into the maximum length (127 bytes) limit anymore.

This can be observed with looking at the bootup messages in dmesg:

[    0.000000] usb_storage.quirks: string doesn't fit in 127 chars.
[    0.000000] Booting kernel: `174c:55aa:u,2109:0715:u,152d:0578:u,152d:0579:u,152d:1561:u,174c:0829:u,14b0:0206:u,174c:225c:u,7825:a2a4:u,152d:0562:u,125f:a88a:u' too large for parameter `usb-storage.quirks'

Thus, the current usb-storage.quirks string is 132 characters long and thus exceeds the limit. Therefore it seems that the whole usb-storage.quirks mechanism is not applied on the RaspberryPi platform versions.

What operating system image do you use?

rpi3-64 (Raspberry Pi 3 64-bit OS)

What version of Home Assistant Operating System is installed?

12.2

Did you upgrade the Operating System.

No

Steps to reproduce the issue

  1. Bootup HAos
  2. look at dmesg output
  3. notice the usb_storage.quirks errors/notices of exceeding 127 characters.

Anything in the Supervisor logs that might be useful for us?

n/a

Anything in the Host logs that might be useful for us?

See above

System information

n/a

Additional information

No response

@jens-maus jens-maus added bug board/raspberrypi Raspberry Pi Boards labels Apr 15, 2024
@jens-maus
Copy link
Contributor Author

jens-maus commented Apr 15, 2024

Please note that this issue seems to be discussed also in the raspberrypi fora, here:

https://forums.raspberrypi.com/viewtopic.php?t=338094

There a kernel patch is suggested to increase the 127 character limit, thought.

@agners
Copy link
Member

agners commented Apr 15, 2024

Ugh! Yeah I guess adding the kernel patch to extend the character limit for now makes most sense.

Some PID/VIDs are actually "quirked" in the driver (as in the driver disables UAS). I was actually wondering if these cases are really a USB host interface <> USB device interaction issue, or if they could just be added to those quirks? Maybe the quirks need to be smarter, and quirk based on USB host interface the device is attached to? 🤔

jens-maus added a commit to jens-maus/RaspberryMatic that referenced this issue Apr 15, 2024
to 256 characters so that the currently specified quirks string does not
result in a kernel bootup error anymore.
(cf. home-assistant/operating-system#3308)
@jens-maus
Copy link
Contributor Author

FYI: See here for the intermediate kernel patch I added to my RaspberryMatic project to fix that issue there:

jens-maus/RaspberryMatic@a5c92c9

sairon added a commit that referenced this issue Apr 15, 2024
With #3281 we hit the maximum length of the quirks parameter. Since
cmdline.txt changes are not applied on OS update, only new installs of
12.2 are affected, effectively disabling all quirks until this patch
lands in future OS release.

Fixes #3308
@sairon sairon added the usb-ssd USB SSD related issue label Apr 15, 2024
@sairon
Copy link
Member

sairon commented Apr 15, 2024

Thanks for catching this, I opened a PR with the same fix for HAOS. Like I wrote in the PR description, only new installs will be affected (on the other hand, old installs don't get the latest updates of quirks with update), and the future kernel will fix this, so hopefully the impact won't be that big.

sairon added a commit that referenced this issue Apr 15, 2024
…3311)

With #3281 we hit the maximum length of the quirks parameter. Since
cmdline.txt changes are not applied on OS update, only new installs of
12.2 are affected, effectively disabling all quirks until this patch
lands in future OS release.

Fixes #3308
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
board/raspberrypi Raspberry Pi Boards bug usb-ssd USB SSD related issue
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants