Re: Persistent module storage [was Linux 2.4 Status / TODO page]

From: James A. Sutherland (jas88@cam.ac.uk)
Date: Mon Nov 06 2000 - 12:33:54 EST


On Mon, 06 Nov 2000, David Woodhouse wrote:
> jas88@cam.ac.uk said:
> > Yippee. As we all know, implementing GUI volume controls and putting
> > the slider in the right place is a kernel function, and nothing to do
> > with userspace...
>
> Don't troll, James. The kernel needs to provide the functionality required
> by userspace. The functionality required in this case is the facility to
> read the current mixer levels.

Except this isn't possible with the hardware in question! If it were, there
would be no problem. In cases where the hardware doesn't support the
functionality userspace "needs", why put the kludge in the kernel?

If userspace wants to know what settings it set last time, it should store
those values somewhere.

> jas88@cam.ac.uk said:
> > The right thing in this context is not to screw with hardware
> > settings unless and until it is given settings to set. Do not set
> > values arbitrarily: set only the values you are explicitly given.
> > Anything else is simply a bug in your driver.
>
> It is unwise to assume that the hardware is in a sane state when the driver
> has been unloaded and reloaded. I agree that you should set the values that
> were explicitly given. That's why we should remember them.

No values are being explicitly given. Loading the driver should not cause
any settings to be changed: changing the settings should do that!

There is no need for the drivers to change any settings. If the settings need
to be (re)set, let userspace do it.
 

James.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Tue Nov 07 2000 - 21:00:19 EST