On Wed, Jun 03, 2020 at 08:10:38PM +0100, Al Viro wrote: > On Wed, Jun 03, 2020 at 08:37:14AM -0700, Guenter Roeck wrote: > > On Fri, Mar 27, 2020 at 11:31:08PM +0000, Al Viro wrote: > > > From: Al Viro <[email protected]> > > > > > > Just inline the call and use memcpy() instead of __copy_from_user() and > > > note that the tail is precisely ia64 csum_partial(). > > > > > > Signed-off-by: Al Viro <[email protected]> > > > > This patch results in: > > > > arch/ia64/lib/csum_partial_copy.c: In function 'csum_partial_copy_nocheck': > > arch/ia64/lib/csum_partial_copy.c:110:9: error: implicit declaration of > > function 'csum_partial' > > > > for ia64:{defconfig, allnoconfig, tinyconfig}. > > Argh... > > Signed-off-by: Al Viro <[email protected]>
Yes, that does the trick. Tested-by: Guenter Roeck <[email protected]> > --- > diff --git a/arch/ia64/lib/csum_partial_copy.c > b/arch/ia64/lib/csum_partial_copy.c > index 5d147a33d648..6e82e0be8040 100644 > --- a/arch/ia64/lib/csum_partial_copy.c > +++ b/arch/ia64/lib/csum_partial_copy.c > @@ -12,7 +12,7 @@ > #include <linux/types.h> > #include <linux/string.h> > > -#include <linux/uaccess.h> > +#include <net/checksum.h> > > /* > * XXX Fixme: those 2 inlines are meant for debugging and will go away
