Title: [7675] branches/2009R1/drivers/net/bfin_mac.c: Task[#5621] If the IP payload len is odd, the inversed FCS should
- Revision
- 7675
- Author
- sonicz
- Date
- 2009-10-19 06:34:07 -0400 (Mon, 19 Oct 2009)
Log Message
Task[#5621] If the IP payload len is odd, the inversed FCS should
begin from odd address and leave first byte zero.
Modified Paths
Diff
Modified: branches/2009R1/drivers/net/bfin_mac.c (7674 => 7675)
--- branches/2009R1/drivers/net/bfin_mac.c 2009-10-19 10:33:33 UTC (rev 7674)
+++ branches/2009R1/drivers/net/bfin_mac.c 2009-10-19 10:34:07 UTC (rev 7675)
@@ -683,7 +683,7 @@
unsigned short len;
#if defined(BFIN_MAC_CSUM_OFFLOAD)
unsigned int i;
- unsigned char fcs[ETH_FCS_LENGTH];
+ unsigned char fcs[ETH_FCS_LENGTH + 1];
#endif
/* allocate a new skb for next time receive */
@@ -719,10 +719,19 @@
* Deduce Ethernet FCS from hardware generated IP payload checksum.
* IP checksum is based on 16-bit one's complement algorithm.
* To deduce a value from checksum is equal to add its inversion.
+ * If the IP payload len is odd, the inversed FCS should also
+ * begin from odd address and leave first byte zero.
*/
- for (i = 0; i < ETH_FCS_LENGTH; i++)
- fcs[i] = ~skb->data[skb->len + i];
- skb->csum = csum_partial(fcs, ETH_FCS_LENGTH, skb->csum);
+ if (skb->len % 2) {
+ fcs[0] = 0;
+ for (i = 0; i < ETH_FCS_LENGTH; i++)
+ fcs[i + 1] = ~skb->data[skb->len + i];
+ skb->csum = csum_partial(fcs, ETH_FCS_LENGTH + 1, skb->csum);
+ } else {
+ for (i = 0; i < ETH_FCS_LENGTH; i++)
+ fcs[i] = ~skb->data[skb->len + i];
+ skb->csum = csum_partial(fcs, ETH_FCS_LENGTH, skb->csum);
+ }
skb->ip_summed = CHECKSUM_COMPLETE;
#endif
_______________________________________________
Linux-kernel-commits mailing list
[email protected]
https://blackfin.uclinux.org/mailman/listinfo/linux-kernel-commits
