https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=243401

--- Comment #8 from Alexander Motin <m...@freebsd.org> ---
I still see the patch from December 21.  Where is the updated version?

It is good that DEV_PRESENT is reported fast enough.  It allows to not increase
timeout in case of device absent and so I am thinking about just increasing the
timeout slightly instead of adding the quirk.

ALTSIG quirk was required by some early Marvell controllers or firmware
versions. I haven't retested it on recent ones, it may no longer be required.

I don't remember why I have disabled the FBS there, its being a while ago. My
guess is to avoid other commands from trying to execute during soft reset.  If
you look lower, you'll see "Kick controller into sane state and enable FBS."
comment, which should call ahci_start(ch, 1) on line 2121 after soft reset
complete.

PS: Why do you need AHCI-specific verbosity tunable?  Why not just enable it
globally?

-- 
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
freebsd-bugs@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-bugs
To unsubscribe, send any mail to "freebsd-bugs-unsubscr...@freebsd.org"

Reply via email to