linux-hotplug.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: stamit <ewstam@softhome.net>
To: linux-hotplug@vger.kernel.org
Subject: Unloading drivers, start-up, shut-down and some clean-ups (a problem)
Date: Tue, 02 Oct 2001 02:00:00 +0000	[thread overview]
Message-ID: <marc-linux-hotplug-100206809619773@msgid-missing> (raw)

[-- Attachment #1: Type: text/plain, Size: 1489 bytes --]

(please don't hate me for this... :-)

There are two obvious problems with the previous patch...

1. From the beginning  I tried to verbosely comment the scripts (so they 
can be self-explanatory and easy to read) and, apparently, I forgot to 
update the comments in the "uhci.agent" after reorganizing the scripts 
(to experiment with), so... the comments in /etc/hotplug/pci/uhci.agent 
are wrong, misleading and confusing (ouch)!

2. Only after posting that previous patch I realised that the way in 
which the "start" and "stop" events are passed down from policy agents 
to driver agents was, well, not so good. "start" was passed if a kernel 
module was already loaded and "stop" was passed if a module was already 
unloaded. Well, it turns out that the right thing is to handle 
start/stop in the same (almost) way that add/remove events are handled. 
The only difference is that, for start/stop, $MODULE is also exported 
and is the name of the currently loaded kernel module ("-" if none, "#" 
if undetermined).

Now, the final thing that remains for unloading to be complete is 
reference counting. I tried to do it with bash (by storing counts in 
*.ref files in the driver directories) and guess what I stumbled to... a 
race condition! Multiple hotplug events can be processed by the scripts 
at the same time. So I took it back. Isn't there a way to use modutils 
programs to play with the module's real refrence counts? It would make 
things a whole lot easier (and safer).

[-- Attachment #2: hotplug-2001_04_24-patch.diff.gz --]
[-- Type: application/x-gzip, Size: 33259 bytes --]

             reply	other threads:[~2001-10-02  2:00 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2001-10-02  2:00 stamit [this message]
2001-10-03  1:41 ` Unloading drivers, start-up, shut-down and some clean-ups (a problem) Keith Owens

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=marc-linux-hotplug-100206809619773@msgid-missing \
    --to=ewstam@softhome.net \
    --cc=linux-hotplug@vger.kernel.org \
    /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;
as well as URLs for NNTP newsgroup(s).