linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Greg KH <greg@kroah.com>
To: David Brownell <david-b@pacbell.net>
Cc: linux-usb-devel@lists.sourceforge.net,
	linux-kernel@vger.kernel.org, Patrick Mochel <mochel@osdl.org>
Subject: Re: [linux-usb-devel] [RFC] consolidate /sbin/hotplug call for pci and usb
Date: Wed, 25 Sep 2002 21:27:15 -0700	[thread overview]
Message-ID: <20020926042715.GB1790@kroah.com> (raw)
In-Reply-To: <3D92749F.9050504@pacbell.net>

On Wed, Sep 25, 2002 at 07:44:47PM -0700, David Brownell wrote:
> 
> >I also found the unload USB module problem.  The driver core was calling
> >hotplug after the device was already removed.  Made it a bit difficult
> >to be able to describe the device that way :)
> 
> On the other hand, since there's no internal "this is an ex-device"
> state, that's also insurance that nothing could use "usbfs" to try
> to re-activate the device.  I seem to recall oopses going away by
> reporting the hotplug "remove" events after the usbfs path could
> no longer be used.  Not that I ever liked that consequence, but
> a fix adding such a "zombie" state would have taken a bit of time.

Yes, Pat and I have talked a lot about the need for a driver "state".  I
think the current goal was to see how far we can get without needing it.
I was certainly cursing the lack of it today when trying to debug this
problem, but in the end, having it would have only masked over the
real problem that was there.

So personally, I keep going back and forth on if it is really necessary
or not to have.  Right now, the USB drivers and developers are very used
to the fact that when the device disappears from the system, they can
not access it anymore, and that this needs to be constantly checked.  I
think that as time goes on, and more subsystems become "hot-pluggable"
either this paranoia will have to spread to the other subsystems, or we
will have to create the notion of a device "state" to make things
easier on everyone.

> The real "module unload problem" has a lot to do with not having any
> way to track how many devices a module is bound to ... that aren't
> necessarily opened at the moment.  (Does Rusty's patch set touch
> any of that?)

As far as I know, it doesn't, but I'm not sure.

> Without having a way to answer that question, today's un-helpful
> "driver is in active use" refcount would encourage rmmodding drivers
> that users will expect to still be available.  Plug in two devices,
> look at one, decide to use the other, unplug the first ... and just
> because you hadn't yet opened the second device, its driver module
> vanishes.  As you start to use it ... huge frustration quotient! :)

Well, that's a driver unload issue, which I think everyone agrees on the
fact that it's not ok to do automatic driver unload when a device is
removed, because of this very problem.

thanks,

greg k-h

  reply	other threads:[~2002-09-26  4:23 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-09-25 21:29 [RFC] consolidate /sbin/hotplug call for pci and usb Greg KH
2002-09-25 22:04 ` Kai Germaschewski
2002-09-25 22:48   ` Greg KH
2002-09-26  0:11 ` [linux-usb-devel] " David Brownell
2002-09-26  0:25   ` Greg KH
2002-09-26  2:44     ` David Brownell
2002-09-26  4:27       ` Greg KH [this message]
2002-09-26 16:14         ` David Brownell
2002-09-26 18:43           ` Greg KH
2002-09-26 19:32             ` David Brownell
2002-09-26 19:34             ` Alan Stern
2002-09-26 23:35               ` [linux-usb-devel] [RFC] consolidate /sbin/hotplug call for pciand usb Oliver Neukum
2002-09-26 17:48 ` [RFC] consolidate /sbin/hotplug call for pci and usb - take 2 Greg KH
     [not found] <20020925212955.GA32487@kroah.com.suse.lists.linux.kernel>
     [not found] ` <3D9250CD.7090409@pacbell.net.suse.lists.linux.kernel>
2002-09-26  0:33   ` [linux-usb-devel] [RFC] consolidate /sbin/hotplug call for pci and usb Andi Kleen
2002-09-26  0:46     ` Matthew Dharm
2002-09-26  1:01       ` Andi Kleen
2002-09-26  2:30       ` 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=20020926042715.GB1790@kroah.com \
    --to=greg@kroah.com \
    --cc=david-b@pacbell.net \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-usb-devel@lists.sourceforge.net \
    --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;
as well as URLs for NNTP newsgroup(s).