public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Andrew Morton <andrewm@uow.edu.au>
To: David Brownell <david-b@pacbell.net>,
	lkml <linux-kernel@vger.kernel.org>,
	"linux-usb-devel@lists.sourceforge.net" 
	<linux-usb-devel@lists.sourceforge.net>
Subject: hotplug mopup
Date: Sun, 10 Dec 2000 23:31:47 +1100	[thread overview]
Message-ID: <3A3377B3.FDCBE4AD@uow.edu.au> (raw)


A compendium of questions and misc stuff concerning hotplug:

- Is everyone happy with call_usermodehelper() being asynchronous? It
  _could_ be given a `synchronous' option, but that's a fair bit of
  obfuscation and it does expose us to deadlocks if the caller has any
  semaphores held.

- One person who definitely needs synchronous semantics is
  drivers/net/hamradio/baycom_epp.c.  The poor thing is currently doing
  the kernel_thread()/waitpid() stuff itself.  So there's a datum.

  hmm..  It's running a usermode app with the rtnl lock held. 
  There's a counter-datum.

- The three USB netdevice drivers look a bit racey in the probe()
  function.  Some can be fixed inline, but it may be better to just
  call dev_probe_lock()/dev_probe_unlock() in
  usb_find_interface_driver().  That's if dev_probe_lock() makes its
  way to kernel.org...

- On the unregister/removal path, the netdevice layer ensures that
  the interface is removed from the kernel namespace prior to launching
  `/sbin/hotplug net unregister eth0'.

  This means that when handling netdevice unregistration
  /sbin/hotplug cannot and must not attempt to do anything with eth0!
  Generally it'll fail to find an interface with this name.  If it does
  find eth0, it'll be the wrong one due to a race.

- I don't think we can say that the kernel hotplug interface is
  complete until we have real, working, tested userspace tools.  David,
  could you please summarise the state of play here? In particular,
  what still needs to be done?
-
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/

             reply	other threads:[~2000-12-10 12:58 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2000-12-10 12:31 Andrew Morton [this message]
2000-12-10 12:58 ` hotplug mopup Jeff Garzik
2000-12-10 13:25   ` Andrew Morton
2000-12-10 15:10 ` Marcus Meissner
2000-12-11  2:36   ` Albert D. Cahalan
2000-12-11  7:04     ` Andrew Morton
2000-12-10 16:35 ` David Brownell
  -- strict thread matches above, loose matches on Subject: below --
2000-12-11 21:10 David Brownell

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=3A3377B3.FDCBE4AD@uow.edu.au \
    --to=andrewm@uow.edu.au \
    --cc=david-b@pacbell.net \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-usb-devel@lists.sourceforge.net \
    /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