Re: [PATCH] Kmod optimization

[Greg KH <gregkh@suse.de>]

On Thu, Apr 13, 2006 at 08:36:17PM +0200, tyler@agat.net wrote:
> On Thu, Apr 13, 2006 at 11:24:01AM -0700, Greg KH wrote:
> > On Thu, Apr 13, 2006 at 08:03:45PM +0200, tyler@agat.net wrote:
> > > Hi,
> > > 
> > > the request_mod functions try to load automatically a module by running
> > > a user mode process helper (modprobe).
> > > 
> > > The user process is launched even if the module is already loaded. I
> > > think it would be better to test if the module is already loaded.
> > 
> > Does this cause a problem somehow?  request_mod is called _very_
> > infrequently from a normal kernel these days, so I really don't think
> > this is necessary.
> 
> Yes I agree it _should_ be very infrequently called but it _will_ be very
> infrequently called just if the user space configuration is done properly.

What do you mean by this?  Almost all 2.6 distros use udev today, which
prevents this code from ever getting called.  So odds are, you are
optimising something that no one will ever use :)

> I personnaly think we shouldn't trust the configuration and the way the
> different modules are loaded.

What way different modules are loaded?  What problem are you referring
to here?

thanks,

greg k-h