]> git.baikalelectronics.ru Git - kernel.git/commit
[MIPS] Fix wrong checksum calculation on 64-bit MIPS
authorAtsushi Nemoto <anemo@mba.ocn.ne.jp>
Wed, 24 Jan 2007 06:43:34 +0000 (15:43 +0900)
committerRalf Baechle <ralf@linux-mips.org>
Wed, 24 Jan 2007 19:23:22 +0000 (19:23 +0000)
commit92c14bb2397b3d0f49a31fa06a9170b23dde2cb2
tree6ba949af13a6f07ceaa7a5e99c71dc513d860849
parent4be4e1d8f5a03cff85334fffca87143fddd4b20f
[MIPS] Fix wrong checksum calculation on 64-bit MIPS

The commit 21d3c3e8470184df83c41cb0a142acf6c667efe1 ([NET]: MIPS
checksum annotations and cleanups) broke 64-bit MIPS.

The problem is the commit replaces some unsigned long with __be32.  On
64bit MIPS, a __be32 (i.e. unsigned int) value is represented as a
sign-extented 32-bit value in a 64-bit argument register.  So the
address 192.168.0.1 (0xc0a80001) is passed as 0xffffffffc0a80001 to
csum_tcpudp_nofold() but the asm code in the function expects
0x00000000c0a80001, therefore it returns a wrong checksum.  Explicit
cast to unsigned long is needed to drop high 32bit.

Signed-off-by: Atsushi Nemoto <anemo@mba.ocn.ne.jp>
Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
include/asm-mips/checksum.h