From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 497C0C433EF for ; Thu, 18 Nov 2021 16:01:18 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 25B76613DB for ; Thu, 18 Nov 2021 16:01:18 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232778AbhKRQER (ORCPT ); Thu, 18 Nov 2021 11:04:17 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:37356 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229936AbhKRQEQ (ORCPT ); Thu, 18 Nov 2021 11:04:16 -0500 Received: from desiato.infradead.org (desiato.infradead.org [IPv6:2001:8b0:10b:1:d65d:64ff:fe57:4e05]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 116F1C061574 for ; Thu, 18 Nov 2021 08:01:16 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=desiato.20200630; h=In-Reply-To:Content-Type:MIME-Version: References:Message-ID:Subject:Cc:To:From:Date:Sender:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description; bh=bLyYtzBJtGfdAn90Z7we9HaBKQkcvI+sJjyTeDMw2Os=; b=aYSBw0dcNSCAhpIshmXqFv/GzW frBO/V/ReYGf9GaLhXC6Z/QfTyytJ4MtFYT1XDsL20EYHb4yJ7IMc7h5EzOHjfcVkLCSlRHhtYxy3 HqhbeofNxqFudyl6Zm6n86uI0TxbpwO24s28+ypGT/KNdB50/7W4o26hSQwjuMMFGARQdS0TMMo3K 1dZo6cg2CvPtimiHW5/iz/H+31mrqMsiRqMlYtRU+xJkErKKQwb8MKU6ycp9ZI1ycgfqBq0DrmKLc 7+qDNVjetvkvu17E0iiPYKpQXiXCFqFFsK8NohYLMi23yRmQBQMY4L1fztc0KLZO+lRJg2ddt9Vqx 2J2fGdXw==; Received: from j217100.upc-j.chello.nl ([24.132.217.100] helo=noisy.programming.kicks-ass.net) by desiato.infradead.org with esmtpsa (Exim 4.94.2 #2 (Red Hat Linux)) id 1mnjqO-00GiaS-BD; Thu, 18 Nov 2021 16:01:00 +0000 Received: from hirez.programming.kicks-ass.net (hirez.programming.kicks-ass.net [192.168.1.225]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (Client did not present a certificate) by noisy.programming.kicks-ass.net (Postfix) with ESMTPS id D1A33300093; Thu, 18 Nov 2021 17:00:58 +0100 (CET) Received: by hirez.programming.kicks-ass.net (Postfix, from userid 1000) id AA5F630553074; Thu, 18 Nov 2021 17:00:58 +0100 (CET) Date: Thu, 18 Nov 2021 17:00:58 +0100 From: Peter Zijlstra To: Eric Dumazet Cc: kernel test robot , kbuild-all@lists.01.org, linux-kernel@vger.kernel.org, x86@kernel.org, Alexander Duyck , linux-um@lists.infradead.org Subject: Re: [tip:x86/core 1/1] arch/x86/um/../lib/csum-partial_64.c:98:12: error: implicit declaration of function 'load_unaligned_zeropad' Message-ID: References: <202111180219.YkyoBMpf-lkp@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Nov 17, 2021 at 11:40:35AM -0800, Eric Dumazet wrote: > On Wed, Nov 17, 2021 at 10:55 AM Eric Dumazet wrote: > > > > On Wed, Nov 17, 2021 at 10:46 AM kernel test robot wrote: > > > > > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git x86/core > > > head: d31c3c683ee668ba5d87c0730610442fd672525f > > > commit: d31c3c683ee668ba5d87c0730610442fd672525f [1/1] x86/csum: Rewrite/optimize csum_partial() > > > config: um-x86_64_defconfig (attached as .config) > > > compiler: gcc-9 (Debian 9.3.0-22) 9.3.0 > > > reproduce (this is a W=1 build): > > > # https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git/commit/?id=d31c3c683ee668ba5d87c0730610442fd672525f > > > git remote add tip https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git > > > git fetch --no-tags tip x86/core > > > git checkout d31c3c683ee668ba5d87c0730610442fd672525f > > > # save the attached .config to linux build tree > > > make W=1 ARCH=um SUBARCH=x86_64 > > > > > > If you fix the issue, kindly add following tag as appropriate > > > Reported-by: kernel test robot > > > > > > All errors (new ones prefixed by >>): > > > > > > arch/x86/um/../lib/csum-partial_64.c: In function 'csum_partial': > > > >> arch/x86/um/../lib/csum-partial_64.c:98:12: error: implicit declaration of function 'load_unaligned_zeropad' [-Werror=implicit-function-declaration] > > > 98 | trail = (load_unaligned_zeropad(buff) << shift) >> shift; > > > | ^~~~~~~~~~~~~~~~~~~~~~ > > > cc1: some warnings being treated as errors > > > > > > > > > > Hmmm... it seems we need to guard this with CONFIG_DCACHE_WORD_ACCESS ? > > Perhaps something like the following ? Dear um folks, is this indeed the best solution? It's a bit sad to have to add this to x86_64, but if that's the way it is... > > diff --git a/arch/x86/lib/csum-partial_64.c b/arch/x86/lib/csum-partial_64.c > index 5ec35626945b6db2f7f41c6d46d5e422810eac46..d419b9345d6dba2e924887671bc6f11c3e17ebd7 > 100644 > --- a/arch/x86/lib/csum-partial_64.c > +++ b/arch/x86/lib/csum-partial_64.c > @@ -91,12 +91,23 @@ __wsum csum_partial(const void *buff, int len, __wsum sum) > : "memory"); > buff += 8; > } > - if (len & 7) { > - unsigned int shift = (8 - (len & 7)) * 8; > + len &= 7; > + if (len) { > unsigned long trail; > +#ifndef CONFIG_DCACHE_WORD_ACCESS > + union { > + unsigned long ulval; > + u8 bytes[sizeof(long)]; > + } v; > > - trail = (load_unaligned_zeropad(buff) << shift) >> shift; > + v.ulval = 0; > + memcpy(v.bytes, buff, len); > + trail = v.ulval; > +#else > + unsigned int shift = (sizeof(long) - len) * BITS_PER_BYTE; > > + trail = (load_unaligned_zeropad(buff) << shift) >> shift; > +#endif > asm("addq %[trail],%[res]\n\t" > "adcq $0,%[res]" > : [res] "+r" (temp64) From mboxrd@z Thu Jan 1 00:00:00 1970 Content-Type: multipart/mixed; boundary="===============6012815697770565002==" MIME-Version: 1.0 From: Peter Zijlstra To: kbuild-all@lists.01.org Subject: Re: [tip:x86/core 1/1] arch/x86/um/../lib/csum-partial_64.c:98:12: error: implicit declaration of function 'load_unaligned_zeropad' Date: Thu, 18 Nov 2021 17:00:58 +0100 Message-ID: In-Reply-To: List-Id: --===============6012815697770565002== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable On Wed, Nov 17, 2021 at 11:40:35AM -0800, Eric Dumazet wrote: > On Wed, Nov 17, 2021 at 10:55 AM Eric Dumazet wro= te: > > > > On Wed, Nov 17, 2021 at 10:46 AM kernel test robot wr= ote: > > > > > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git x= 86/core > > > head: d31c3c683ee668ba5d87c0730610442fd672525f > > > commit: d31c3c683ee668ba5d87c0730610442fd672525f [1/1] x86/csum: Rewr= ite/optimize csum_partial() > > > config: um-x86_64_defconfig (attached as .config) > > > compiler: gcc-9 (Debian 9.3.0-22) 9.3.0 > > > reproduce (this is a W=3D1 build): > > > # https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git= /commit/?id=3Dd31c3c683ee668ba5d87c0730610442fd672525f > > > git remote add tip https://git.kernel.org/pub/scm/linux/kerne= l/git/tip/tip.git > > > git fetch --no-tags tip x86/core > > > git checkout d31c3c683ee668ba5d87c0730610442fd672525f > > > # save the attached .config to linux build tree > > > make W=3D1 ARCH=3Dum SUBARCH=3Dx86_64 > > > > > > If you fix the issue, kindly add following tag as appropriate > > > Reported-by: kernel test robot > > > > > > All errors (new ones prefixed by >>): > > > > > > arch/x86/um/../lib/csum-partial_64.c: In function 'csum_partial': > > > >> arch/x86/um/../lib/csum-partial_64.c:98:12: error: implicit declar= ation of function 'load_unaligned_zeropad' [-Werror=3Dimplicit-function-dec= laration] > > > 98 | trail =3D (load_unaligned_zeropad(buff) << shift) >> shi= ft; > > > | ^~~~~~~~~~~~~~~~~~~~~~ > > > cc1: some warnings being treated as errors > > > > > > > > > > Hmmm... it seems we need to guard this with CONFIG_DCACHE_WORD_ACCESS ? > = > Perhaps something like the following ? Dear um folks, is this indeed the best solution? It's a bit sad to have to add this to x86_64, but if that's the way it is... > = > diff --git a/arch/x86/lib/csum-partial_64.c b/arch/x86/lib/csum-partial_6= 4.c > index 5ec35626945b6db2f7f41c6d46d5e422810eac46..d419b9345d6dba2e924887671= bc6f11c3e17ebd7 > 100644 > --- a/arch/x86/lib/csum-partial_64.c > +++ b/arch/x86/lib/csum-partial_64.c > @@ -91,12 +91,23 @@ __wsum csum_partial(const void *buff, int len, __wsum= sum) > : "memory"); > buff +=3D 8; > } > - if (len & 7) { > - unsigned int shift =3D (8 - (len & 7)) * 8; > + len &=3D 7; > + if (len) { > unsigned long trail; > +#ifndef CONFIG_DCACHE_WORD_ACCESS > + union { > + unsigned long ulval; > + u8 bytes[sizeof(long)]; > + } v; > = > - trail =3D (load_unaligned_zeropad(buff) << shift) >> shif= t; > + v.ulval =3D 0; > + memcpy(v.bytes, buff, len); > + trail =3D v.ulval; > +#else > + unsigned int shift =3D (sizeof(long) - len) * BITS_PER_BY= TE; > = > + trail =3D (load_unaligned_zeropad(buff) << shift) >> shif= t; > +#endif > asm("addq %[trail],%[res]\n\t" > "adcq $0,%[res]" > : [res] "+r" (temp64) --===============6012815697770565002==-- From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Thu, 18 Nov 2021 17:00:58 +0100 From: Peter Zijlstra Subject: Re: [tip:x86/core 1/1] arch/x86/um/../lib/csum-partial_64.c:98:12: error: implicit declaration of function 'load_unaligned_zeropad' Message-ID: References: <202111180219.YkyoBMpf-lkp@intel.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-um" Errors-To: linux-um-bounces+geert=linux-m68k.org@lists.infradead.org To: Eric Dumazet Cc: kernel test robot , kbuild-all@lists.01.org, linux-kernel@vger.kernel.org, x86@kernel.org, Alexander Duyck , linux-um@lists.infradead.org On Wed, Nov 17, 2021 at 11:40:35AM -0800, Eric Dumazet wrote: > On Wed, Nov 17, 2021 at 10:55 AM Eric Dumazet wrote: > > > > On Wed, Nov 17, 2021 at 10:46 AM kernel test robot wrote: > > > > > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git x86/core > > > head: d31c3c683ee668ba5d87c0730610442fd672525f > > > commit: d31c3c683ee668ba5d87c0730610442fd672525f [1/1] x86/csum: Rewrite/optimize csum_partial() > > > config: um-x86_64_defconfig (attached as .config) > > > compiler: gcc-9 (Debian 9.3.0-22) 9.3.0 > > > reproduce (this is a W=1 build): > > > # https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git/commit/?id=d31c3c683ee668ba5d87c0730610442fd672525f > > > git remote add tip https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git > > > git fetch --no-tags tip x86/core > > > git checkout d31c3c683ee668ba5d87c0730610442fd672525f > > > # save the attached .config to linux build tree > > > make W=1 ARCH=um SUBARCH=x86_64 > > > > > > If you fix the issue, kindly add following tag as appropriate > > > Reported-by: kernel test robot > > > > > > All errors (new ones prefixed by >>): > > > > > > arch/x86/um/../lib/csum-partial_64.c: In function 'csum_partial': > > > >> arch/x86/um/../lib/csum-partial_64.c:98:12: error: implicit declaration of function 'load_unaligned_zeropad' [-Werror=implicit-function-declaration] > > > 98 | trail = (load_unaligned_zeropad(buff) << shift) >> shift; > > > | ^~~~~~~~~~~~~~~~~~~~~~ > > > cc1: some warnings being treated as errors > > > > > > > > > > Hmmm... it seems we need to guard this with CONFIG_DCACHE_WORD_ACCESS ? > > Perhaps something like the following ? Dear um folks, is this indeed the best solution? It's a bit sad to have to add this to x86_64, but if that's the way it is... > > diff --git a/arch/x86/lib/csum-partial_64.c b/arch/x86/lib/csum-partial_64.c > index 5ec35626945b6db2f7f41c6d46d5e422810eac46..d419b9345d6dba2e924887671bc6f11c3e17ebd7 > 100644 > --- a/arch/x86/lib/csum-partial_64.c > +++ b/arch/x86/lib/csum-partial_64.c > @@ -91,12 +91,23 @@ __wsum csum_partial(const void *buff, int len, __wsum sum) > : "memory"); > buff += 8; > } > - if (len & 7) { > - unsigned int shift = (8 - (len & 7)) * 8; > + len &= 7; > + if (len) { > unsigned long trail; > +#ifndef CONFIG_DCACHE_WORD_ACCESS > + union { > + unsigned long ulval; > + u8 bytes[sizeof(long)]; > + } v; > > - trail = (load_unaligned_zeropad(buff) << shift) >> shift; > + v.ulval = 0; > + memcpy(v.bytes, buff, len); > + trail = v.ulval; > +#else > + unsigned int shift = (sizeof(long) - len) * BITS_PER_BYTE; > > + trail = (load_unaligned_zeropad(buff) << shift) >> shift; > +#endif > asm("addq %[trail],%[res]\n\t" > "adcq $0,%[res]" > : [res] "+r" (temp64) _______________________________________________ linux-um mailing list linux-um@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-um