Messages in this thread | | | Date | Thu, 31 Mar 2022 06:23:07 +0800 | From | kernel test robot <> | Subject | Re: [PATCH v2 2/2] mtd: mtdoops: Create a header structure for the saved mtdoops. |
| |
Hi Jean-Marc,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on mtd/mtd/next] [also build test WARNING on mtd/mtd/fixes linux/master linus/master v5.17 next-20220330] [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/intel-lab-lkp/linux/commits/Jean-Marc-Eurin/mtd-mtdoops-Structure-the-header-of-the-dumped-oops/20220331-023808 base: https://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux.git mtd/next config: x86_64-randconfig-a015 (https://download.01.org/0day-ci/archive/20220331/202203310648.it4f2xXD-lkp@intel.com/config) compiler: gcc-9 (Ubuntu 9.4.0-1ubuntu1~20.04.1) 9.4.0 reproduce (this is a W=1 build): # https://github.com/intel-lab-lkp/linux/commit/0d39801219fd826554caf69402424346799810d5 git remote add linux-review https://github.com/intel-lab-lkp/linux git fetch --no-tags linux-review Jean-Marc-Eurin/mtd-mtdoops-Structure-the-header-of-the-dumped-oops/20220331-023808 git checkout 0d39801219fd826554caf69402424346799810d5 # save the config file to linux build tree mkdir build_dir make W=1 O=build_dir ARCH=x86_64 SHELL=/bin/bash drivers/mtd/
If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
In file included from include/linux/kernel.h:29, from drivers/mtd/mtdoops.c:10: drivers/mtd/mtdoops.c: In function 'find_next_position': >> include/linux/kern_levels.h:5:18: warning: format '%d' expects argument of type 'int', but argument 4 has type 'long unsigned int' [-Wformat=] 5 | #define KERN_SOH "\001" /* ASCII Start Of Header */ | ^~~~~~ include/linux/printk.h:418:11: note: in definition of macro 'printk_index_wrap' 418 | _p_func(_fmt, ##__VA_ARGS__); \ | ^~~~ drivers/mtd/mtdoops.c:243:4: note: in expansion of macro 'printk' 243 | printk(KERN_ERR "mtdoops: read failure at %ld (%td of %d read), err %d\n", | ^~~~~~ include/linux/kern_levels.h:11:18: note: in expansion of macro 'KERN_SOH' 11 | #define KERN_ERR KERN_SOH "3" /* error conditions */ | ^~~~~~~~ drivers/mtd/mtdoops.c:243:11: note: in expansion of macro 'KERN_ERR' 243 | printk(KERN_ERR "mtdoops: read failure at %ld (%td of %d read), err %d\n", | ^~~~~~~~ drivers/mtd/mtdoops.c:243:59: note: format string is defined here 243 | printk(KERN_ERR "mtdoops: read failure at %ld (%td of %d read), err %d\n", | ~^ | | | int | %ld
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
-- 0-DAY CI Kernel Test Service https://01.org/lkp
| |