linux-kernel - Re: [PATCH] f2fs: issue discard when discard_cmd_cnt is not 0 in f2fs_put

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

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)