svn commit: r208183 - in head/sys: conf modules/mvs

[M. Warner Losh]

In message: <4BF8D0FE.4060700 at FreeBSD.org>
            Alexander Motin <mav at FreeBSD.org> writes:
: M. Warner Losh wrote:
: > In message: <201005170351.o4H3pvuG006399 at svn.freebsd.org>
: >             Alexander Motin <mav at FreeBSD.org> writes:
: > : Author: mav
: > : Date: Mon May 17 03:51:57 2010
: > : New Revision: 208183
: > : URL: http://svn.freebsd.org/changeset/base/208183
: > : 
: > : Log:
: > :   Make mvs_if.c to not be always linked statically into the kernel.
: > :   Link it same way as mvs.c.
: > 
: > A consequence of this will be that if you kldload modules that use the
: > mvs_if.m services, but don't have the mvs.ko module, then you'll have
: > problems.  I ran into this with pccard/pcic/pccbb/cardbus/*pccard.c
: > drivers so I compiled pccard_if.m into the kernel statically.
: > 
: > I don't know if that's a big deal for you, but it is why things like
: > this are usually linked in statically.
: 
: Not a big deal for now, until mvs_if.m used only inside mvs.ko itself
: for PCI devices or statically linked kernel for PCI/SoC. Controller
: driver is useless without channel driver, so probably it could be
: declared as dependency, if sometimes needed.
: 
: I just thought it is a bit dirty to have it in a kernel when all code
: loaded as module.

In that case, that should be fine.  Just wanted to warn about the
potential problem...

Warner