turning on kernel modules now?

[Sean McNeil]

On Sat, 2004-08-28 at 21:26, Ian Dowse wrote:
> In message <1093752734.69450.2.camel at server.mcneil.com>, Sean McNeil writes:
> >I saw the great news about pre-loading kernel modules now working.  This
> >is fantastic.  Does this mean that the sound drivers will become kernel
> >loadable modules now?
> 
> I don't know if there's a good reason why sound modules are not
> built on amd64 at the moment. They do build fine if enabled, and
> at least snd_via8233 works here (both preloaded and kldloaded). I
> suppose there might be amd64-specific problems with some of the
> other drivers.
> 
> To enable building the sound modules on amd64 by default, just apply
> the following patch to /usr/src/sys/modules/Makefile.
> 
> Ian
> 
> Index: Makefile
> ===================================================================
> RCS file: /dump/FreeBSD-CVS/src/sys/modules/Makefile,v
> retrieving revision 1.394
> diff -u -r1.394 Makefile
> --- Makefile	18 Aug 2004 11:59:28 -0000	1.394
> +++ Makefile	29 Aug 2004 04:15:30 -0000
> @@ -378,6 +378,7 @@
>  _drm=		drm
>  _io=		io
>  #_ndis=		ndis
> +_sound=		sound
>  .endif
>  
>  .if ${MACHINE_ARCH} == "ia64"

This sounds like something that should be done by default.  If they
compile, then they are there for users to try.  Otherwise, there would
be no telling if anything else besides snd_via8233 works (I have that
device too) for the generic user.

I can't think of anything harmful in doing this, can you?  Can we get it
committed?

Cheers,
Sean