From: Ben Collins <bcollins@debian.org>
To: Patrick Mochel <mochel@osdl.org>
Cc: Greg KH <greg@kroah.com>, linux-kernel@vger.kernel.org
Subject: Re: [RFC] [PATCH] Device removal callback
Date: Mon, 10 Mar 2003 11:55:48 -0500 [thread overview]
Message-ID: <20030310165548.GA753@phunnypharm.org> (raw)
In-Reply-To: <Pine.LNX.4.33.0303100949490.1002-100000@localhost.localdomain>
> I much prefer this, as I would like to see it eventually, but I'd rather
> see the implications worked out before it's generalized.
Then I have to be concerned about parts of the driver model removing
parents of my devices without my knowing it. Didn't PCI already go
through this problem with bus's being removed?
If my PCI devices gets removed, it simply calls my PCI callbacks, but
then my PCI drivers have to link into the core and call remove on all
the host devices, then node devices, then unit directories. All this has
to happen manually, and it puts the burden all the way down the tree,
when it should remain only in the bus.
It also does not help the case where something emulates an IEEE-1394
node on the locally handled bus. If it creates a node, and then behind
that, creates unit directories, and then attaches some other sort of
children unknown to the ieee1394 core. There's no possible way that
device can safely be removed by the ieee1394 core. So then I have to
export all sorts of extra functionality to provide the same thing this
2 line callback can do.
I'm not sure what the problem is in allowing the bus driver to know when
a device is about to be removed for some reason. At the very least it
makes for a good sanity check mechanism.
--
Debian - http://www.debian.org/
Linux 1394 - http://www.linux1394.org/
Subversion - http://subversion.tigris.org/
Deqo - http://www.deqo.com/
next prev parent reply other threads:[~2003-03-10 16:45 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2003-03-09 18:14 [RFC] [PATCH] Device removal callback Ben Collins
2003-03-10 0:11 ` Greg KH
2003-03-10 1:02 ` Ben Collins
2003-03-10 15:59 ` Patrick Mochel
2003-03-10 16:55 ` Ben Collins [this message]
2003-03-10 17:21 ` Greg KH
2003-03-10 18:12 ` Ben Collins
2003-03-10 15:45 ` Patrick Mochel
2003-03-10 16:30 ` Ben Collins
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=20030310165548.GA753@phunnypharm.org \
--to=bcollins@debian.org \
--cc=greg@kroah.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mochel@osdl.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