From owner-freebsd-stable@FreeBSD.ORG Mon Apr 1 16:11:38 2013 Return-Path: Delivered-To: freebsd-stable@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by hub.freebsd.org (Postfix) with ESMTP id 9374183C; Mon, 1 Apr 2013 16:11:38 +0000 (UTC) (envelope-from victor@bsdes.net) Received: from equilibrium.bsdes.net (244.Red-217-126-240.staticIP.rima-tde.net [217.126.240.244]) by mx1.freebsd.org (Postfix) with ESMTP id 3BB6FFDA; Mon, 1 Apr 2013 16:11:37 +0000 (UTC) Received: by equilibrium.bsdes.net (Postfix, from userid 1001) id 4D64D2283F; Mon, 1 Apr 2013 18:11:36 +0200 (CEST) Date: Mon, 1 Apr 2013 18:11:36 +0200 From: Victor Balada Diaz To: Stefan Esser Subject: Re: Any objections/comments on axing out old ATA stack? Message-ID: <20130401161136.GR3178@equilibrium.bsdes.net> References: <51536306.5030907@FreeBSD.org> <20130331130409.GO3178@equilibrium.bsdes.net> <20130401131415.GQ3178@equilibrium.bsdes.net> <5159A2A8.3090705@freebsd.org> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <5159A2A8.3090705@freebsd.org> User-Agent: Mutt/1.5.21 (2010-09-15) Cc: freebsd-stable@freebsd.org 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: Mon, 01 Apr 2013 16:11:38 -0000 On Mon, Apr 01, 2013 at 05:07:20PM +0200, Stefan Esser wrote: > Am 01.04.2013 15:14, schrieb Victor Balada Diaz: > > Being able to configure quirks from loader.conf for disks AND controllers would be great > > and is not hard to do. If you want i can do a patch in two weeks and send it to you. That > > way it's easy to test disabling NCQ and/or other things in case of hitting a bug. Also > > being able to modify the configuration without a kernel recompile would be a big > > improvement because we could still use freebsd-update to keep systems updated. > > Something like: > > kern.cam.ada.0.quirks=1 > > to force 4KB sectors? > > No need to implement that, it is in -CURRENT (did not check -STABLE). > But there is no quirk, that disables NCQ, currently, although it is > easy to implement. See the places where "ADA_FLAG_CAN_NCQ" is set and > make that value depend on a new quirk flag being unset ... > > But instead of setting that flag in the loader, it would be good to > collect drive signatures that need it and to add quirk entries for > them in ata_da.c ... > > Regards, STefan Yep, something like that but also for controllers. Looking here[1] i don't see it implemented for controllers on current. I agree that we should collect drive and controller signatures and add that quirks to the OS, but being able to play with quirks from loader is still useful. If your FreeBSD version don't have yet the quirks needed for the disk/controller that you're using, you'd need to patch and rebuild a custom kernel. Having a loader tunable helps maintaining "old" FreeBSD versions easier. Regards. Victor. [1]: http://fxr.watson.org/fxr/source/dev/ahci/ahci.c -- La prueba más fehaciente de que existe vida inteligente en otros planetas, es que no han intentado contactar con nosotros.