Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 04 Jan 2021 17:22:33 +0000
From:      bugzilla-noreply@freebsd.org
To:        bugs@FreeBSD.org
Subject:   [Bug 243401] [patch] ahci driver problems with Marvell 88SE9230 (Dell BOSS-S1)
Message-ID:  <bug-243401-227-d7NBaGICRS@https.bugs.freebsd.org/bugzilla/>
In-Reply-To: <bug-243401-227@https.bugs.freebsd.org/bugzilla/>

index | next in thread | previous in thread | raw e-mail

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

--- Comment #8 from Alexander Motin <mav@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.

help

Want to link to this message? Use this
URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?bug-243401-227-d7NBaGICRS>