Date: Sun, 23 May 2010 09:53:50 +0300 From: Alexander Motin <mav@FreeBSD.org> To: "M. Warner Losh" <imp@bsdimp.com> Cc: svn-src-head@FreeBSD.org, svn-src-all@FreeBSD.org, src-committers@FreeBSD.org Subject: Re: svn commit: r208183 - in head/sys: conf modules/mvs Message-ID: <4BF8D0FE.4060700@FreeBSD.org> In-Reply-To: <20100523.003526.1142818667088610804.imp@bsdimp.com> References: <201005170351.o4H3pvuG006399@svn.freebsd.org> <20100523.003526.1142818667088610804.imp@bsdimp.com>
next in thread | previous in thread | raw e-mail | index | archive | help
M. Warner Losh wrote: > In message: <201005170351.o4H3pvuG006399@svn.freebsd.org> > Alexander Motin <mav@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
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?4BF8D0FE.4060700>