On Friday 04 April 2008 02:14:44 Randy Dunlap wrote:
quoted text
> On Thu, 3 Apr 2008 12:04:25 -0400 Tom Horsley wrote:
> > I know that part, I'm just wondering if it would be nice to have
> > even dynamically loaded modules notice the kernel option
> > so that the "usbcore" module would see the "blinkenlights=1"
> > parameter from the kernel boot options when dynamically loaded?
> >
> > That way, you wouldn't have to stop and figure out if a module
> > was static or dynamic to decide if you need to edit the grub boot
> > line or the modprobe.conf file.
>
> OK, that makes some sense to me.
>
> Rusty added to cc for his objective viewpoint.

I agree with Randy; it has some appeal, but it's not a killer feature.

We already do the reverse: sysfs exposed parameters are the same for modules 
and builtins.  If they're modifiable (most aren't), you don't have to care 
whether it's a module or not.

But there are also minor downsides.  The first is that we currently warn about 
unused parameters (at least those containing '.').  We'd have to come up with 
something cleverer if we want to warn only about parameters which cannot be 
used by modules (ie. do it in userspace).

The second is that users might be surprised when they take 
the 'usbcore.blinkenlights' line out of their modprobe config file and it 
still applies.  Most users use everything-is-a-module distributions.

Finally, it would be logical to take all the module parameters and autogen 
them into the kernel commandline, but it will soon hit cmdline limits (hmm, 
actually on my Ubuntu Hardy system here I think I'd be OK).

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