All of lore.kernel.org
 help / color / mirror / Atom feed
From: Duncan Sands <baldrick@wanadoo.fr>
To: chas williams <chas@cmf.nrl.navy.mil>,
	Werner Almesberger <wa@almesberger.net>
Cc: "David S. Miller" <davem@redhat.com>, linux-kernel@vger.kernel.org
Subject: Re: [PATCH][ATM] use rtnl_{lock,unlock} during device operations (take 2)
Date: Mon, 9 Jun 2003 15:37:13 +0200	[thread overview]
Message-ID: <200306091537.13345.baldrick@wanadoo.fr> (raw)
In-Reply-To: <200306070350.h573oIsG004491@ginger.cmf.nrl.navy.mil>

> btw, you might get async device 'releases' more often than
> you would like.  some atm devices are usb and could be
> unplugged during operation (yes, that's really bad but it
> would be wise to be prepared for this.)  i actually have
> a cardbus atm interface.  i might eject it accidentally.

Right.  In the speedtouch (USB ATM DSL modem) driver, when the
device is unplugged I would have liked to be able to say to the
ATM layer: I've gone, don't call me any more.  But since there
is no way to do this, instead I do:

- refuse to open any more vccs; fail all attempts to send packets
- call shutdown_atm_dev.  This means that when the last vcc is
closed, atm_dev_close will be called in my driver.
- really shutdown in atm_dev_close

In practice that means that the vcc remains open until pppd realises
that the connection has gone down (no more echo requests getting
through, for example).  Maybe I should push a NULL skb down into
each vcc to get it to close?

Another thing: if the modem is plugged in again, and pppd relaunched,
it would be nice if connections that were open when the modem was
unplugged automagically recovered.  Is that possible?

Ciao,

Duncan.

  reply	other threads:[~2003-06-09 13:23 UTC|newest]

Thread overview: 74+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-06-05 15:28 [PATCH][ATM] use rtnl_{lock,unlock} during device operations (take 2) chas williams
2003-06-06  9:36 ` David S. Miller
2003-06-06 10:58   ` chas williams
2003-06-06 11:04     ` David S. Miller
2003-06-06 13:57       ` Werner Almesberger
2003-06-06 14:07         ` David S. Miller
2003-06-06 15:13           ` Werner Almesberger
2003-06-06 15:16             ` David S. Miller
2003-06-06 15:26               ` Werner Almesberger
2003-06-06 15:28                 ` David S. Miller
2003-06-06 15:54                   ` Werner Almesberger
2003-06-06 15:55                     ` David S. Miller
2003-06-06 16:40                       ` Werner Almesberger
2003-06-06 21:54                       ` Mitchell Blank Jr
2003-06-06 23:19                         ` Werner Almesberger
2003-06-06 23:44                           ` Mitchell Blank Jr
2003-06-07  0:44                           ` chas williams
2003-06-07  0:59                             ` Werner Almesberger
2003-06-07 11:10                           ` chas williams
2003-06-06 23:37                         ` chas williams
2003-06-17 17:52                       ` chas williams
2003-06-17 17:53                         ` David S. Miller
2003-06-06 21:43                     ` Mitchell Blank Jr
2003-06-06 22:56                       ` Werner Almesberger
2003-06-06 23:52                     ` chas williams
2003-06-07  0:20                       ` Werner Almesberger
2003-06-07  0:51                         ` chas williams
2003-06-07  1:12                           ` Werner Almesberger
2003-06-07  6:58                         ` David S. Miller
2003-06-07 19:01                           ` Roman Zippel
2003-06-08  6:57                             ` David S. Miller
2003-06-08 22:32                               ` Roman Zippel
2003-06-09  5:35                                 ` David S. Miller
2003-06-09 22:59                                   ` Roman Zippel
2003-06-09 23:00                                     ` David S. Miller
2003-06-09 23:14                                       ` Roman Zippel
2003-06-09 23:14                                         ` David S. Miller
2003-06-09 23:34                                           ` Roman Zippel
2003-06-09 23:39                                             ` David S. Miller
2003-06-10 18:27                                               ` Roman Zippel
2003-06-08  3:45                           ` Werner Almesberger
2003-06-08  6:43                             ` David S. Miller
2003-06-10 21:34                               ` Werner Almesberger
2003-06-10 22:16                                 ` David S. Miller
2003-06-06 23:58                 ` chas williams
2003-06-07  0:06                   ` Werner Almesberger
2003-06-07  0:45                     ` chas williams
2003-06-07  0:56                       ` Werner Almesberger
2003-06-07  6:59                       ` David S. Miller
2003-06-07 18:18                         ` Ryan Anderson
2003-06-07 11:19                       ` James Stevenson
2003-06-07 11:19                         ` chas williams
2003-06-07 15:36                           ` James Stevenson
2003-06-07 16:03                         ` Mr. James W. Laferriere
2003-06-07  6:53                     ` David S. Miller
2003-06-08  3:31                       ` Werner Almesberger
2003-06-06 23:55             ` chas williams
2003-06-07  0:10               ` Werner Almesberger
2003-06-07  0:56                 ` chas williams
2003-06-07  1:11                   ` Werner Almesberger
2003-06-07  3:48                     ` chas williams
2003-06-09 13:37                       ` Duncan Sands [this message]
2003-06-09 13:38                         ` David S. Miller
2003-06-09 13:58                           ` chas williams
2003-06-09 14:00                             ` David S. Miller
2003-06-09 14:54                               ` chas williams
2003-06-09 14:57                                 ` David S. Miller
2003-06-07  7:02                     ` David S. Miller
2003-06-08  4:05                       ` Werner Almesberger
2003-06-07 11:06                 ` chas williams
2003-06-06 15:05       ` chas williams
2003-06-06 15:08         ` David S. Miller
2003-06-06 17:03         ` Werner Almesberger
2003-06-06 11:25     ` David Anderson

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=200306091537.13345.baldrick@wanadoo.fr \
    --to=baldrick@wanadoo.fr \
    --cc=chas@cmf.nrl.navy.mil \
    --cc=davem@redhat.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=wa@almesberger.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.