From owner-freebsd-stable@FreeBSD.ORG Sun Mar 31 09:30:04 2013 Return-Path: Delivered-To: freebsd-stable@freebsd.org Received: from mx1.freebsd.org (mx1.FreeBSD.org [8.8.178.115]) by hub.freebsd.org (Postfix) with ESMTP id 65CE1106; Sun, 31 Mar 2013 09:30:04 +0000 (UTC) (envelope-from mavbsd@gmail.com) Received: from mail-ee0-f48.google.com (mail-ee0-f48.google.com [74.125.83.48]) by mx1.freebsd.org (Postfix) with ESMTP id C91622C3; Sun, 31 Mar 2013 09:30:03 +0000 (UTC) Received: by mail-ee0-f48.google.com with SMTP id b15so726582eek.21 for ; Sun, 31 Mar 2013 02:29:56 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=x-received:sender:message-id:date:from:user-agent:mime-version:to :cc:subject:references:in-reply-to:content-type :content-transfer-encoding; bh=nGyuimu7oPNSnlJu5IhuDpDDqSMsVdSLmG6nI6NwDtY=; b=eXIBJ28eBZuzXcwkP6Vpu9ONyOPbIUaIGHFSLoJ9qrpcos91K1dKHPlX/sNQBnY1Xp vrakGzviKdj96eB6xOF/8VzU8a+ooaDta2DeJJI8l6tmtSZA0cd9UKWiY0XvrmAs2Dm4 0Xde3hsMhFkeKKPnSp8oXO6TKwj59hcSPrnwKWOt8BvxqZrgItSK5yLg/80lhPlAh5GZ wQ75CP+FFEbK/nItj4EmXepLopuhNew21mf3qY1rkEdzJcb9zbVLMg9ic546e6VlgYlt eDTNfrVPQ/1hifvcj6Rdhy5hIIbVORXv0dfVkQ/i2aWqEalJbwW1GlD5b+sQAFM1ZWBD aCpA== X-Received: by 10.14.183.198 with SMTP id q46mr26109106eem.1.1364722196577; Sun, 31 Mar 2013 02:29:56 -0700 (PDT) Received: from mavbook.mavhome.dp.ua ([91.198.175.1]) by mx.google.com with ESMTPS id 44sm14550209eek.5.2013.03.31.02.29.54 (version=TLSv1 cipher=ECDHE-RSA-RC4-SHA bits=128/128); Sun, 31 Mar 2013 02:29:55 -0700 (PDT) Sender: Alexander Motin Message-ID: <5157F401.1080609@FreeBSD.org> Date: Sun, 31 Mar 2013 11:29:53 +0300 From: Alexander Motin User-Agent: Mozilla/5.0 (X11; FreeBSD amd64; rv:17.0) Gecko/20130326 Thunderbird/17.0.4 MIME-Version: 1.0 To: Ian Smith Subject: Re: Any objections/comments on axing out old ATA stack? References: <51536306.5030907@FreeBSD.org> <5157756F.4040908@FreeBSD.org> <20130331160157.G36471@sola.nimnet.asn.au> In-Reply-To: <20130331160157.G36471@sola.nimnet.asn.au> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: freebsd-stable@freebsd.org, Matthias Andree , Peter Wemm X-BeenThere: freebsd-stable@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: Production branch of FreeBSD source code List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 31 Mar 2013 09:30:04 -0000 On 31.03.2013 08:13, Ian Smith wrote: > On Sat, 30 Mar 2013 21:00:24 -0700, Peter Wemm wrote: > > On Sat, Mar 30, 2013 at 4:29 PM, Matthias Andree wrote: > > > Am 27.03.2013 22:22, schrieb Alexander Motin: > > >> Hi. > > >> > > >> 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. > > >> > > >> 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 drop > > >> it now? > > > > > > Alexander, > > > > > > The regression in http://www.freebsd.org/cgi/query-pr.cgi?pr=kern/157397 > > > 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.) > > > > We're talking about 10.x, so if you want it fixed, you need update > > with 10.x information. > > > > Please put 10.x diagnostics in the PR. > > Given Alexander also posted this to -stable, just for clarity, are we > _only_ talking about 10.x here, or might this change get MFC'd to 9? Yes, I am only going to drop it from 10.x, but bug reports from 9-STABLE users are welcome, as at some point they will become 10.x users. -- Alexander Motin