On Jan 10, 2023 / 12:27, kernel test robot wrote: > Hi Shin'ichiro, > > Thank you for the patch! Perhaps something to improve: > > [auto build test WARNING on jejb-scsi/for-next] > [also build test WARNING on mkp-scsi/for-next linus/master v6.2-rc3 next-20230109] > [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/Shin-ichiro-Kawasaki/scsi-mpi3mr-remove-unnecessary-memcpy/20230110-095717 > base: https://git.kernel.org/pub/scm/linux/kernel/git/jejb/scsi.git for-next > patch link: https://lore.kernel.org/r/20230110015538.201332-2-shinichiro.kawasaki%40wdc.com > patch subject: [PATCH v3 1/5] scsi: mpi3mr: remove unnecessary memcpy > config: ia64-allyesconfig > compiler: ia64-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/fad43146fb864da8124537dd3a548f9c638cddd6 > git remote add linux-review https://github.com/intel-lab-lkp/linux > git fetch --no-tags linux-review Shin-ichiro-Kawasaki/scsi-mpi3mr-remove-unnecessary-memcpy/20230110-095717 > git checkout fad43146fb864da8124537dd3a548f9c638cddd6 > # 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=ia64 olddefconfig > COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 O=build_dir ARCH=ia64 SHELL=/bin/bash drivers/scsi/mpi3mr/ > > If you fix the issue, kindly add following tag where applicable > | Reported-by: kernel test robot <lkp@xxxxxxxxx> > > All warnings (new ones prefixed by >>): > > drivers/scsi/mpi3mr/mpi3mr_app.c: In function 'mpi3mr_get_all_tgt_info': > >> drivers/scsi/mpi3mr/mpi3mr_app.c:301:18: warning: variable 'min_entrylen' set but not used [-Wunused-but-set-variable] > 301 | uint32_t min_entrylen = 0, kern_entrylen = 0, usr_entrylen = 0; > | ^~~~~~~~~~~~ This patch removes the reference to min_entrylen, and the 3rd patch adds back the reference. So, this warning can be fixed by moving the 1st patch in the series after the 3rd patch. Will do so in v4. -- Shin'ichiro Kawasaki
