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 B1113C7EE23 for ; Wed, 1 Mar 2023 07:56:22 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229844AbjCAH4U (ORCPT ); Wed, 1 Mar 2023 02:56:20 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:60708 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229602AbjCAH4P (ORCPT ); Wed, 1 Mar 2023 02:56:15 -0500 Received: from relay10.mail.gandi.net (relay10.mail.gandi.net [IPv6:2001:4b98:dc4:8::230]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 516BB37562; Tue, 28 Feb 2023 23:56:10 -0800 (PST) Received: (Authenticated sender: alex@ghiti.fr) by mail.gandi.net (Postfix) with ESMTPSA id E71E7240009; Wed, 1 Mar 2023 07:56:04 +0000 (UTC) Message-ID: Date: Wed, 1 Mar 2023 08:56:04 +0100 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.7.1 Subject: Re: [PATCH v5 1/2] riscv: Get rid of riscv_pfn_base variable Content-Language: en-US To: kernel test robot , Alexandre Ghiti , Paul Walmsley , Palmer Dabbelt , Albert Ou , Guo Ren , Rob Herring , Frank Rowand , Conor Dooley , linux-riscv@lists.infradead.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, linux-arch@vger.kernel.org Cc: oe-kbuild-all@lists.linux.dev References: <20230125081214.1576313-2-alexghiti@rivosinc.com> <202301282230.sz4DCUe6-lkp@intel.com> From: Alexandre Ghiti In-Reply-To: <202301282230.sz4DCUe6-lkp@intel.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 1/28/23 15:58, kernel test robot wrote: > Hi Alexandre, > > Thank you for the patch! Perhaps something to improve: > > [auto build test WARNING on robh/for-next] > [also build test WARNING on linus/master v6.2-rc5 next-20230127] > [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#_base_tree_information] > > url: https://github.com/intel-lab-lkp/linux/commits/Alexandre-Ghiti/riscv-Get-rid-of-riscv_pfn_base-variable/20230125-161537 > base: https://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git for-next > patch link: https://lore.kernel.org/r/20230125081214.1576313-2-alexghiti%40rivosinc.com > patch subject: [PATCH v5 1/2] riscv: Get rid of riscv_pfn_base variable > config: riscv-allnoconfig (https://download.01.org/0day-ci/archive/20230128/202301282230.sz4DCUe6-lkp@intel.com/config) > compiler: riscv64-linux-gcc (GCC) 12.1.0 > reproduce (this is a W=1 build): > wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross > chmod +x ~/bin/make.cross > # https://github.com/intel-lab-lkp/linux/commit/90b21402dc8a7e6e36a62ad19c4969ff13fad168 > git remote add linux-review https://github.com/intel-lab-lkp/linux > git fetch --no-tags linux-review Alexandre-Ghiti/riscv-Get-rid-of-riscv_pfn_base-variable/20230125-161537 > git checkout 90b21402dc8a7e6e36a62ad19c4969ff13fad168 > # save the config file > mkdir build_dir && cp config build_dir/.config > COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 O=build_dir ARCH=riscv olddefconfig > COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 O=build_dir ARCH=riscv SHELL=/bin/bash > > If you fix the issue, kindly add following tag where applicable > | Reported-by: kernel test robot > > All warnings (new ones prefixed by >>): > > In file included from include/asm-generic/bug.h:22, > from arch/riscv/include/asm/bug.h:83, > from include/linux/bug.h:5, > from arch/riscv/include/asm/cmpxchg.h:9, > from arch/riscv/include/asm/atomic.h:19, > from include/linux/atomic.h:7, > from include/linux/jump_label.h:255, > from arch/riscv/include/asm/vdso/processor.h:7, > from include/vdso/processor.h:10, > from arch/riscv/include/asm/processor.h:11, > from arch/riscv/include/asm/irqflags.h:10, > from include/linux/irqflags.h:16, > from arch/riscv/include/asm/bitops.h:14, > from include/linux/bitops.h:68, > from include/linux/kernel.h:22, > from mm/debug.c:9: > mm/debug.c: In function '__dump_page': >>> include/linux/kern_levels.h:5:25: warning: format '%lx' expects argument of type 'long unsigned int', but argument 7 has type 'long long unsigned int' [-Wformat=] > 5 | #define KERN_SOH "\001" /* ASCII Start Of Header */ > | ^~~~~~ > include/linux/printk.h:429:25: note: in definition of macro 'printk_index_wrap' > 429 | _p_func(_fmt, ##__VA_ARGS__); \ > | ^~~~ > include/linux/printk.h:510:9: note: in expansion of macro 'printk' > 510 | printk(KERN_WARNING pr_fmt(fmt), ##__VA_ARGS__) > | ^~~~~~ > include/linux/kern_levels.h:12:25: note: in expansion of macro 'KERN_SOH' > 12 | #define KERN_WARNING KERN_SOH "4" /* warning conditions */ > | ^~~~~~~~ > include/linux/printk.h:510:16: note: in expansion of macro 'KERN_WARNING' > 510 | printk(KERN_WARNING pr_fmt(fmt), ##__VA_ARGS__) > | ^~~~~~~~~~~~ > mm/debug.c:93:9: note: in expansion of macro 'pr_warn' > 93 | pr_warn("page:%p refcount:%d mapcount:%d mapping:%p index:%#lx pfn:%#lx\n", > | ^~~~~~~ > > > vim +5 include/linux/kern_levels.h > > 314ba3520e513a Joe Perches 2012-07-30 4 > 04d2c8c83d0e3a Joe Perches 2012-07-30 @5 #define KERN_SOH "\001" /* ASCII Start Of Header */ > 04d2c8c83d0e3a Joe Perches 2012-07-30 6 #define KERN_SOH_ASCII '\001' > 04d2c8c83d0e3a Joe Perches 2012-07-30 7 > And this one was mine, sorry I i overlooked that!