Re: [PATCH 06/13] USBATM: shutdown open connections when disconnected

From: Duncan Sands
Date: Mon Jun 19 2006 - 04:42:52 EST


Hi Juampe,

On Monday 19 June 2006 10:13, juampe wrote:
> > This patch causes vcc_release_async to be applied to any *open
> >** v*cc's when the modem is *disconnected*.
>
>
> > Unfortunately this patch may create problems
> > for those rare users like me who use routed IP or some other
> > non-ppp connection method that goes via the ATM ARP daemon: the
> > daemon is buggy, and with this patch will crash when the modem
> > is *disconnected*. Users with a buggy atmarpd can simply restart
> > it after disconnecting the modem.
>
> First i must thanks all effort in usbatm development.
> IMHO New fatures to a driver that works well and can block the use,
> especially if it can disable internet access and the problem is know,
> MUST be disabled by default or provide a mechanism to disable it.
>
> I'm a rare user with routed IP and this patch blocks the normal use of internet
> I dont understand how this patch can be accepted for a stable release without
> any kind of disable mechanism.
>
> Yeah, i know that atmarp is buggy, but before speedtouch driver and atm works well during months.

why don't you just restart atmarpd? After all, if you are unplugging and
replugging your modem, surely you can restart the daemon at the same time?

I didn't feel it was necessary to have an override mechanism for this new,
correct behavior (which makes things better for people using pppd, i.e. most
people) since a simple workaround exists.

What is very bad however is that atmarpd is still not fixed. I had a look,
got stuck, and asked for help on the linux-atm list, but no-one replied.
There has been no progress since then. I will have another look - maybe
you can too?

Best wishes,

Duncan.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/