public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Greg KH <greg@kroah.com>
To: "Kevin P. Fleming" <kpfleming@cox.net>
Cc: linux-hotplug-devel@lists.sourceforge.net,
	linux-kernel@vger.kernel.org, message-bus-list@redhat.com
Subject: Re: [ANNOUNCE] udev 0.1 release
Date: Fri, 11 Apr 2003 12:07:17 -0700	[thread overview]
Message-ID: <20030411190717.GH1821@kroah.com> (raw)
In-Reply-To: <3E970A00.2050204@cox.net>

On Fri, Apr 11, 2003 at 11:31:28AM -0700, Kevin P. Fleming wrote:
> 
> OK, this is all fun and games, but this is a valid point. All it takes for 
> the driver for a Fibre Channel host adapter to load, and enumerate the 
> devices it can see. In a matter of seconds many hundreds or thousands of 
> disk devices could be registered with the kernel.

Sure, the kernel can handle spawning hundreds or thousands of tasks all
at once, it's not a problem.

> This is definitely an issue that will need to be addressed, and I think 
> Oliver's suggestion of using a pipe (i'm going to say it: like devfs did 
> :-) to forward the events to /sbin/hotplug in a FIFO fashion makes some 
> sense.

I agree too.  Having /sbin/hotplug send events to a pipe where a daemon
can get them from makes a lot of sense.  It will handle most of the
synchronization and spawning a zillion tasks problems.

> I have also been considering this issue from another angle; I am 
> working on userspace partition discovery, which will be driven by 
> /sbin/hotplug (and udev, probably). I have concerns that the following 
> scenario will cause problems, if not extreme problems:
> 
> - kernel driver finds an IDE drive, registers it and the hotplug event 
> happens
> - udev gets called and gives it device node /dev/discs/disc0 (or whatever)
> - /sbin/hotplug calls userspace partition discovery, which opens the device 
> and scans for partitions
> - if any partitions are found, they are registered with the kernel using 
> device-mapper ioctls
> - because these new "mapped sections" of the drive are _also_ usable block 
> devices in their own right, they generate hotplug events
> - because these hotplug events are for new block devices, userspace 
> partition discovery will get called _again_ to handle them (it may not find 
> anything (the normal case), but this model will support nearly infinite 
> levels of partitioning on any block device supported by the kernel)
> 
> What happens if these secondary hotplug events occur while /sbin/hotplug 
> has not yet finished processing the first one? Ignoring locking/race issues 
> for the moment, I'm concerned about memory consumption as many layers of 
> hotplug/udev/kpartx/etc. are running processing these events.

Yes, this can quickly get recursive up to a point.  There will never be
an infinite number of partitions, so we will eventually quiet down.

> Of course, another possibility I'll look into this weekend is to actually 
> have kpartx run as a daemon and receive messages over D-BUS, instead of 
> being invoked directly by /sbin/hotplug. This would mean it could serialize 
> the events itself and reduce some of the load (if D-BUS supports message 
> queueing, which I believe it does).

Problem is I don't think we can use D-BUS messages during early boot,
before init is called, so we still have to be able to handle startup
issues.  But hopefully the D-BUS code can be small enough to possibly be
used in this manner, I haven't checked that out yet.

> Actually, here's another thought: have the kernel continue to call 
> /sbin/hotplug for every event, just as it does now. However, /sbin/hotplug 
> would do _nothing_ but translate that into D-BUS messages and post them. 
> udev, kpartx, etc. would all just be D-BUS clients that would respond to 
> their messages as they are received.

That's another possibility too.  This is getting interesting :)

thanks,

greg k-h

  reply	other threads:[~2003-04-11 18:53 UTC|newest]

