Date: Sun, 28 Aug 2011 17:42:18 -0700 From: Garrett Cooper <yanegomi@gmail.com> To: Adrian Chadd <adrian@freebsd.org> Cc: FreeBSD Current <freebsd-current@freebsd.org> Subject: Re: if_ath - should it be compiled without ath_hal support? Message-ID: <CAGH67wT_JT8zJ1T%2BOMsdOthmYdeKvePvKnt5DRQ%2BqW1h8NDgqw@mail.gmail.com> In-Reply-To: <CAJ-Vmo=F9d9_Tvt9eLWCRK3W4X17j3JDSUzQxGubMrdHg7JvLA@mail.gmail.com> References: <CAGH67wRz6X=ZU0AZFM1G-fo=NKHNVV7SD547KD-BBp4VOofDaw@mail.gmail.com> <CAJ-VmonnTxoWMbRE-PqtoDH1_16OpwK0Jw9hZ6WVkQjnr-JTfw@mail.gmail.com> <CAGH67wSX3d8AM%2B-Qc_M2RfBL_VMyRPB7na_JmNxq2nfb4fL0Bg@mail.gmail.com> <CAJ-Vmo=F9d9_Tvt9eLWCRK3W4X17j3JDSUzQxGubMrdHg7JvLA@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Sun, Aug 28, 2011 at 5:21 PM, Adrian Chadd <adrian@freebsd.org> wrote:
> that looks a bit special.
>
> On 29 August 2011 07:38, Garrett Cooper <yanegomi@gmail.com> wrote:
>
>>> I'd like to eventually split out the ath_hal again into a separate
>>> module and re-instate the separation between HAL and driver code
>>> (primarily to enforce cleaner code and more stable APIs.)
>>
>> Good to know. I was just a bit confused with the manpages that say
>> they're optional after I discovered looking at the driver that they're
>> hardwired in.
>
> I plan on fleshing out the driver documentation in the Wiki a little more=
.
> Like everything, all I need is time. :)
>
>>> If you're seeing issues like the above, chances are that you're seeing
>>> some corruption. Can you please print out the whole value of *rs (ie,
>>> all the fields of the ath_rx_status struct) so I can see how valid it
>>> is?
>>
>> Sure:
>>
>> (kgdb) p *rs
>> $1 =3D {rs_datalen =3D 244, rs_status =3D 0 '\0', rs_phyerr =3D 0 '\0',
>> rs_rssi =3D 0 '\0', rs_keyix =3D 127 '\177', rs_rate =3D 27 '\033', rs_m=
ore
>> =3D 0 '\0', rs_tstamp =3D 2097280, rs_antenna =3D 8399232, rs_rssi_ctl =
=3D
>> "0\200)", rs_rssi_ext =3D "\200\000\037", rs_isaggr =3D 1 '\001',
>> rs_moreaggr =3D 1 '\001', rs_num_delims =3D 0 '\0', rs_flags =3D 252 '=
=FC',
>> rs_evm0 =3D 0, rs_evm1 =3D 0, rs_evm2 =3D 0, rs_evm3 =3D 0, rs_evm4 =3D =
0}
>
> It doesn't look like garbage to begin with .. and then it does.
>
> How have you compiled it? Some of those fields are currently optional
> (ie, require the AR5416 (11n) support to be enabled) - have you
> somehow managed to compile part of the code with it, part of the code
> without it?
Just
options ATH_DEBUG
in my $KERNCONF and I built the ath and ath_pci modules.
> Which NIC is this for again? AR9285?
Yup:
ath0@pci0:2:0:0: class=3D0x028000 card=3D0x10891a3b chip=3D0x002b168=
c
rev=3D0x01 hdr=3D0x00
vendor =3D 'Atheros Communications Inc.'
device =3D 'AR9285 Wireless Network Adapter (PCI-Express)'
class =3D network
Thanks!
-Garrett
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAGH67wT_JT8zJ1T%2BOMsdOthmYdeKvePvKnt5DRQ%2BqW1h8NDgqw>