path: root/lib/checksum.c
diff options
authorAl Viro <>2020-07-22 22:14:36 -0400
committerAl Viro <>2020-08-20 15:45:14 -0400
commit6e41c585e38ff696de3a11509a0ad0a11150b0c3 (patch)
tree6af77b846b7523063a645913fa6a2c6563beb238 /lib/checksum.c
parent3ea7ca80d9c0ff606b5a129dce2354f771a9e4e8 (diff)
unify generic instances of csum_partial_copy_nocheck()
quite a few architectures have the same csum_partial_copy_nocheck() - simply memcpy() the data and then return the csum of the copy. hexagon, parisc, ia64, s390, um: explicitly spelled out that way. arc, arm64, csky, h8300, m68k/nommu, microblaze, mips/GENERIC_CSUM, nds32, nios2, openrisc, riscv, unicore32: end up picking the same thing spelled out in lib/checksum.h (with varying amounts of perversions along the way). everybody else (alpha, arm, c6x, m68k/mmu, mips/!GENERIC_CSUM, powerpc, sh, sparc, x86, xtensa) have non-generic variants. For all except c6x the declaration is in their asm/checksum.h. c6x uses the wrapper from asm-generic/checksum.h that would normally lead to the lib/checksum.h instance, but in case of c6x we end up using an asm function from arch/c6x instead. Screw that mess - have architectures with private instances define _HAVE_ARCH_CSUM_AND_COPY in their asm/checksum.h and have the default one right in net/checksum.h conditional on _HAVE_ARCH_CSUM_AND_COPY *not* defined. Signed-off-by: Al Viro <>
Diffstat (limited to 'lib/checksum.c')
1 files changed, 0 insertions, 11 deletions
diff --git a/lib/checksum.c b/lib/checksum.c
index c7861e84c526..6860d6b05a17 100644
--- a/lib/checksum.c
+++ b/lib/checksum.c
@@ -145,17 +145,6 @@ __sum16 ip_compute_csum(const void *buff, int len)
- * copy from ds while checksumming, otherwise like csum_partial
- */
-csum_partial_copy_nocheck(const void *src, void *dst, int len, __wsum sum)
- memcpy(dst, src, len);
- return csum_partial(dst, len, sum);
#ifndef csum_tcpudp_nofold
static inline u32 from64to32(u64 x)