Re: [PATCH 1/4] asm-generic headers: Allow csum_partial arch override

[Guenter Roeck <linux@roeck-us.net>]

On Wed, Feb 21, 2024 at 06:37:11PM -0800, Charlie Jenkins wrote:
> Arches can have more a efficient implementation of csum_partial.
> 
> Signed-off-by: Charlie Jenkins <charlie@rivosinc.com>

Tested-by: Guenter Roeck <linux@roeck-us.net>

> ---
>  include/asm-generic/checksum.h | 2 ++
>  lib/checksum.c                 | 2 ++
>  2 files changed, 4 insertions(+)
> 
> diff --git a/include/asm-generic/checksum.h b/include/asm-generic/checksum.h
> index ad928cce268b..3309830ba2cb 100644
> --- a/include/asm-generic/checksum.h
> +++ b/include/asm-generic/checksum.h
> @@ -4,6 +4,7 @@
>  
>  #include <linux/bitops.h>
>  
> +#ifndef csum_partial
>  /*
>   * computes the checksum of a memory block at buff, length len,
>   * and adds in "sum" (32-bit)
> @@ -17,6 +18,7 @@
>   * it's best to have buff aligned on a 32-bit boundary
>   */
>  extern __wsum csum_partial(const void *buff, int len, __wsum sum);
> +#endif
>  
>  #ifndef ip_fast_csum
>  /*
> diff --git a/lib/checksum.c b/lib/checksum.c
> index 6860d6b05a17..c115a9ac71d9 100644
> --- a/lib/checksum.c
> +++ b/lib/checksum.c
> @@ -110,6 +110,7 @@ __sum16 ip_fast_csum(const void *iph, unsigned int ihl)
>  EXPORT_SYMBOL(ip_fast_csum);
>  #endif
>  
> +#ifndef csum_partial
>  /*
>   * computes the checksum of a memory block at buff, length len,
>   * and adds in "sum" (32-bit)
> @@ -134,6 +135,7 @@ __wsum csum_partial(const void *buff, int len, __wsum wsum)
>  	return (__force __wsum)result;
>  }
>  EXPORT_SYMBOL(csum_partial);
> +#endif
>  
>  /*
>   * this routine is used for miscellaneous IP-like checksums, mainly
> 
> -- 
> 2.34.1
>