cvs commit: src/sys/dev/sound/pcm mixer.c

Sean Kelly smkelly at FreeBSD.org
Sat Jun 26 04:11:48 PDT 2004


On Thu, Jun 24, 2004 at 08:18:01PM +0200, Josef El-Rayes wrote:
> John Baldwin <jhb at FreeBSD.org> wrote:
> > Does this really need to be here considering that with rcNG (on by default on 
> > current) and /etc/rc.d/mixer this has no effect?
> 
> Eugene Grosbein requested this (kern/63771) implicitely
> and Cameron Grant decided that it is good to accomplish
> this via kenv variables.
> 
> It literally adds only ~5 lines of code, it does not
> decrease stability but adds a feature, I see
> lots of kernel modules configurated via hints and
> bootloader, why not let pcm be configured via this too.
> Why do we have to force users to use mixer script, which
> is not documented anywhere too.
> 
> I will, of course, remove it, if it is wished, I just
> dont see why anyone would have a problem.

There is a real argument for keeping this and not just letting the rcNG
script do it all. Other than being a feature, the real argument goes
something like this.

If I have something plugged into "Line In" on my soundcard and I mute teh
channel, next time I reboot I will hear that "Line In" signal until the
rcNG script runs and mutes it. There is a small time period between sound
being up and the rcNG script running. If I had something constantly,  loud
and annoying connected to "Line In", that would suck.

If you really wanted to be evil, you could have the rcNG script load and
save the mixer settings from/to the loader.conf/device.hints files (or save
to another file and 'include' it somehow). Then you don't have to wait for
the rc scripts to run for the volumes to be reset.

-- 
Sean Kelly          | PGP KeyID: D2E5E296
smkelly at FreeBSD.org | http://www.sean-kelly.org/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 187 bytes
Desc: not available
Url : http://lists.freebsd.org/pipermail/cvs-src/attachments/20040626/5cc3f865/attachment.bin


More information about the cvs-src mailing list