Re: CFT: ath hal 0.9.18.0

On Sat, September 9, 2006 11:34 am, Sam Leffler wrote:
> You can find a new ath hal at:
> http://people.freebsd.org/~sam/ath_hal-20060909.tgz
>
> It's gone through basic testing but not the more extensive testing I
> do before a release.

> To make use of the new tkip mic keycache support you need to patch
> the driver with this diff:
> http://people.freebsd.org/~sam/keycache.patch

> Should work on both HEAD and RELENG_6 w/o any driver mods.

It's most likely that I'm missing something simple in the process, but
I can't get this new HAL to work with my 6.1-RELEASE system.  If it's
as simple as "It only works with RELENG_6 not RELENG_6_1" then I guess
I'll wait.  Otherwise, here's what happens when I try to use it:

- copy /usr/src/sys/contrib/dev/ath to ath.orig
- untar and rename the new HAL directory to /usr/src/sys/contrib/dev/ath
- run "patch -p0 < keycache.diff" in /usr/src/sys/dev/ath/
- "make clean; make depend; make; make install" in
/usr/src/sys/modules/ath gives

rm -f export_syms if_ath.ko if_ath.kld if_ath.o if_ath_pci.o _at_ machine
opt_inet.h opt_ath.h pci_if.h bus_if.h device_if.h
_at_ -> /usr/src/sys
machine -> /usr/src/sys/i386/include
awk -f _at_/tools/makeobjops.awk _at_/kern/device_if.m -h
awk -f _at_/tools/makeobjops.awk _at_/kern/bus_if.m -h
awk -f _at_/tools/makeobjops.awk _at_/dev/pci/pci_if.m -h
touch opt_inet.h
echo > opt_ath.h
rm -f .depend
mkdep -f .depend -a   -nostdinc -D_KERNEL -DKLD_MODULE -I- -I.
-I/usr/src/sys/modules/ath/../../contrib/dev/ath/freebsd
-I/usr/src/sys/modules/ath/../../contrib/dev/ath -I. -I_at_
-I_at_/contrib/altq -I_at_/../include -I/usr/include
/usr/src/sys/modules/ath/../../dev/ath/if_ath.c
/usr/src/sys/modules/ath/../../dev/ath/if_ath_pci.c
Warning: Object directory not changed from original
/usr/src/sys/modules/ath
cc -O2 -fno-strict-aliasing -pipe -march=pentium4 -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 -I. -I_at_
-I_at_/contrib/altq -I_at_/../include -finline-limit=8000 -fno-common 
-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.

 - building the ath_hal module works correctly, althought trying to
use it with a stock if_ath module gives

ath_hal: 0.9.18.0 (AR5210, AR5211, AR5212, RF5111, RF5112, RF2413,
RF5413)
ath0: <Atheros 5212> mem 0xd0010000-0xd001ffff irq 17 at device 4.0 on
pci2
ath0: HAL ABI mismatch detected (HAL:0x6090700 != driver:0x5122200)
device_attach: ath0 attach returned 6
Warning: memory type ath_hal leaked memory on destroy (5 allocations,
25984 bytes leaked).
warning: KLD '/boot/kernel/if_ath.ko' is newer than the linker.hints file
warning: KLD '/boot/kernel/ath_rate.ko' is newer than the linker.hints
file
warning: KLD '/boot/kernel/ath_hal.ko' is newer than the linker.hints
file


----
Freddie Cash
fcash_at_ocis.net