| To: | "David S. Miller" <davem@xxxxxxxxxxxxx> |
|---|---|
| Subject: | Re: pppoe and receive checksum offload |
| From: | shemminger@xxxxxxxx |
| Date: | Mon, 28 Feb 2005 16:01:33 -0800 (PST) |
| Cc: | "Stephen Hemminger" <shemminger@xxxxxxxx>, mostrows@xxxxxxxxxxxxx, kuznet@xxxxxxxxxxxxx, netdev@xxxxxxxxxxx |
| Importance: | Normal |
| In-reply-to: | <20050228153228.0c5cb46d.davem@davemloft.net> |
| References: | <20050224155906.73890361@dxpl.pdx.osdl.net><20050227202011.5ccefb22.davem@davemloft.net><42235114.3070109@osdl.org> <20050228153228.0c5cb46d.davem@davemloft.net> |
| Sender: | netdev-bounce@xxxxxxxxxxx |
| User-agent: | SquirrelMail/1.4.2-1_osdl_00 |
> On Mon, 28 Feb 2005 09:12:52 -0800 > Stephen Hemminger <shemminger@xxxxxxxx> wrote: > >> The original sk98lin attempted to receive hardware checksumming >> but never actually turned it on. > > Because there is a bug in the chip wherein checksums can be > miscalculated. I forget the details, but I do remember that > you can't enable checksumming safely because of this. > > In drivers/net/sk98lin/skcsum.c it mentions this: > > * Note: > * There is a bug in the GENESIS ASIC which may lead to wrong > checksums. > > I know this comment is above the send checksum routine, but I > am pretty sure the bug applies to both directions. The driver covers two types of chips (Genesis and Yukon). The new driver only allows receive checksum on Yukon chipset. Almost all current hardware uses Yukon and it has been pretty well tested. The issues the sk98lin had were sloppy coding and handling of unsigned arithmetic. |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: pppoe and receive checksum offload, David S. Miller |
|---|---|
| Next by Date: | [PATCH] mv643xx: permit VLAN tagged rx packets + minor cleanup, Dale Farnsworth |
| Previous by Thread: | Re: pppoe and receive checksum offload, David S. Miller |
| Next by Thread: | Re: pppoe and receive checksum offload, Herbert Xu |
| Indexes: | [Date] [Thread] [Top] [All Lists] |