Messages in this thread | | | Date | Wed, 1 Mar 2023 08:56:04 +0100 | Subject | Re: [PATCH v5 1/2] riscv: Get rid of riscv_pfn_base variable | From | Alexandre Ghiti <> |
| |
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 <lkp@intel.com> > > 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!
| |