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/
next 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