From owner-freebsd-stable@FreeBSD.ORG Sat Apr 7 11:52:37 2012 Return-Path: Delivered-To: freebsd-stable@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id 41D26106566B for ; Sat, 7 Apr 2012 11:52:37 +0000 (UTC) (envelope-from marius@alchemy.franken.de) Received: from alchemy.franken.de (alchemy.franken.de [194.94.249.214]) by mx1.freebsd.org (Postfix) with ESMTP id CE3548FC0C for ; Sat, 7 Apr 2012 11:52:36 +0000 (UTC) Received: from alchemy.franken.de (localhost [127.0.0.1]) by alchemy.franken.de (8.14.4/8.14.4/ALCHEMY.FRANKEN.DE) with ESMTP id q37Bq6S9012750; Sat, 7 Apr 2012 13:52:06 +0200 (CEST) (envelope-from marius@alchemy.franken.de) Received: (from marius@localhost) by alchemy.franken.de (8.14.4/8.14.4/Submit) id q37Bq5PA012749; Sat, 7 Apr 2012 13:52:05 +0200 (CEST) (envelope-from marius) Date: Sat, 7 Apr 2012 13:52:05 +0200 From: Marius Strobl To: Daniel Braniss Message-ID: <20120407115205.GW93449@alchemy.franken.de> References: <20120406171644.GA9162@alchemy.franken.de> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.4.2.3i Cc: freebsd-stable@freebsd.org Subject: Re: 8.3-PRERELEASE and ATA_CAM X-BeenThere: freebsd-stable@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Production branch of FreeBSD source code List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 07 Apr 2012 11:52:37 -0000 On Sat, Apr 07, 2012 at 10:53:39AM +0300, Daniel Braniss wrote: > > On Fri, Apr 06, 2012 at 10:48:13AM +0300, Daniel Braniss wrote: > > > with the latest svn, I can't compile kernel with options ATA_CAM: > > > > > > ... > > > linking kernel.debug > > > ata-disk.o(.text+0x93): In function `ad_init': > > > /r+d/stable/8.3/sys/dev/ata/ata-disk.c:389: undefined reference to > > > `ata_setmode' > > > ata-disk.o(.text+0xaa):/r+d/stable/8.3/sys/dev/ata/ata-disk.c:397: undefined > > > reference to `ata_wc' > > > ata-disk.o(.text+0xc5):/r+d/stable/8.3/sys/dev/ata/ata-disk.c:398: undefined > > > reference to `ata_controlcmd' > > > ata-disk.o(.text+0x113):/r+d/stable/8.3/sys/dev/ata/ata-disk.c:400: undefined > > > reference to `ata_controlcmd' > > > ata-disk.o(.text+0x133):/r+d/stable/8.3/sys/dev/ata/ata-disk.c:393: undefined > > > reference to `ata_controlcmd' > > > ata-disk.o(.text+0x16d):/r+d/stable/8.3/sys/dev/ata/ata-disk.c:407: undefined > > > reference to `ata_controlcmd' > > > ata-disk.o(.text+0x21a): In function `ad_shutdown': > > > /r+d/stable/8.3/sys/dev/ata/ata-disk.c:196: undefined reference to > > > `ata_controlcmd' > > > ata-disk.o(.text+0x45c): In function `ad_detach': > > > /r+d/stable/8.3/sys/dev/ata/ata-disk.c:182: undefined reference to > > > `ata_fail_requests' > > > ... > > > > > > > You seem to be using a mutually exclusive set of ata(4) options and > > devices (previously, this erroneously wasn't a bug). When including > > options ATA_CAM you do _not_ want to also include any of the following > > devices: > > device atapicam > > device atadisk > > device ataraid > > device atapicd > > device atapifd > > device atapist > > > > Instead you need the corresponding driver from the following set: > > device scbus > > device ch > > device da > > device sa > > device cd > > device pass > > > > Marius > > > they are included by GENERIC, which i include, bummer. It should work to include GENERIC but disable the "old" devices via "nodevice atadisk" etc. in your custom kernel configuration file. > what about ATA_STATIC_ID, I guess that is also a nono? No, ATA_STATIC_ID is still available with ATA_CAM, in this case it's purpose is a bit different though; when present, it provides /dev/adX symbolic links to the /dev/adaY device nodes, trying to mimic the non-ATA_CAM numbering for /dev/adX. If you've already updated your /etc/fstab etc. to use /dev/adaY instead, you don't need ATA_STATIC_ID though. Marius