From owner-freebsd-current@FreeBSD.ORG Sun May 7 18:29:03 2006 Return-Path: X-Original-To: current@freebsd.org Delivered-To: freebsd-current@FreeBSD.ORG Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id EDD4C16A463 for ; Sun, 7 May 2006 18:29:02 +0000 (UTC) (envelope-from anderson@centtech.com) Received: from mh1.centtech.com (moat3.centtech.com [207.200.51.50]) by mx1.FreeBSD.org (Postfix) with ESMTP id A11B643D6B for ; Sun, 7 May 2006 18:29:02 +0000 (GMT) (envelope-from anderson@centtech.com) Received: from [192.168.42.21] (andersonbox1.centtech.com [192.168.42.21]) by mh1.centtech.com (8.13.1/8.13.1) with ESMTP id k47ISxwa028619; Sun, 7 May 2006 13:28:59 -0500 (CDT) (envelope-from anderson@centtech.com) Message-ID: <445E3C6C.6080103@centtech.com> Date: Sun, 07 May 2006 13:29:00 -0500 From: Eric Anderson User-Agent: Thunderbird 1.5.0.2 (X11/20060506) MIME-Version: 1.0 To: Sam Leffler References: <445D3C94.10102@errno.com> <445D5D42.9040107@centtech.com> <445D795A.6000301@errno.com> In-Reply-To: <445D795A.6000301@errno.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit X-Virus-Scanned: ClamAV 0.87.1/1448/Sun May 7 06:41:19 2006 on mh1.centtech.com X-Virus-Status: Clean Cc: current@freebsd.org Subject: Re: CFT: new ath hal (take 2) X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.5 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: Sun, 07 May 2006 18:29:03 -0000 Sam Leffler wrote: > Eric Anderson wrote: >> Sam Leffler wrote: >>> A new test snapshot is available at: >>> >>> http://people.freebsd.org/~sam/ath_hal-20060506.tgz >>> >>> This is 0.9.17.0 and is known to work on sparc (tested) and may well >>> work on alpha (untested but likely the same issue that broke sparc). >>> >>> This code should be a drop-in replacement for what's in cvs in HEAD >>> but be sure to rebuild both the driver and rate control module as >>> there are ABI changes. Note that to use this hal with RELENG_6 or >>> other branches other than HEAD you must have rev 1.143 of if_ath.c. >>> To build a sparc module you must have rev 1.5 of >>> modules/ath_hal/Makefile and rev 1.539 of conf/options. >>> >>> Please test and report regressions. >> >> On 6.1-RC2, I get compile errors: >> 6.1-RC FreeBSD 6.1-RC #15: Fri May 5 08:29:44 CDT 2006 >> if_ath.c: __FBSDID("$FreeBSD: src/sys/dev/ath/if_ath.c,v 1.94.2.18 >> 2006/05/02 17:06:35 sam Exp $"); >> >> [..snip..] >> ===> ath (all) >> cc -O2 -pipe -march=pentiumpro -fno-strict-aliasing -Werror -D_KERNEL >> -DKLD_MODULE -nostdinc -I- -I. >> -I/usr/src/sys/modules/ath/../../contrib/dev/ath/freebsd >> -I/usr/src/sys/modules/ath/../../contrib/dev/ath >> -DHAVE_KERNEL_OPTION_HEADERS -include >> /usr/obj/usr/src/sys/NEUTRINO/opt_global.h -I. -I@ -I@/contrib/altq >> -I@/../include -finline-limit=8000 -fno-common -g >> -I/usr/obj/usr/src/sys/NEUTRINO -mno-align-long-strings >> -mpreferred-stack-boundary=2 -mno-mmx -mno-3dnow -mno-sse -mno-sse2 >> -ffreestanding -Wall -Wredundant-decls -Wnested-externs >> -Wstrict-prototypes -Wmissing-prototypes -Wpointer-arith -Winline >> -Wcast-qual -fformat-extensions -std=c99 -c >> /usr/src/sys/modules/ath/../../dev/ath/if_ath.c >> /usr/src/sys/modules/ath/../../dev/ath/if_ath.c: In function >> `ath_attach': >> /usr/src/sys/modules/ath/../../dev/ath/if_ath.c:296: warning: passing >> arg 3 of `ath_hal_attach' makes pointer from integer without a cast >> /usr/src/sys/modules/ath/../../dev/ath/if_ath.c:296: warning: passing >> arg 4 of `ath_hal_attach' makes pointer from integer without a cast >> *** Error code 1 >> >> Stop in /usr/src/sys/modules/ath. >> *** Error code 1 >> >> Stop in /usr/src/sys/modules. >> *** Error code 1 >> >> Stop in /usr/obj/usr/src/sys/NEUTRINO. >> *** Error code 1 >> >> Stop in /usr/src. >> *** Error code 1 >> >> Stop in /usr/src. >> >> >> >> >> > > Like I said: > > Note that to use this hal with RELENG_6 or > >> other branches other than HEAD you must have rev 1.143 of if_ath.c. *sigh* - my brain converted 1.94 to 1.194.. Sorry for the noise, and thanks for smacking me around.. :) Eric -- ------------------------------------------------------------------------ Eric Anderson Sr. Systems Administrator Centaur Technology Anything that works is better than anything that doesn't. ------------------------------------------------------------------------