[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <202212010132.r31sxx09-lkp@intel.com>
Date: Thu, 1 Dec 2022 01:36:29 +0800
From: kernel test robot <lkp@...el.com>
To: Yangtao Li <frank.li@...o.com>, jaegeuk@...nel.org, chao@...nel.org
Cc: oe-kbuild-all@...ts.linux.dev,
linux-f2fs-devel@...ts.sourceforge.net,
linux-kernel@...r.kernel.org, Yangtao Li <frank.li@...o.com>
Subject: Re: [PATCH] f2fs: issue discard when discard_cmd_cnt is not 0 in
f2fs_put_super()
Hi Yangtao,
I love your patch! Perhaps something to improve:
[auto build test WARNING on jaegeuk-f2fs/dev-test]
[also build test WARNING on jaegeuk-f2fs/dev linus/master v6.1-rc7]
[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/Yangtao-Li/f2fs-issue-discard-when-discard_cmd_cnt-is-not-0-in-f2fs_put_super/20221130-110654
base: https://git.kernel.org/pub/scm/linux/kernel/git/jaegeuk/f2fs.git dev-test
patch link: https://lore.kernel.org/r/20221129134018.34120-1-frank.li%40vivo.com
patch subject: [PATCH] f2fs: issue discard when discard_cmd_cnt is not 0 in f2fs_put_super()
config: x86_64-randconfig-a011
compiler: gcc-11 (Debian 11.3.0-8) 11.3.0
reproduce (this is a W=1 build):
# https://github.com/intel-lab-lkp/linux/commit/ecfaeeb102cb22ad0b39670f674763735763db2f
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review Yangtao-Li/f2fs-issue-discard-when-discard_cmd_cnt-is-not-0-in-f2fs_put_super/20221130-110654
git checkout ecfaeeb102cb22ad0b39670f674763735763db2f
# save the config file
mkdir build_dir && cp config build_dir/.config
make W=1 O=build_dir ARCH=x86_64 SHELL=/bin/bash
If you fix the issue, kindly add following tag where applicable
| Reported-by: kernel test robot <lkp@...el.com>
All warnings (new ones prefixed by >>):
fs/f2fs/super.c: In function 'f2fs_remount':
>> fs/f2fs/super.c:2227:37: warning: variable 'dcc' set but not used [-Wunused-but-set-variable]
2227 | struct discard_cmd_control *dcc;
| ^~~
vim +/dcc +2227 fs/f2fs/super.c
4354994f097d06 Daniel Rosenberg 2018-08-20 2209
696c018c7718f5 Namjae Jeon 2013-06-16 2210 static int f2fs_remount(struct super_block *sb, int *flags, char *data)
696c018c7718f5 Namjae Jeon 2013-06-16 2211 {
696c018c7718f5 Namjae Jeon 2013-06-16 2212 struct f2fs_sb_info *sbi = F2FS_SB(sb);
696c018c7718f5 Namjae Jeon 2013-06-16 2213 struct f2fs_mount_info org_mount_opt;
0abd675e97e60d Chao Yu 2017-07-09 2214 unsigned long old_sb_flags;
63189b785960c3 Chao Yu 2018-03-08 2215 int err;
3fd9735908287c Chao Yu 2021-03-17 2216 bool need_restart_gc = false, need_stop_gc = false;
3fd9735908287c Chao Yu 2021-03-17 2217 bool need_restart_ckpt = false, need_stop_ckpt = false;
3fd9735908287c Chao Yu 2021-03-17 2218 bool need_restart_flush = false, need_stop_flush = false;
4d67490498acb4 Fengnan Chang 2021-08-19 2219 bool need_restart_discard = false, need_stop_discard = false;
9cd81ce3c2f01f Chao Yu 2015-09-18 2220 bool no_extent_cache = !test_opt(sbi, EXTENT_CACHE);
277afbde6ca2b3 Chao Yu 2021-07-29 2221 bool enable_checkpoint = !test_opt(sbi, DISABLE_CHECKPOINT);
1f78adfab379e5 Chao Yu 2019-07-12 2222 bool no_io_align = !F2FS_IO_ALIGNED(sbi);
093749e296e29a Chao Yu 2020-08-04 2223 bool no_atgc = !test_opt(sbi, ATGC);
4d67490498acb4 Fengnan Chang 2021-08-19 2224 bool no_discard = !test_opt(sbi, DISCARD);
6ce19aff0b8cd3 Chao Yu 2021-05-20 2225 bool no_compress_cache = !test_opt(sbi, COMPRESS_CACHE);
4f993264fe2965 Chao Yu 2021-08-03 2226 bool block_unit_discard = f2fs_block_unit_discard(sbi);
4d67490498acb4 Fengnan Chang 2021-08-19 @2227 struct discard_cmd_control *dcc;
4b2414d04e9912 Chao Yu 2017-08-08 2228 #ifdef CONFIG_QUOTA
4b2414d04e9912 Chao Yu 2017-08-08 2229 int i, j;
4b2414d04e9912 Chao Yu 2017-08-08 2230 #endif
696c018c7718f5 Namjae Jeon 2013-06-16 2231
696c018c7718f5 Namjae Jeon 2013-06-16 2232 /*
696c018c7718f5 Namjae Jeon 2013-06-16 2233 * Save the old mount options in case we
696c018c7718f5 Namjae Jeon 2013-06-16 2234 * need to restore them.
696c018c7718f5 Namjae Jeon 2013-06-16 2235 */
696c018c7718f5 Namjae Jeon 2013-06-16 2236 org_mount_opt = sbi->mount_opt;
0abd675e97e60d Chao Yu 2017-07-09 2237 old_sb_flags = sb->s_flags;
696c018c7718f5 Namjae Jeon 2013-06-16 2238
--
0-DAY CI Kernel Test Service
https://01.org/lkp
View attachment "config" of type "text/plain" (158567 bytes)
Powered by blists - more mailing lists