Page MenuHomeFreeBSD

Introduce quirks tunable for da driver
ClosedPublic

Authored by bevan_bi-co.net on Jun 17 2019, 7:41 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mar 6 2024, 6:58 PM
Unknown Object (File)
Dec 22 2023, 11:25 PM
Unknown Object (File)
Nov 12 2023, 5:21 AM
Unknown Object (File)
Nov 1 2023, 2:36 AM
Unknown Object (File)
Sep 7 2023, 9:14 PM
Unknown Object (File)
Aug 16 2023, 5:16 AM
Unknown Object (File)
Aug 16 2023, 4:52 AM
Unknown Object (File)
Jun 2 2023, 12:48 PM

Details

Summary

Introduce a tunable kern.cam.da.X.quirks for devices handled by da, similar to that already existing for ada. This simplifies testing quirks for new devices as the kernel does not need to be recompiled.

I used this while trying to fix issues with a misbehaving drive. I don't know if this is something you may want to include, however I thought I'd post it here before it gets lost.

Test Plan

I tested this on FreeBSD 12.0-RELEASE and 13.0-CURRENT by adding the tunable to loader.conf, setting it to a combination of several quirks, and observing the kernel output when loading the corresponding device.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

mav requested changes to this revision.Sep 17 2019, 10:04 PM

Generally I think it is good, but I think it should be set higher, after the "softc->quirks |= DA_Q_NO_6_BYTE;" line.

This revision now requires changes to proceed.Sep 17 2019, 10:04 PM

Thanks for the review! I agree it makes sense to move the code up so that other settings are derived accordingly.

This revision is now accepted and ready to land.Sep 18 2019, 7:42 PM