Thread overview: 107+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-04-11  3:24 [ANNOUNCE] udev 0.1 release Greg KH
2003-04-11  6:37 ` Oliver Neukum
2003-04-11 17:20   ` Greg KH
2003-04-11 17:46     ` John Bradford
2003-04-11 18:03       ` Michael Buesch
2003-04-11 18:12         ` Nicholas Berry
2003-04-11 18:41           ` Eric Weigle
2003-04-11 21:54           ` Alex Bligh - linux-kernel
2003-04-11 18:23       ` Antonio Vargas
2003-04-11 18:31         ` Kevin P. Fleming
2003-04-11 19:07           ` Greg KH [this message]
2003-04-11 19:29             ` Havoc Pennington
2003-04-12  8:07               ` Greg KH
2003-04-12 23:27                 ` Havoc Pennington
2003-04-11 23:39             ` Miquel van Smoorenburg
2003-04-12  0:08               ` Greg KH
2003-04-12  0:21                 ` Miquel van Smoorenburg
2003-04-12  8:40                   ` Oliver Neukum
2003-04-12  8:52                     ` Andrew Morton
2003-04-11 19:28           ` Joel Becker
2003-04-11 19:38             ` Kevin P. Fleming
2003-04-11 23:26               ` Joel Becker
2003-04-11 19:58             ` Greg KH
2003-04-11 23:25               ` Joel Becker
2003-04-11 23:37                 ` Greg KH
2003-04-12  0:19                   ` Joel Becker
2003-04-12  1:06                     ` H. Peter Anvin
2003-04-12  4:43                       ` Greg KH
2003-04-12 12:56                       ` Roman Zippel
2003-04-12  4:20                     ` Greg KH
2003-04-11 20:29           ` Steven Dake
2003-04-11 20:43             ` Greg KH
2003-04-11 22:30               ` Steven Dake
2003-04-11 22:38                 ` Lars Marowsky-Bree
2003-04-11 22:43                   ` Steven Dake
2003-04-11 22:58                     ` Greg KH
2003-04-11 23:32                       ` Steven Dake
2003-04-11 23:45                         ` Greg KH
2003-04-11 22:56                 ` Greg KH
2003-04-11 23:31                   ` Steven Dake
2003-04-12  6:45                     ` Lars Marowsky-Bree
2003-04-12 14:45                 ` Alan Cox
2003-04-11 22:09             ` Andrew Morton
2003-04-11 22:19               ` Tim Hockin
2003-04-11 22:47                 ` Andrew Morton
2003-04-11 23:03                   ` Greg KH
2003-04-12  8:04                   ` Oliver Neukum
2003-04-11 23:01               ` Greg KH
2003-04-11 23:23                 ` Andrew Morton
2003-04-11 23:35                   ` Greg KH
2003-04-11 23:37                 ` Steven Dake
2003-04-12  7:49                 ` Oliver Neukum
2003-04-19  4:39               ` David Brownell
2003-04-19  4:16           ` David Brownell
2003-04-11 18:30       ` Oliver Neukum
2003-04-11 19:00         ` Oliver Neukum
2003-04-11 19:09       ` Mike Dresser
2003-04-11 19:54         ` Richard B. Johnson
2003-04-11 19:59           ` Mike Dresser
2003-04-11 20:16             ` John Bradford
2003-04-11 20:16               ` Mike Dresser
2003-04-11 20:23                 ` Chris Hanson
2003-04-11 20:32                   ` Mike Dresser
2003-04-11 20:47                     ` Richard B. Johnson
2003-04-11 20:39               ` Richard B. Johnson
2003-04-11 22:03                 ` Alex Bligh - linux-kernel
2003-04-11 22:00             ` Alex Bligh - linux-kernel
2003-04-11 21:28         ` Martin Mares
2003-04-11 18:12     ` Oliver Neukum
2003-04-11 18:52       ` Greg KH
2003-04-11 19:31         ` Oliver Neukum
2003-04-11 20:10           ` Greg KH
2003-04-11 20:56             ` Oliver Neukum
2003-04-11 21:03             ` Oliver Neukum
2003-04-11 22:27             ` Perez-Gonzalez, Inaky
2003-04-11 22:53               ` Greg KH
2003-04-11 17:10 ` Jeremy Jackson
2003-04-11 17:18   ` Justin Cormack
2003-04-11 17:21   ` Greg KH
2003-04-11 18:02 ` Roman Zippel
2003-04-11 18:12   ` Greg KH
2003-04-11 20:09     ` Nick Craig-Wood
     [not found] <200304112018.11931.freesoftwaredeveloper@web.de>
2003-04-11 18:50 ` Steve Lee
2003-04-11 19:09   ` Michael Buesch
  -- strict thread matches above, loose matches on Subject: below --
2003-04-11 20:42 Perez-Gonzalez, Inaky
2003-04-11 20:48 ` David Lang
2003-04-11 20:59   ` Greg KH
2003-04-11 22:32     ` Steven Dake
2003-04-11 22:41       ` David Lang
2003-04-11 22:51       ` Greg KH
2003-04-11 23:27         ` Steven Dake
2003-04-11 23:32           ` Greg KH
2003-04-11 23:39             ` Steven Dake
2003-04-12  0:04     ` Joel Becker
     [not found] <20030411173018$2695@gated-at.bofh.it>
     [not found] ` <20030411175011$3d7e@gated-at.bofh.it>
     [not found]   ` <20030411182022$7f7a@gated-at.bofh.it>
     [not found]     ` <20030411184016$1180@gated-at.bofh.it>
     [not found]       ` <20030411204006$0496@gated-at.bofh.it>
     [not found]         ` <20030411205018$7440@gated-at.bofh.it>
2003-04-11 21:09           ` Arnd Bergmann
2003-04-11 21:57             ` Greg KH
2003-04-11 22:12               ` Arnd Bergmann
2003-04-12  7:39               ` John Bradford
2003-04-11 22:35             ` Steven Dake
2003-04-11 23:05               ` Greg KH
2003-04-11 23:30                 ` Arnd Bergmann
2003-04-11 22:36 Perez-Gonzalez, Inaky
2003-04-12  7:53 ` Oliver Neukum
2003-04-11 22:42 Perez-Gonzalez, Inaky
2003-04-11 22:59 Perez-Gonzalez, Inaky
2003-04-12  4:22 Perez-Gonzalez, Inaky
2003-04-12 12:18 Arnd Bergmann

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=20030411190717.GH1821@kroah.com \
    --to=greg@kroah.com \
    --cc=kpfleming@cox.net \
    --cc=linux-hotplug-devel@lists.sourceforge.net \
    --cc=linux-kernel@vger.kernel.org \
    --cc=message-bus-list@redhat.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox