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/>
References:  <bug-243401-227@https.bugs.freebsd.org/bugzilla/>

next in thread | previous in thread | raw e-mail | index | archive | help
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=3D243401

--- 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 incr=
ease
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 require=
d.

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?

--=20
You are receiving this mail because:
You are the assignee for the bug.=



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