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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id BCBEEC433F5 for ; Fri, 26 Nov 2021 05:20:28 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1351035AbhKZFXj (ORCPT ); Fri, 26 Nov 2021 00:23:39 -0500 Received: from mga02.intel.com ([134.134.136.20]:41881 "EHLO mga02.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1352518AbhKZFVi (ORCPT ); Fri, 26 Nov 2021 00:21:38 -0500 X-IronPort-AV: E=McAfee;i="6200,9189,10179"; a="222835192" X-IronPort-AV: E=Sophos;i="5.87,263,1631602800"; d="scan'208";a="222835192" Received: from orsmga008.jf.intel.com ([10.7.209.65]) by orsmga101.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 25 Nov 2021 21:15:58 -0800 X-IronPort-AV: E=Sophos;i="5.87,263,1631602800"; d="scan'208";a="510513613" Received: from rongch2-mobl.ccr.corp.intel.com (HELO [10.255.31.165]) ([10.255.31.165]) by orsmga008-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 25 Nov 2021 21:15:55 -0800 Subject: Re: [kbuild-all] Re: [PATCH 1/8] powerpc/mm: Make slice specific to book3s/64 To: Christophe Leroy , kernel test robot , Benjamin Herrenschmidt , Paul Mackerras , Michael Ellerman , alex@ghiti.fr Cc: linux-mm@kvack.org, kbuild-all@lists.01.org, linuxppc-dev@lists.ozlabs.org, linux-kernel@vger.kernel.org References: <14e2c0b0d4fff49c1cb30166f54ce8e445e17b16.1637570556.git.christophe.leroy@csgroup.eu> <202111222218.RhruLp7S-lkp@intel.com> From: "Chen, Rong A" Message-ID: <802212cd-87d6-cb7b-768d-9794d08eb4f8@intel.com> Date: Fri, 26 Nov 2021 13:15:53 +0800 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:78.0) Gecko/20100101 Firefox/78.0 Thunderbird/78.12.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 11/24/2021 9:49 PM, Christophe Leroy wrote: > > > Le 24/11/2021 à 13:10, Christophe Leroy a écrit : >> >> >> Le 22/11/2021 à 15:48, kernel test robot a écrit : >>> Hi Christophe, >>> >>> I love your patch! Perhaps something to improve: >>> >>> [auto build test WARNING on powerpc/next] >>> [also build test WARNING on hnaz-mm/master linus/master v5.16-rc2 >>> next-20211118] >>> [If your patch is applied to the wrong git tree, kindly drop us a note. >>> And when submitting patch, we suggest to use '--base' as documented in >>> https://git-scm.com/docs/git-format-patch] >>> >>> url: >>> https://github.com/0day-ci/linux/commits/Christophe-Leroy/Convert-powerpc-to-default-topdown-mmap-layout/20211122-165115 >>> >>> base: >>> https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git next >>> config: powerpc64-randconfig-s031-20211122 (attached as .config) >>> compiler: powerpc64-linux-gcc (GCC) 11.2.0 >>> reproduce: >>>          wget >>> https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross >>> -O ~/bin/make.cross >>>          chmod +x ~/bin/make.cross >>>          # apt-get install sparse >>>          # sparse version: v0.6.4-dirty >>>          # >>> https://github.com/0day-ci/linux/commit/1d0b7cc86d08f25f595b52d8c39ba9ca1d29a30a >>> >>>          git remote add linux-review https://github.com/0day-ci/linux >>>          git fetch --no-tags linux-review >>> Christophe-Leroy/Convert-powerpc-to-default-topdown-mmap-layout/20211122-165115 >>> >>>          git checkout 1d0b7cc86d08f25f595b52d8c39ba9ca1d29a30a >>>          # save the attached .config to linux build tree >>>          COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 >>> make.cross C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' >>> ARCH=powerpc64 >>> >>> If you fix the issue, kindly add following tag as appropriate >>> Reported-by: kernel test robot >>> >>> All warnings (new ones prefixed by >>): >>> >>>     arch/powerpc/mm/book3s64/slice.c: In function >>> 'slice_get_unmapped_area': >>>>> arch/powerpc/mm/book3s64/slice.c:639:1: warning: the frame size of >>>>> 1040 bytes is larger than 1024 bytes [-Wframe-larger-than=] >>>       639 | } >>>           | ^ >> >> >> The problem was already existing when slice.c was in arch/powerpc/mm/ >> >> This patch doesn't introduce the problem. >> > > In fact the problem is really added by yourself mister 'kernel test robot'. > > CONFIG_FRAME_WARN is supposed to be 2048 on 64 bit architectures. > > It the robot starts to reduce that value, it is on its own .... Hi Christophe, Thanks for the information, we'll set the default value for FRAME_WARN in randconfig tests. Best Regards, Rong Chen > > > config FRAME_WARN >     int "Warn for stack frames larger than" >     range 0 8192 >     default 2048 if GCC_PLUGIN_LATENT_ENTROPY >     default 1536 if (!64BIT && (PARISC || XTENSA)) >     default 1024 if (!64BIT && !PARISC) >     default 2048 if 64BIT >     help >       Tell gcc to warn at build time for stack frames larger than this. >       Setting this too low will cause a lot of warnings. >       Setting it to 0 disables the warning. > _______________________________________________ > kbuild-all mailing list -- kbuild-all@lists.01.org > To unsubscribe send an email to kbuild-all-leave@lists.01.org From mboxrd@z Thu Jan 1 00:00:00 1970 Content-Type: multipart/mixed; boundary="===============5091211089499941631==" MIME-Version: 1.0 From: Chen, Rong A To: kbuild-all@lists.01.org Subject: Re: [PATCH 1/8] powerpc/mm: Make slice specific to book3s/64 Date: Fri, 26 Nov 2021 13:15:53 +0800 Message-ID: <802212cd-87d6-cb7b-768d-9794d08eb4f8@intel.com> In-Reply-To: List-Id: --===============5091211089499941631== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable On 11/24/2021 9:49 PM, Christophe Leroy wrote: > = > = > Le 24/11/2021 =C3=A0 13:10, Christophe Leroy a =C3=A9crit=C2=A0: >> >> >> Le 22/11/2021 =C3=A0 15:48, kernel test robot a =C3=A9crit=C2=A0: >>> Hi Christophe, >>> >>> I love your patch! Perhaps something to improve: >>> >>> [auto build test WARNING on powerpc/next] >>> [also build test WARNING on hnaz-mm/master linus/master v5.16-rc2 = >>> next-20211118] >>> [If your patch is applied to the wrong git tree, kindly drop us a note. >>> And when submitting patch, we suggest to use '--base' as documented in >>> https://git-scm.com/docs/git-format-patch] >>> >>> url: = >>> https://github.com/0day-ci/linux/commits/Christophe-Leroy/Convert-power= pc-to-default-topdown-mmap-layout/20211122-165115 = >>> >>> base: = >>> https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git next >>> config: powerpc64-randconfig-s031-20211122 (attached as .config) >>> compiler: powerpc64-linux-gcc (GCC) 11.2.0 >>> reproduce: >>> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 wget = >>> https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cros= s = >>> -O ~/bin/make.cross >>> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 chmod +x ~/bin/make.cr= oss >>> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 # apt-get install spar= se >>> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 # sparse version: v0.6= .4-dirty >>> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 # = >>> https://github.com/0day-ci/linux/commit/1d0b7cc86d08f25f595b52d8c39ba9c= a1d29a30a = >>> >>> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 git remote add linux-r= eview https://github.com/0day-ci/linux >>> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 git fetch --no-tags li= nux-review = >>> Christophe-Leroy/Convert-powerpc-to-default-topdown-mmap-layout/2021112= 2-165115 = >>> >>> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 git checkout 1d0b7cc86= d08f25f595b52d8c39ba9ca1d29a30a >>> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 # save the attached .c= onfig to linux build tree >>> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 COMPILER_INSTALL_PATH= =3D$HOME/0day COMPILER=3Dgcc-11.2.0 = >>> make.cross C=3D1 CF=3D'-fdiagnostic-prefix -D__CHECK_ENDIAN__' = >>> ARCH=3Dpowerpc64 >>> >>> If you fix the issue, kindly add following tag as appropriate >>> Reported-by: kernel test robot >>> >>> All warnings (new ones prefixed by >>): >>> >>> =C2=A0=C2=A0=C2=A0 arch/powerpc/mm/book3s64/slice.c: In function = >>> 'slice_get_unmapped_area': >>>>> arch/powerpc/mm/book3s64/slice.c:639:1: warning: the frame size of = >>>>> 1040 bytes is larger than 1024 bytes [-Wframe-larger-than=3D] >>> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 639 | } >>> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 | ^ >> >> >> The problem was already existing when slice.c was in arch/powerpc/mm/ >> >> This patch doesn't introduce the problem. >> > = > In fact the problem is really added by yourself mister 'kernel test robot= '. > = > CONFIG_FRAME_WARN is supposed to be 2048 on 64 bit architectures. > = > It the robot starts to reduce that value, it is on its own .... Hi Christophe, Thanks for the information, we'll set the default value for FRAME_WARN = in randconfig tests. Best Regards, Rong Chen > = > = > config FRAME_WARN > =C2=A0=C2=A0=C2=A0=C2=A0int "Warn for stack frames larger than" > =C2=A0=C2=A0=C2=A0=C2=A0range 0 8192 > =C2=A0=C2=A0=C2=A0=C2=A0default 2048 if GCC_PLUGIN_LATENT_ENTROPY > =C2=A0=C2=A0=C2=A0=C2=A0default 1536 if (!64BIT && (PARISC || XTENSA)) > =C2=A0=C2=A0=C2=A0=C2=A0default 1024 if (!64BIT && !PARISC) > =C2=A0=C2=A0=C2=A0=C2=A0default 2048 if 64BIT > =C2=A0=C2=A0=C2=A0=C2=A0help > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 Tell gcc to warn at build time for stack = frames larger than this. > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 Setting this too low will cause a lot of = warnings. > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 Setting it to 0 disables the warning. > _______________________________________________ > kbuild-all mailing list -- kbuild-all(a)lists.01.org > To unsubscribe send an email to kbuild-all-leave(a)lists.01.org --===============5091211089499941631==--