mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Alistair John Strachan <s0348365@sms.ed.ac.uk>
To: "Giampaolo Tomassoni" <g.tomassoni@libero.it>
Cc: linux-kernel@vger.kernel.org, linux-atm-general@lists.sourceforge.net
Subject: Re: [ATMSAR] Request for review - update #1
Date: Sun, 4 Sep 2005 17:20:55 +0100	[thread overview]
Message-ID: <200509041720.55588.s0348365@sms.ed.ac.uk> (raw)
In-Reply-To: <NBBBIHMOBLOHKCGIMJMDGEHPEKAA.g.tomassoni@libero.it>

On Sunday 04 September 2005 12:05, Giampaolo Tomassoni wrote:
> Dears,
>
> thanks to Jiri Slaby who found a bug in the AAL0 handling of the ATMSAR
> module.
>
> I attach a fixed version of the atmsar patch as a diff against the 2.6.13
> kernel tree.
>
[snip]

Just out of curiosity, is there ANY reason why this has to be done in the 
kernel? The PPPoATM module for pppd implements (via linux-atm) a completely 
userspace ATM decoder.. if anything, now redundant ATM stack code should be 
REMOVED from Linux!

Most distributions (to my knowledge) supporting the speedtouch modem do so 
using the method prescribed on speedtouch.sf.net; an entirely userspace 
procedure. pppd does all the ATM magic.

Does this have real-world applications beyond the Speedtouch DSL modems? If 
not, I propose adding this code to linux-atm, not the kernel, since most 
users of USB speedtouch DSL modems will not be using the kernel's ATM.

-- 
Cheers,
Alistair.

'No sense being pessimistic, it probably wouldn't work anyway.'
Third year Computer Science undergraduate.
1F2 55 South Clerk Street, Edinburgh, UK.

  parent reply	other threads:[~2005-09-04 16:20 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-09-04 11:05 Giampaolo Tomassoni
2005-09-04 12:00 ` [Linux-ATM-General] " Francois Romieu
2005-09-04 13:13   ` R: " Giampaolo Tomassoni
2005-09-04 15:32     ` Francois Romieu
2005-09-04 18:44       ` R: " Giampaolo Tomassoni
2005-09-04 19:11         ` matthieu castet
2005-09-04 19:33           ` R: " Giampaolo Tomassoni
2005-09-04 19:42           ` Giampaolo Tomassoni
2005-09-05  6:05     ` Zoran Stojsavljevic
2005-09-05  6:21       ` R: " Giampaolo Tomassoni
2005-09-04 16:20 ` Alistair John Strachan [this message]
2005-09-04 16:41   ` Grzegorz Kulewski
2005-09-04 16:54     ` Alistair John Strachan
2005-09-04 17:22       ` Grzegorz Kulewski
2005-09-05 15:04       ` Duncan Sands
2005-09-04 18:36   ` R: " Giampaolo Tomassoni
2005-09-04 18:51     ` Alistair John Strachan
2005-09-05  9:36   ` David Woodhouse
2005-09-05 13:52     ` Alistair John Strachan
2005-09-05 13:56       ` David Woodhouse
2005-09-05 14:18         ` Alistair John Strachan
2005-09-05 14:32           ` David Woodhouse
2005-09-05 14:46   ` Duncan Sands
2005-09-04 16:55 ` Jiri Slaby

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=200509041720.55588.s0348365@sms.ed.ac.uk \
    --to=s0348365@sms.ed.ac.uk \
    --cc=g.tomassoni@libero.it \
    --cc=linux-atm-general@lists.sourceforge.net \
    --cc=linux-kernel@vger.kernel.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

all inboxes | Powered by JetHome®