[Fwd: cvs commit: src/sys/modules Makefile src/sys/conf files src/sys/modules/ath_rate_sample Makefile src/sys/i386/conf NOTES]

[Frank Mayhar]

Eric Anderson wrote:
> Hmm - this was working this morning before I updated, with a -CURRENT as of last week, and I didn't notice anything in /usr/src/UPDATING.  So what I should have is:
> 
> ath_load="YES"
> ath_hal_load="YES"
> ath_rate_load="YES"

I just have

if_ath_load="YES"

in my /boot/loader.conf.  That picks up everything.

> I guess what I missed (and just noticed), is that there are not individual modules for each rate algorithm now, one module (named ath_rate) that is the algorithm of whatever is built. 

It has been this way for some time, actually.

> > You want to go to sys/modules/ath_rate_{amrr|onoe|sample} and do a 'make
> > install'.
> I did that already - but why doesn't a 'make installkernel ...' not accomplish that?  (forgive my ignorance)

It does.  It installs ath_rate.ko for each of amrr, onoe and (now) sample.
Obviously, the last one in wins.  You'll have to ask Sam about what shortcoming
of the FreeBSD module system leads to this.

> > Hmm.  Is ath_rate_sample linked properly?
> How can I help you determine that?   (is readelf -a output what you want?)

Already fixed; read your -current email.
-- 
Frank Mayhar frank at exit.com	http://www.exit.com/
Exit Consulting                 http://www.gpsclock.com/
                                http://www.exit.com/blog/frank/