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

Alexander Motin mav at FreeBSD.org
Sun May 23 06:54:13 UTC 2010


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.

-- 
Alexander Motin


More information about the svn-src-head mailing list