All of lore.kernel.org
 help / color / mirror / Atom feed
From: Greg KH <gregkh@suse.de>
To: Per Liden <per@fukt.bth.se>
Cc: linux-hotplug-devel@lists.sourceforge.net, linux-kernel@vger.kernel.org
Subject: Re: [ANNOUNCE] hotplug-ng 002 release
Date: Wed, 11 May 2005 05:36:23 +0000	[thread overview]
Message-ID: <20050511053622.GD8287@suse.de> (raw)
In-Reply-To: <Pine.LNX.4.63.0505101843050.2271@1-1-2-5a.f.sth.bostream.se>

On Tue, May 10, 2005 at 11:51:04PM +0200, Per Liden wrote:
> On Mon, 9 May 2005, Greg KH wrote:
> 
> > On Mon, May 09, 2005 at 12:52:02AM +0200, Per Liden wrote:
> > > On Fri, 6 May 2005, Greg KH wrote:
> > > 
> > > [...]
> > > > Now, with the 2.6.12-rc3 kernel, and a patch for module-init-tools, the
> > > > USB hotplug program can be written with a simple one line shell script:
> > > > 	modprobe $MODALIAS
> > > 
> > > Nice, but why not just convert all this to a call to 
> > > request_module($MODALIAS)? Seems to me like the natural thing to do.
> > 
> > Because that's not the only thing that the hotplug event causes to
> > happen. It's easier to have userspace decide what to do with this 
> > instead.
> 
> Sure, the hotplug event could still be issued so that userspace could do 
> magic things when it wants to (load firmware or whatever), but since the 
> kernel already has all the infrastructure in place to load modules on 
> demand, and it's used all over the place, it doesn't make sense to use a 
> completely different approach here.

Well, do you really want your kernel to be creating 2 userspace programs
for every device in sysfs that is created?  Remember, we can't just not
emit that hotplug event, too many programs need it....

> Also, since most people never need to do anything except modprobe, they 
> can still have a working system without any scripts what so ever... again 
> just like normal on demand module loading.

Lots of programs do lots of stuff other than modprobe on hotplug events.
Look at HAL for just one example.

thanks,

greg k-h


-------------------------------------------------------
This SF.Net email is sponsored by Oracle Space Sweepstakes
Want to be the first software developer in space?
Enter now for the Oracle Space Sweepstakes!
http://ads.osdn.com/?ad_ids93&alloc_id\x16281&op=click
_______________________________________________
Linux-hotplug-devel mailing list  http://linux-hotplug.sourceforge.net
Linux-hotplug-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-hotplug-devel

WARNING: multiple messages have this Message-ID (diff)
From: Greg KH <gregkh@suse.de>
To: Per Liden <per@fukt.bth.se>
Cc: linux-hotplug-devel@lists.sourceforge.net, linux-kernel@vger.kernel.org
Subject: Re: [ANNOUNCE] hotplug-ng 002 release
Date: Tue, 10 May 2005 22:36:23 -0700	[thread overview]
Message-ID: <20050511053622.GD8287@suse.de> (raw)
In-Reply-To: <Pine.LNX.4.63.0505101843050.2271@1-1-2-5a.f.sth.bostream.se>

On Tue, May 10, 2005 at 11:51:04PM +0200, Per Liden wrote:
> On Mon, 9 May 2005, Greg KH wrote:
> 
> > On Mon, May 09, 2005 at 12:52:02AM +0200, Per Liden wrote:
> > > On Fri, 6 May 2005, Greg KH wrote:
> > > 
> > > [...]
> > > > Now, with the 2.6.12-rc3 kernel, and a patch for module-init-tools, the
> > > > USB hotplug program can be written with a simple one line shell script:
> > > > 	modprobe $MODALIAS
> > > 
> > > Nice, but why not just convert all this to a call to 
> > > request_module($MODALIAS)? Seems to me like the natural thing to do.
> > 
> > Because that's not the only thing that the hotplug event causes to
> > happen. It's easier to have userspace decide what to do with this 
> > instead.
> 
> Sure, the hotplug event could still be issued so that userspace could do 
> magic things when it wants to (load firmware or whatever), but since the 
> kernel already has all the infrastructure in place to load modules on 
> demand, and it's used all over the place, it doesn't make sense to use a 
> completely different approach here.

