Hi Gabriel, I love your patch! Perhaps something to improve:
[auto build test WARNING on tytso-ext4/dev] [also build test WARNING on jaegeuk-f2fs/dev-test linus/master v5.18-rc6 next-20220509] [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/Gabriel-Krisman-Bertazi/Clean-up-the-case-insenstive-lookup-path/20220430-022957 base: https://git.kernel.org/pub/scm/linux/kernel/git/tytso/ext4.git dev config: x86_64-rhel-8.3-func (https://download.01.org/0day-ci/archive/20220510/[email protected]/config) compiler: gcc-11 (Debian 11.2.0-20) 11.2.0 reproduce (this is a W=1 build): # https://github.com/intel-lab-lkp/linux/commit/293ba304c3d9ce0d65df81e519822c3e66152acc git remote add linux-review https://github.com/intel-lab-lkp/linux git fetch --no-tags linux-review Gabriel-Krisman-Bertazi/Clean-up-the-case-insenstive-lookup-path/20220430-022957 git checkout 293ba304c3d9ce0d65df81e519822c3e66152acc # save the config file mkdir build_dir && cp config build_dir/.config make W=1 O=build_dir ARCH=x86_64 SHELL=/bin/bash fs/f2fs/ If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <[email protected]> All warnings (new ones prefixed by >>): fs/f2fs/dir.c: In function 'f2fs_match_name': >> fs/f2fs/dir.c:216:29: warning: unused variable 'u' [-Wunused-variable] 216 | struct unicode_name u; | ^ vim +/u +216 fs/f2fs/dir.c 210 211 static inline int f2fs_match_name(const struct inode *dir, 212 const struct f2fs_filename *fname, 213 const u8 *de_name, u32 de_name_len) 214 { 215 struct fscrypt_name f; > 216 struct unicode_name u; 217 -- 0-DAY CI Kernel Test Service https://01.org/lkp _______________________________________________ Linux-f2fs-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel
