Date: Tue, 23 May 2006 11:22:05 -0600 (MDT) From: Warner Losh <imp@bsdimp.com> To: scottl@samsco.org Cc: freebsd-multimedia@freebsd.org, des@des.no, freebsd-current@freebsd.org, Alexander@Leidinger.net Subject: Re: Call for testing: emu10kx driver for Creative sound cards Message-ID: <20060523.112205.115998971.imp@bsdimp.com> In-Reply-To: <4473336F.1030900@samsco.org> References: <20060523171238.6392afdb@Magellan.Leidinger.net> <867j4clqaj.fsf@xps.des.no> <4473336F.1030900@samsco.org>
next in thread | previous in thread | raw e-mail | index | archive | help
From: Scott Long <scottl@samsco.org> Subject: Re: Call for testing: emu10kx driver for Creative sound cards Date: Tue, 23 May 2006 10:08:15 -0600 > Dag-Erling Sm=F8rgrav wrote: > > Alexander Leidinger <Alexander@Leidinger.net> writes: > > = > >>Quoting des@des.no (Dag-Erling Sm=F8rgrav) (Tue, 23 May 2006 14:26:= 58 +0200): > >> > >>>Yuriy Tsibizov <Yuriy.Tsibizov@gfk.ru> writes: > >>> > >>>>2. Complete mixer support. Some controls that can't fit into OSS > >>>>mixer are available as sysctl under debug.emu10kxX. > >>> > >>>That is not the correct place for it. Please use the device's sys= ctl > >>>context (obtained with device_get_sysctl_ctx()) > >> > >>This was based upon a suggestion by me. We want to get rid of most > >>sound related syscalls (at least those which belong into the realm = of > >>the user, and not into the realm of the administrator). Until we ha= ve > >>an application and an interface, we have to life with the sysctls, = but > >>to let the users know that this is not an interface but a temporary= > >>workaround, it's put into the debug MIB. I hope we will not ship > >>7.0-RELEASE with any such sysctl (any help appreciated). > > = > > = > > Regardless, device-specific sysctl knobs belong in the individual > > device's sysctl context, which automatically places them in the > > correct location in the dev tree and automatically destroys them wh= en > > the device is destroyed. Please do not create further precedent fo= r > > breaking this rule, no matter how good your intentions. > > = > > DES > = > The problem is that Alexander wants these sysctls to only be temporar= y. > Recall that big thread from a month or two ago about treating sysctls= > as an API, and how there was heavy disagreement over how to define > "stable" sysctls that apps could depend on? If a temporary set of = > sysctls get put under the dev tree, then it risks becoming permanent,= > which is not what Alexander wants. So, either we need to decide what= > parts of the sysctl to define as stable, like I asked for in the = > previous thread, or we need to pretend that it's not a problem that w= e > should address, and let you and Alexander continue to argue over the > 'correct place'. Then put them under the right place, but create a subtree that's "tmp" In general, drivers should avoid using the debug.* space. Warner
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20060523.112205.115998971.imp>