From owner-freebsd-current@FreeBSD.ORG Sat Mar 30 23:29:58 2013 Return-Path: Delivered-To: freebsd-current@freebsd.org Received: from mandree.no-ip.org (freefall.freebsd.org [IPv6:2001:1900:2254:206c::16:87]) by hub.freebsd.org (Postfix) with ESMTP id D3A8999C; Sat, 30 Mar 2013 23:29:57 +0000 (UTC) (envelope-from mandree@FreeBSD.org) Received: from [127.0.0.1] (localhost.localdomain [127.0.0.1]) by apollo.emma.line.org (Postfix) with ESMTP id 5424B23CEDF; Sun, 31 Mar 2013 00:29:55 +0100 (CET) Message-ID: <5157756F.4040908@FreeBSD.org> Date: Sun, 31 Mar 2013 00:29:51 +0100 From: Matthias Andree User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130308 Thunderbird/17.0.4 MIME-Version: 1.0 To: freebsd-stable@freebsd.org Subject: Re: Any objections/comments on axing out old ATA stack? References: <51536306.5030907@FreeBSD.org> In-Reply-To: <51536306.5030907@FreeBSD.org> X-Enigmail-Version: 1.4.6 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="------------enig338DC6674EB2008BF0825764" Cc: Alexander Motin , freebsd-current@freebsd.org X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 30 Mar 2013 23:29:58 -0000 This is an OpenPGP/MIME signed message (RFC 2440 and 3156) --------------enig338DC6674EB2008BF0825764 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable Am 27.03.2013 22:22, schrieb Alexander Motin: > Hi. >=20 > Since FreeBSD 9.0 we are successfully running on the new CAM-based ATA > stack, using only some controller drivers of old ata(4) by having > `options ATA_CAM` enabled in all kernels by default. I have a wish to > drop non-ATA_CAM ata(4) code, unused since that time from the head > branch to allow further ATA code cleanup. >=20 > Does any one here still uses legacy ATA stack (kernel explicitly built > without `options ATA_CAM`) for some reason, for example as workaround > for some regression? Does anybody have good ideas why we should not dro= p > it now? Alexander, The regression in http://www.freebsd.org/cgi/query-pr.cgi?pr=3Dkern/15739= 7 where the SATA NCQ slots stall for some Samsung drives in the new stack, and consequently hang the computer for prolonged episodes where it is in the NCQ error handling, disallows removal of the old driver. (Last checked with 9.1-RELEASE at current patchlevel.) Chances are that limiting the open queue slots to 31 might help, but that is hearsay from what Linux would be doing. Unless we get a fix, if you want to drop the old driver, you'll need to add features so that 1. the new driver to lets users (down-)configure the max. number of tagged openings 2. the new driver allows disabling NCQ altogether for individual drives 3. list the relevant Samsung drives in some quirks data base so that we avoid the stalls while permitting users to "open it up to 32 NCQ slots". So unless these are all addressed, I'd veto removal of the old ATA driver - sorry! Best regards Matthias --------------enig338DC6674EB2008BF0825764 Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.11 (GNU/Linux) Comment: Using GnuPG with undefined - http://www.enigmail.net/ iEYEARECAAYFAlFXdW8ACgkQvmGDOQUufZVJ2ACg0eUnW4Xe5nVJo3zLTaYjUARG KcgAniDIIJ8oveV3YB5GV7sNTYc1+ar8 =2f7y -----END PGP SIGNATURE----- --------------enig338DC6674EB2008BF0825764--