Commit 0ffb7122 authored by Al Viro's avatar Al Viro Committed by David S. Miller

[NET]: ARM26 checksum annotations and cleanups.

* sanitize prototypes, annotate
* kill csum_partial_copy
Signed-off-by: default avatarAl Viro <viro@zeniv.linux.org.uk>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent eb5a9658
...@@ -23,7 +23,7 @@ ...@@ -23,7 +23,7 @@
* *
* it's best to have buff aligned on a 32-bit boundary * it's best to have buff aligned on a 32-bit boundary
*/ */
unsigned int csum_partial(const unsigned char * buff, int len, unsigned int sum); __wsum csum_partial(const void *buff, int len, __wsum sum);
/* /*
* the same as csum_partial, but copies from src while it * the same as csum_partial, but copies from src while it
...@@ -33,26 +33,18 @@ unsigned int csum_partial(const unsigned char * buff, int len, unsigned int sum) ...@@ -33,26 +33,18 @@ unsigned int csum_partial(const unsigned char * buff, int len, unsigned int sum)
* better 64-bit) boundary * better 64-bit) boundary
*/ */
unsigned int __wsum
csum_partial_copy_nocheck(const char *src, char *dst, int len, int sum); csum_partial_copy_nocheck(const void *src, void *dst, int len, __wsum sum);
unsigned int __wsum
csum_partial_copy_from_user(const char __user *src, char *dst, int len, int sum, int *err_ptr); csum_partial_copy_from_user(const void __user *src, void *dst, int len, __wsum sum, int *err_ptr);
/*
* This is the old (and unsafe) way of doing checksums, a warning message will
* be printed if it is used and an exception occurs.
*
* this functions should go away after some time.
*/
#define csum_partial_copy(src,dst,len,sum) csum_partial_copy_nocheck(src,dst,len,sum)
/* /*
* This is a version of ip_compute_csum() optimized for IP headers, * This is a version of ip_compute_csum() optimized for IP headers,
* which always checksum on 4 octet boundaries. * which always checksum on 4 octet boundaries.
*/ */
static inline unsigned short static inline __sum16
ip_fast_csum(unsigned char * iph, unsigned int ihl) ip_fast_csum(const void *iph, unsigned int ihl)
{ {
unsigned int sum, tmp1; unsigned int sum, tmp1;
...@@ -78,14 +70,13 @@ ip_fast_csum(unsigned char * iph, unsigned int ihl) ...@@ -78,14 +70,13 @@ ip_fast_csum(unsigned char * iph, unsigned int ihl)
: "=r" (sum), "=r" (iph), "=r" (ihl), "=r" (tmp1) : "=r" (sum), "=r" (iph), "=r" (ihl), "=r" (tmp1)
: "1" (iph), "2" (ihl) : "1" (iph), "2" (ihl)
: "cc"); : "cc");
return sum; return (__force __sum16)sum;
} }
/* /*
* Fold a partial checksum without adding pseudo headers * Fold a partial checksum without adding pseudo headers
*/ */
static inline unsigned int static inline __sum16 csum_fold(__wsum sum)
csum_fold(unsigned int sum)
{ {
__asm__( __asm__(
"adds %0, %1, %1, lsl #16 @ csum_fold \n\ "adds %0, %1, %1, lsl #16 @ csum_fold \n\
...@@ -93,12 +84,12 @@ csum_fold(unsigned int sum) ...@@ -93,12 +84,12 @@ csum_fold(unsigned int sum)
: "=r" (sum) : "=r" (sum)
: "r" (sum) : "r" (sum)
: "cc"); : "cc");
return (~sum) >> 16; return (__force __sum16)(~(__force u32)sum >> 16);
} }
static inline unsigned int static inline __wsum
csum_tcpudp_nofold(unsigned long saddr, unsigned long daddr, unsigned short len, csum_tcpudp_nofold(__be32 saddr, __be32 daddr, unsigned short len,
unsigned int proto, unsigned int sum) unsigned short proto, __wsum sum)
{ {
__asm__( __asm__(
"adds %0, %1, %2 @ csum_tcpudp_nofold \n\ "adds %0, %1, %2 @ csum_tcpudp_nofold \n\
...@@ -107,7 +98,7 @@ csum_tcpudp_nofold(unsigned long saddr, unsigned long daddr, unsigned short len, ...@@ -107,7 +98,7 @@ csum_tcpudp_nofold(unsigned long saddr, unsigned long daddr, unsigned short len,
adcs %0, %0, %5 \n\ adcs %0, %0, %5 \n\
adc %0, %0, #0" adc %0, %0, #0"
: "=&r"(sum) : "=&r"(sum)
: "r" (sum), "r" (daddr), "r" (saddr), "r" (ntohs(len)), "Ir" (ntohs(proto)) : "r" (sum), "r" (daddr), "r" (saddr), "r" (htons(len)), "Ir" (htons(proto))
: "cc"); : "cc");
return sum; return sum;
} }
...@@ -115,9 +106,9 @@ csum_tcpudp_nofold(unsigned long saddr, unsigned long daddr, unsigned short len, ...@@ -115,9 +106,9 @@ csum_tcpudp_nofold(unsigned long saddr, unsigned long daddr, unsigned short len,
* computes the checksum of the TCP/UDP pseudo-header * computes the checksum of the TCP/UDP pseudo-header
* returns a 16-bit checksum, already complemented * returns a 16-bit checksum, already complemented
*/ */
static inline unsigned short int static inline __sum16
csum_tcpudp_magic(unsigned long saddr, unsigned long daddr, unsigned short len, csum_tcpudp_magic(__be32 saddr, __be32 daddr, unsigned short len,
unsigned int proto, unsigned int sum) unsigned short proto, __wsum sum)
{ {
__asm__( __asm__(
"adds %0, %1, %2 @ csum_tcpudp_magic \n\ "adds %0, %1, %2 @ csum_tcpudp_magic \n\
...@@ -129,9 +120,9 @@ csum_tcpudp_magic(unsigned long saddr, unsigned long daddr, unsigned short len, ...@@ -129,9 +120,9 @@ csum_tcpudp_magic(unsigned long saddr, unsigned long daddr, unsigned short len,
addcs %0, %0, #0x10000 \n\ addcs %0, %0, #0x10000 \n\
mvn %0, %0" mvn %0, %0"
: "=&r"(sum) : "=&r"(sum)
: "r" (sum), "r" (daddr), "r" (saddr), "r" (ntohs(len)), "Ir" (ntohs(proto)) : "r" (sum), "r" (daddr), "r" (saddr), "r" (htons(len)), "Ir" (htons(proto))
: "cc"); : "cc");
return sum >> 16; return (__force __sum16)((__force u32)sum >> 16);
} }
...@@ -139,20 +130,20 @@ csum_tcpudp_magic(unsigned long saddr, unsigned long daddr, unsigned short len, ...@@ -139,20 +130,20 @@ csum_tcpudp_magic(unsigned long saddr, unsigned long daddr, unsigned short len,
* this routine is used for miscellaneous IP-like checksums, mainly * this routine is used for miscellaneous IP-like checksums, mainly
* in icmp.c * in icmp.c
*/ */
static inline unsigned short static inline __sum16
ip_compute_csum(unsigned char * buff, int len) ip_compute_csum(const void *buff, int len)
{ {
return csum_fold(csum_partial(buff, len, 0)); return csum_fold(csum_partial(buff, len, 0));
} }
#define _HAVE_ARCH_IPV6_CSUM #define _HAVE_ARCH_IPV6_CSUM
extern unsigned long extern __wsum
__csum_ipv6_magic(struct in6_addr *saddr, struct in6_addr *daddr, __u32 len, __csum_ipv6_magic(const struct in6_addr *saddr, const struct in6_addr *daddr, __be32 len,
__u32 proto, unsigned int sum); __be32 proto, __wsum sum);
static inline unsigned short int static inline __sum16
csum_ipv6_magic(struct in6_addr *saddr, struct in6_addr *daddr, __u32 len, csum_ipv6_magic(const struct in6_addr *saddr, const struct in6_addr *daddr, __u32 len,
unsigned short proto, unsigned int sum) unsigned short proto, __wsum sum)
{ {
return csum_fold(__csum_ipv6_magic(saddr, daddr, htonl(len), return csum_fold(__csum_ipv6_magic(saddr, daddr, htonl(len),
htonl(proto), sum)); htonl(proto), sum));
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment