netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "David S. Miller" <davem@redhat.com>
To: Russell King <rmk@arm.linux.org.uk>
Cc: jgarzik@pobox.com, shemminger@osdl.org, netdev@oss.sgi.com,
	greg@kroah.com
Subject: Re: [Fwd: pcmcia ether drivers can't be unloaded]
Date: Tue, 27 Jul 2004 17:19:29 -0700	[thread overview]
Message-ID: <20040727171929.17858c7b.davem@redhat.com> (raw)
In-Reply-To: <20040727233614.B30782@flint.arm.linux.org.uk>

On Tue, 27 Jul 2004 23:36:14 +0100
Russell King <rmk@arm.linux.org.uk> wrote:

> > One fix is to not set owner field but then there is a hot plug/module
> > remove race. But the right fix seems to fix up pcmcia to be a true bus
> > in the driver model and have the same hotplug as other buses; usb and
> > pci don't have the problem.
> 
> No, the right fix is not to try to fsck with PCMCIA refcounting - it
> isn't up to having drivers randomly unloaded.  IOW, it remains 2.4
> behaviour.

I totally disagree.  This is a bogus argument for two reasons:

1) If the PCMCIA layer can handle cards popping out at any time,
   physically, it is illogical to handicap it on the software
   side in terms of this.

2) There is no way I can stomache "some" networking cards not being
   able to have their modules yanked at any point in time.

   An enormous amount of effort has gone into making networking
   devices and drivers yankable regardless of how configured they
   are and what resources are attached to it.

You have to be kidding me to say that the one subsystem that should
handle hot plugging of stuff the best (PCMCIA) can't handle something
that all of our "normal" PCI and USB drivers can?

The fact is this, device unload of an arbitrary network driver should
work at any point in time, and succeed unloading within a second or
two.  Anything else is a bug.

  reply	other threads:[~2004-07-28  0:19 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <41068BEF.7010200@pobox.com>
2004-07-27 22:36 ` [Fwd: pcmcia ether drivers can't be unloaded] Russell King
2004-07-28  0:19   ` David S. Miller [this message]
2004-07-28 15:50     ` Russell King
2004-07-28 15:54       ` David S. Miller
2004-08-02 19:02         ` Russell King
2004-08-07 11:09         ` Russell King

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=20040727171929.17858c7b.davem@redhat.com \
    --to=davem@redhat.com \
    --cc=greg@kroah.com \
    --cc=jgarzik@pobox.com \
    --cc=netdev@oss.sgi.com \
    --cc=rmk@arm.linux.org.uk \
    --cc=shemminger@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).