Well, do you really want your kernel to be creating 2 userspace programs
for every device in sysfs that is created?  Remember, we can't just not
emit that hotplug event, too many programs need it....

> Also, since most people never need to do anything except modprobe, they 
> can still have a working system without any scripts what so ever... again 
> just like normal on demand module loading.

Lots of programs do lots of stuff other than modprobe on hotplug events.
Look at HAL for just one example.

thanks,

greg k-h

  reply	other threads:[~2005-05-11  5:36 UTC|newest]

Thread overview: 116+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-05-06 21:22 [ANNOUNCE] hotplug-ng 002 release Greg KH
2005-05-06 21:22 ` Greg KH
2005-05-08 22:52 ` Per Liden
2005-05-08 22:52   ` Per Liden
2005-05-09 21:13   ` Per Svennerbrandt
2005-05-09 21:13     ` Per Svennerbrandt
2005-05-10 22:17     ` Per Liden
2005-05-10 22:17       ` Per Liden
2005-05-10 22:41       ` Greg KH
2005-05-10 22:41         ` Greg KH
2005-05-10 23:56         ` Per Liden
2005-05-10 23:56           ` Per Liden
2005-05-11  1:22           ` Brian Gerst
2005-05-11  1:22             ` Brian Gerst
2005-05-11  5:33           ` Greg KH
2005-05-11  5:33             ` Greg KH
2005-05-11 17:36             ` Per Liden
2005-05-11 17:41               ` Greg KH
2005-05-18 23:00       ` Per Svennerbrandt
2005-05-18 23:00         ` Per Svennerbrandt
2005-05-18 23:00       ` [PATCH][RFC] __request_module: fixed argument request_module with waitflag Per Svennerbrandt
2005-05-18 23:00         ` Per Svennerbrandt
2005-05-18 23:01       ` [PATCH][RFC] request_modalias: MODALIAS based module loading Per Svennerbrandt
2005-05-18 23:01         ` Per Svennerbrandt
2005-05-18 23:37         ` Per Svennerbrandt
2005-05-10 22:41     ` [ANNOUNCE] hotplug-ng 002 release Greg KH
2005-05-10 22:41       ` Greg KH
2005-05-12 21:42     ` Greg KH
2005-05-12 21:42       ` Greg KH
2005-05-13  8:19       ` Michael Tokarev
2005-05-13  8:19         ` Michael Tokarev
2005-05-13 16:02         ` Greg KH
2005-05-13 16:02           ` Greg KH
2005-05-13 23:21       ` Per Svennerbrandt
2005-05-13 23:21         ` Per Svennerbrandt
2005-05-14  5:59         ` Greg KH
2005-05-14  5:59           ` Greg KH
2005-05-15 22:37           ` Per Svennerbrandt
2005-05-18  9:27     ` David Weinehall
2005-05-18  9:27       ` David Weinehall
2005-05-09 23:22   ` Greg KH
2005-05-09 23:22     ` Greg KH
2005-05-10 21:51     ` Per Liden
2005-05-10 21:51       ` Per Liden
2005-05-11  5:36       ` Greg KH [this message]
2005-05-11  5:36         ` Greg KH
2005-05-09  3:57 ` Rusty Russell
2005-05-09  3:57   ` Rusty Russell
2005-05-09 23:21   ` Greg KH
2005-05-09 23:21     ` Greg KH
2005-05-10  9:29     ` Rusty Russell
2005-05-10  9:29       ` Rusty Russell
2005-05-10  9:43       ` Marco d'Itri
2005-05-10  9:43         ` Marco d'Itri
2005-05-10 12:58         ` Alexander E. Patrakov
2005-05-10 12:58           ` Alexander E. Patrakov
2005-05-10 17:24           ` Marco d'Itri
2005-05-10 17:24             ` Marco d'Itri
2005-05-10 20:13             ` Greg KH
2005-05-10 20:13               ` Greg KH
2005-05-10 20:28               ` Lee Revell
2005-05-10 20:28                 ` Lee Revell
2005-05-10 20:59                 ` Greg KH
2005-05-10 20:59                   ` Greg KH
2005-05-10 21:02                   ` Marco d'Itri
2005-05-10 21:02                     ` Marco d'Itri
2005-05-10 20:31               ` Marco d'Itri
2005-05-10 20:31                 ` Marco d'Itri
2005-05-10 20:52                 ` Greg KH
2005-05-10 20:52                   ` Greg KH
2005-05-10 20:59                   ` Bill Nottingham
2005-05-10 20:59                     ` Bill Nottingham
2005-05-10 21:08                   ` Marco d'Itri
2005-05-10 21:08                     ` Marco d'Itri
2005-05-10 21:22                     ` Erik van Konijnenburg
2005-05-10 21:22                       ` Erik van Konijnenburg
2005-05-10 23:55                       ` [PATCH] " Erik van Konijnenburg
2005-05-10 23:55                         ` Erik van Konijnenburg
2005-05-11  0:05                         ` Marco d'Itri
2005-05-11  0:05                           ` Marco d'Itri
2005-05-11  5:40                           ` Greg KH
2005-05-11  5:40                             ` Greg KH
2005-05-11  0:08                         ` [PATCH] " Rusty Russell
2005-05-11  0:08                           ` Rusty Russell
2005-05-11  1:11                           ` Erik van Konijnenburg
2005-05-11  1:11                             ` Erik van Konijnenburg
2005-05-11  3:39                             ` Rusty Russell
2005-05-11  3:39                               ` Rusty Russell
2005-05-11  9:59                               ` Erik van Konijnenburg
2005-05-11  9:59                                 ` Erik van Konijnenburg
2005-05-11 10:52                                 ` Rusty Russell
2005-05-11 10:52                                   ` Rusty Russell
2005-05-11 10:58                                   ` Marco d'Itri
2005-05-11 10:58                                     ` Marco d'Itri
2005-05-11 13:06                                     ` Erik van Konijnenburg
2005-05-11 13:06                                       ` Erik van Konijnenburg
2005-05-12  4:39                                       ` Rusty Russell
2005-05-12  4:39                                         ` Rusty Russell
2005-05-12  7:47                                         ` Erik van Konijnenburg
2005-05-12  7:47                                           ` Erik van Konijnenburg
2005-05-10 21:21               ` Giuseppe Bilotta
2005-05-11  0:01               ` Rusty Russell
2005-05-11  0:01                 ` Rusty Russell
2005-05-11  0:10                 ` Marco d'Itri
2005-05-11  0:10                   ` Marco d'Itri
2005-05-11  1:09                   ` Rusty Russell
2005-05-11  1:09                     ` Rusty Russell
2005-05-11  7:31 ` Christian Zoz
2005-05-14 23:02 ` Michael Tokarev
2005-05-14 23:02   ` Michael Tokarev
2005-05-16 19:11   ` Greg KH
2005-05-16 19:11     ` Greg KH
2005-05-16 21:24 ` Marco d'Itri
     [not found] <41iyE-8mI-11@gated-at.bofh.it>
     [not found] ` <427KM-h4-9@gated-at.bofh.it>
     [not found]   ` <42pRx-75A-19@gated-at.bofh.it>
     [not found]     ` <42znJ-6x7-25@gated-at.bofh.it>
     [not found]       ` <42zQL-70r-25@gated-at.bofh.it>
     [not found]         ` <42CF0-YV-37@gated-at.bofh.it>
     [not found]           ` <42GIH-4u3-31@gated-at.bofh.it>
     [not found]             ` <42Jn3-6Qj-5@gated-at.bofh.it>
     [not found]               ` <42KsY-7KW-33@gated-at.bofh.it>
2005-05-11  2:04                 ` Bodo Eggert <harvested.in.lkml@posting.7eggert.dyndns.org>
2005-05-11  2:04                   ` Bodo Eggert <harvested.in.lkml@posting.7eggert.dyndns.org>
2005-05-18  7:23                   ` Giuseppe Bilotta

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20050511053622.GD8287@suse.de \
    --to=gregkh@suse.de \
    --cc=linux-hotplug-devel@lists.sourceforge.net \
    --cc=linux-kernel@vger.kernel.org \
    --cc=per@fukt.bth.se \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.