* Re: [f2fs:dev-test 15/23] fs//f2fs/super.c:830:2: error: too many arguments to function 'f2fs_wait_discard_bios' [not found] <201710110240.fDBixdgs%fengguang.wu@intel.com> @ 2017-10-10 18:32 ` Jaegeuk Kim 2017-10-11 1:22 ` Chao Yu 0 siblings, 1 reply; 2+ messages in thread From: Jaegeuk Kim @ 2017-10-10 18:32 UTC (permalink / raw) To: kbuild test robot; +Cc: kbuild-all, linux-f2fs-devel FYI; I fixed this in the branch. Thanks, On 10/11, kbuild test robot wrote: > tree: https://git.kernel.org/pub/scm/linux/kernel/git/jaegeuk/f2fs.git dev-test > head: 1b5d38c7280014133ddec78603d231006adba05b > commit: 5814edee9182ef648388de43df62d56f159cb5f9 [15/23] f2fs: support issuing/waiting discard in range > config: i386-randconfig-x008-201741 (attached as .config) > compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901 > reproduce: > git checkout 5814edee9182ef648388de43df62d56f159cb5f9 > # save the attached .config to linux build tree > make ARCH=i386 > > Note: the f2fs/dev-test HEAD 1b5d38c7280014133ddec78603d231006adba05b builds fine. > It only hurts bisectibility. > > All errors (new ones prefixed by >>): > > fs//f2fs/super.c: In function 'f2fs_put_super': > >> fs//f2fs/super.c:830:2: error: too many arguments to function 'f2fs_wait_discard_bios' > f2fs_wait_discard_bios(sbi, true); > ^~~~~~~~~~~~~~~~~~~~~~ > In file included from fs//f2fs/super.c:30:0: > fs//f2fs/f2fs.h:2551:6: note: declared here > void f2fs_wait_discard_bios(struct f2fs_sb_info *sbi); > ^~~~~~~~~~~~~~~~~~~~~~ > In file included from include/uapi/linux/stddef.h:1:0, > from include/linux/stddef.h:4, > from include/uapi/linux/posix_types.h:4, > from include/uapi/linux/types.h:13, > from include/linux/types.h:5, > from include/linux/list.h:4, > from include/linux/module.h:9, > from fs//f2fs/super.c:11: > fs//f2fs/super.c: At top level: > include/linux/compiler.h:162:4: warning: '______f' is static but declared in inline function 'strcpy' which is not static > ______f = { \ > ^ > include/linux/compiler.h:154:23: note: in expansion of macro '__trace_if' > #define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) ) > ^~~~~~~~~~ > include/linux/string.h:421:2: note: in expansion of macro 'if' > if (p_size == (size_t)-1 && q_size == (size_t)-1) > ^~ > include/linux/compiler.h:162:4: warning: '______f' is static but declared in inline function 'kmemdup' which is not static > ______f = { \ > ^ > include/linux/compiler.h:154:23: note: in expansion of macro '__trace_if' > #define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) ) > ^~~~~~~~~~ > include/linux/string.h:411:2: note: in expansion of macro 'if' > if (p_size < size) > ^~ > include/linux/compiler.h:162:4: warning: '______f' is static but declared in inline function 'kmemdup' which is not static > ______f = { \ > ^ > include/linux/compiler.h:154:23: note: in expansion of macro '__trace_if' > #define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) ) > ^~~~~~~~~~ > include/linux/string.h:409:2: note: in expansion of macro 'if' > if (__builtin_constant_p(size) && p_size < size) > ^~ > include/linux/compiler.h:162:4: warning: '______f' is static but declared in inline function 'memchr_inv' which is not static > ______f = { \ > ^ > include/linux/compiler.h:154:23: note: in expansion of macro '__trace_if' > #define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) ) > ^~~~~~~~~~ > include/linux/string.h:400:2: note: in expansion of macro 'if' > if (p_size < size) > ^~ > include/linux/compiler.h:162:4: warning: '______f' is static but declared in inline function 'memchr_inv' which is not static > ______f = { \ > ^ > include/linux/compiler.h:154:23: note: in expansion of macro '__trace_if' > #define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) ) > ^~~~~~~~~~ > include/linux/string.h:398:2: note: in expansion of macro 'if' > if (__builtin_constant_p(size) && p_size < size) > ^~ > include/linux/compiler.h:162:4: warning: '______f' is static but declared in inline function 'memchr' which is not static > ______f = { \ > ^ > include/linux/compiler.h:154:23: note: in expansion of macro '__trace_if' > #define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) ) > ^~~~~~~~~~ > include/linux/string.h:389:2: note: in expansion of macro 'if' > if (p_size < size) > ^~ > include/linux/compiler.h:162:4: warning: '______f' is static but declared in inline function 'memchr' which is not static > ______f = { \ > ^ > include/linux/compiler.h:154:23: note: in expansion of macro '__trace_if' > #define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) ) > ^~~~~~~~~~ > include/linux/string.h:387:2: note: in expansion of macro 'if' > if (__builtin_constant_p(size) && p_size < size) > ^~ > include/linux/compiler.h:162:4: warning: '______f' is static but declared in inline function 'memcmp' which is not static > ______f = { \ > ^ > include/linux/compiler.h:154:23: note: in expansion of macro '__trace_if' > #define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) ) > ^~~~~~~~~~ > include/linux/string.h:379:2: note: in expansion of macro 'if' > if (p_size < size || q_size < size) > ^~ > include/linux/compiler.h:162:4: warning: '______f' is static but declared in inline function 'memcmp' which is not static > ______f = { \ > ^ > include/linux/compiler.h:154:23: note: in expansion of macro '__trace_if' > #define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) ) > ^~~~~~~~~~ > include/linux/string.h:376:3: note: in expansion of macro 'if' > if (q_size < size) > ^~ > include/linux/compiler.h:162:4: warning: '______f' is static but declared in inline function 'memcmp' which is not static > ______f = { \ > ^ > include/linux/compiler.h:154:23: note: in expansion of macro '__trace_if' > > vim +/f2fs_wait_discard_bios +830 fs//f2fs/super.c > > 3c62be17 Jaegeuk Kim 2016-10-06 805 > aff063e2 Jaegeuk Kim 2012-11-02 806 static void f2fs_put_super(struct super_block *sb) > aff063e2 Jaegeuk Kim 2012-11-02 807 { > aff063e2 Jaegeuk Kim 2012-11-02 808 struct f2fs_sb_info *sbi = F2FS_SB(sb); > a398101a Chao Yu 2017-06-14 809 int i; > aff063e2 Jaegeuk Kim 2012-11-02 810 > 0abd675e Chao Yu 2017-07-09 811 f2fs_quota_off_umount(sb); > aff063e2 Jaegeuk Kim 2012-11-02 812 > 2658e50d Jaegeuk Kim 2015-06-19 813 /* prevent remaining shrinker jobs */ > 2658e50d Jaegeuk Kim 2015-06-19 814 mutex_lock(&sbi->umount_mutex); > 2658e50d Jaegeuk Kim 2015-06-19 815 > 85dc2f2c Jaegeuk Kim 2015-01-14 816 /* > 85dc2f2c Jaegeuk Kim 2015-01-14 817 * We don't need to do checkpoint when superblock is clean. > 85dc2f2c Jaegeuk Kim 2015-01-14 818 * But, the previous checkpoint was not done by umount, it needs to do > 85dc2f2c Jaegeuk Kim 2015-01-14 819 * clean checkpoint again. > 85dc2f2c Jaegeuk Kim 2015-01-14 820 */ > caf0047e Chao Yu 2015-01-28 821 if (is_sbi_flag_set(sbi, SBI_IS_DIRTY) || > aaec2b1d Chao Yu 2016-09-20 822 !is_set_ckpt_flags(sbi, CP_UMOUNT_FLAG)) { > 75ab4cb8 Jaegeuk Kim 2014-09-20 823 struct cp_control cpc = { > 75ab4cb8 Jaegeuk Kim 2014-09-20 824 .reason = CP_UMOUNT, > 75ab4cb8 Jaegeuk Kim 2014-09-20 825 }; > 75ab4cb8 Jaegeuk Kim 2014-09-20 826 write_checkpoint(sbi, &cpc); > 75ab4cb8 Jaegeuk Kim 2014-09-20 827 } > aff063e2 Jaegeuk Kim 2012-11-02 828 > 4e6a8d9b Jaegeuk Kim 2016-12-29 829 /* be sure to wait for any on-going discard commands */ > 638164a2 Chao Yu 2017-10-02 @830 f2fs_wait_discard_bios(sbi, true); > 4e6a8d9b Jaegeuk Kim 2016-12-29 831 > 663f387b Chao Yu 2017-06-14 832 if (f2fs_discard_en(sbi) && !sbi->discard_blks) { > 1f43e2ad Chao Yu 2017-04-28 833 struct cp_control cpc = { > 1f43e2ad Chao Yu 2017-04-28 834 .reason = CP_UMOUNT | CP_TRIMMED, > 1f43e2ad Chao Yu 2017-04-28 835 }; > 1f43e2ad Chao Yu 2017-04-28 836 write_checkpoint(sbi, &cpc); > 1f43e2ad Chao Yu 2017-04-28 837 } > 1f43e2ad Chao Yu 2017-04-28 838 > eca616f8 Jaegeuk Kim 2015-06-15 839 /* write_checkpoint can update stat informaion */ > eca616f8 Jaegeuk Kim 2015-06-15 840 f2fs_destroy_stats(sbi); > eca616f8 Jaegeuk Kim 2015-06-15 841 > cf779cab Jaegeuk Kim 2014-08-11 842 /* > cf779cab Jaegeuk Kim 2014-08-11 843 * normally superblock is clean, so we need to release this. > cf779cab Jaegeuk Kim 2014-08-11 844 * In addition, EIO will skip do checkpoint, we need this as well. > cf779cab Jaegeuk Kim 2014-08-11 845 */ > 74ef9241 Jaegeuk Kim 2016-05-02 846 release_ino_entry(sbi, true); > 6f12ac25 Jaegeuk Kim 2014-08-19 847 > 2658e50d Jaegeuk Kim 2015-06-19 848 f2fs_leave_shrinker(sbi); > 2658e50d Jaegeuk Kim 2015-06-19 849 mutex_unlock(&sbi->umount_mutex); > 2658e50d Jaegeuk Kim 2015-06-19 850 > 17c19120 Jaegeuk Kim 2016-01-29 851 /* our cp_error case, we can wait for any writeback page */ > b9109b0e Jaegeuk Kim 2017-05-10 852 f2fs_flush_merged_writes(sbi); > 17c19120 Jaegeuk Kim 2016-01-29 853 > aff063e2 Jaegeuk Kim 2012-11-02 854 iput(sbi->node_inode); > aff063e2 Jaegeuk Kim 2012-11-02 855 iput(sbi->meta_inode); > aff063e2 Jaegeuk Kim 2012-11-02 856 > aff063e2 Jaegeuk Kim 2012-11-02 857 /* destroy f2fs internal modules */ > aff063e2 Jaegeuk Kim 2012-11-02 858 destroy_node_manager(sbi); > aff063e2 Jaegeuk Kim 2012-11-02 859 destroy_segment_manager(sbi); > aff063e2 Jaegeuk Kim 2012-11-02 860 > aff063e2 Jaegeuk Kim 2012-11-02 861 kfree(sbi->ckpt); > a398101a Chao Yu 2017-06-14 862 > dc6b2055 Jaegeuk Kim 2017-07-26 863 f2fs_unregister_sysfs(sbi); > aff063e2 Jaegeuk Kim 2012-11-02 864 > aff063e2 Jaegeuk Kim 2012-11-02 865 sb->s_fs_info = NULL; > 43b6573b Keith Mok 2016-03-02 866 if (sbi->s_chksum_driver) > 43b6573b Keith Mok 2016-03-02 867 crypto_free_shash(sbi->s_chksum_driver); > b39f0de2 Yunlei He 2015-12-15 868 kfree(sbi->raw_super); > 523be8a6 Jaegeuk Kim 2016-05-13 869 > 3c62be17 Jaegeuk Kim 2016-10-06 870 destroy_device_list(sbi); > b6895e8f Chao Yu 2017-02-27 871 mempool_destroy(sbi->write_io_dummy); > 4b2414d0 Chao Yu 2017-08-08 872 #ifdef CONFIG_QUOTA > 4b2414d0 Chao Yu 2017-08-08 873 for (i = 0; i < MAXQUOTAS; i++) > 4b2414d0 Chao Yu 2017-08-08 874 kfree(sbi->s_qf_names[i]); > 4b2414d0 Chao Yu 2017-08-08 875 #endif > 523be8a6 Jaegeuk Kim 2016-05-13 876 destroy_percpu_info(sbi); > a912b54d Jaegeuk Kim 2017-05-10 877 for (i = 0; i < NR_PAGE_TYPE; i++) > a912b54d Jaegeuk Kim 2017-05-10 878 kfree(sbi->write_io[i]); > aff063e2 Jaegeuk Kim 2012-11-02 879 kfree(sbi); > aff063e2 Jaegeuk Kim 2012-11-02 880 } > aff063e2 Jaegeuk Kim 2012-11-02 881 > > :::::: The code at line 830 was first introduced by commit > :::::: 638164a2718f337ea224b747cf5977ef143166a4 f2fs: fix potential panic during fstrim > > :::::: TO: Chao Yu <yuchao0@huawei.com> > :::::: CC: Jaegeuk Kim <jaegeuk@kernel.org> > > --- > 0-DAY kernel test infrastructure Open Source Technology Center > https://lists.01.org/pipermail/kbuild-all Intel Corporation ------------------------------------------------------------------------------ Check out the vibrant tech community on one of the world's most engaging tech sites, Slashdot.org! http://sdm.link/slashdot ^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [f2fs:dev-test 15/23] fs//f2fs/super.c:830:2: error: too many arguments to function 'f2fs_wait_discard_bios' 2017-10-10 18:32 ` [f2fs:dev-test 15/23] fs//f2fs/super.c:830:2: error: too many arguments to function 'f2fs_wait_discard_bios' Jaegeuk Kim @ 2017-10-11 1:22 ` Chao Yu 0 siblings, 0 replies; 2+ messages in thread From: Chao Yu @ 2017-10-11 1:22 UTC (permalink / raw) To: Jaegeuk Kim, kbuild test robot; +Cc: kbuild-all, linux-f2fs-devel Ah, thank you. ;) Thanks, On 2017/10/11 2:32, Jaegeuk Kim wrote: > FYI; I fixed this in the branch. > > Thanks, > > On 10/11, kbuild test robot wrote: >> tree: https://git.kernel.org/pub/scm/linux/kernel/git/jaegeuk/f2fs.git dev-test >> head: 1b5d38c7280014133ddec78603d231006adba05b >> commit: 5814edee9182ef648388de43df62d56f159cb5f9 [15/23] f2fs: support issuing/waiting discard in range >> config: i386-randconfig-x008-201741 (attached as .config) >> compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901 >> reproduce: >> git checkout 5814edee9182ef648388de43df62d56f159cb5f9 >> # save the attached .config to linux build tree >> make ARCH=i386 >> >> Note: the f2fs/dev-test HEAD 1b5d38c7280014133ddec78603d231006adba05b builds fine. >> It only hurts bisectibility. >> >> All errors (new ones prefixed by >>): >> >> fs//f2fs/super.c: In function 'f2fs_put_super': >>>> fs//f2fs/super.c:830:2: error: too many arguments to function 'f2fs_wait_discard_bios' >> f2fs_wait_discard_bios(sbi, true); >> ^~~~~~~~~~~~~~~~~~~~~~ >> In file included from fs//f2fs/super.c:30:0: >> fs//f2fs/f2fs.h:2551:6: note: declared here >> void f2fs_wait_discard_bios(struct f2fs_sb_info *sbi); >> ^~~~~~~~~~~~~~~~~~~~~~ >> In file included from include/uapi/linux/stddef.h:1:0, >> from include/linux/stddef.h:4, >> from include/uapi/linux/posix_types.h:4, >> from include/uapi/linux/types.h:13, >> from include/linux/types.h:5, >> from include/linux/list.h:4, >> from include/linux/module.h:9, >> from fs//f2fs/super.c:11: >> fs//f2fs/super.c: At top level: >> include/linux/compiler.h:162:4: warning: '______f' is static but declared in inline function 'strcpy' which is not static >> ______f = { \ >> ^ >> include/linux/compiler.h:154:23: note: in expansion of macro '__trace_if' >> #define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) ) >> ^~~~~~~~~~ >> include/linux/string.h:421:2: note: in expansion of macro 'if' >> if (p_size == (size_t)-1 && q_size == (size_t)-1) >> ^~ >> include/linux/compiler.h:162:4: warning: '______f' is static but declared in inline function 'kmemdup' which is not static >> ______f = { \ >> ^ >> include/linux/compiler.h:154:23: note: in expansion of macro '__trace_if' >> #define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) ) >> ^~~~~~~~~~ >> include/linux/string.h:411:2: note: in expansion of macro 'if' >> if (p_size < size) >> ^~ >> include/linux/compiler.h:162:4: warning: '______f' is static but declared in inline function 'kmemdup' which is not static >> ______f = { \ >> ^ >> include/linux/compiler.h:154:23: note: in expansion of macro '__trace_if' >> #define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) ) >> ^~~~~~~~~~ >> include/linux/string.h:409:2: note: in expansion of macro 'if' >> if (__builtin_constant_p(size) && p_size < size) >> ^~ >> include/linux/compiler.h:162:4: warning: '______f' is static but declared in inline function 'memchr_inv' which is not static >> ______f = { \ >> ^ >> include/linux/compiler.h:154:23: note: in expansion of macro '__trace_if' >> #define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) ) >> ^~~~~~~~~~ >> include/linux/string.h:400:2: note: in expansion of macro 'if' >> if (p_size < size) >> ^~ >> include/linux/compiler.h:162:4: warning: '______f' is static but declared in inline function 'memchr_inv' which is not static >> ______f = { \ >> ^ >> include/linux/compiler.h:154:23: note: in expansion of macro '__trace_if' >> #define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) ) >> ^~~~~~~~~~ >> include/linux/string.h:398:2: note: in expansion of macro 'if' >> if (__builtin_constant_p(size) && p_size < size) >> ^~ >> include/linux/compiler.h:162:4: warning: '______f' is static but declared in inline function 'memchr' which is not static >> ______f = { \ >> ^ >> include/linux/compiler.h:154:23: note: in expansion of macro '__trace_if' >> #define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) ) >> ^~~~~~~~~~ >> include/linux/string.h:389:2: note: in expansion of macro 'if' >> if (p_size < size) >> ^~ >> include/linux/compiler.h:162:4: warning: '______f' is static but declared in inline function 'memchr' which is not static >> ______f = { \ >> ^ >> include/linux/compiler.h:154:23: note: in expansion of macro '__trace_if' >> #define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) ) >> ^~~~~~~~~~ >> include/linux/string.h:387:2: note: in expansion of macro 'if' >> if (__builtin_constant_p(size) && p_size < size) >> ^~ >> include/linux/compiler.h:162:4: warning: '______f' is static but declared in inline function 'memcmp' which is not static >> ______f = { \ >> ^ >> include/linux/compiler.h:154:23: note: in expansion of macro '__trace_if' >> #define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) ) >> ^~~~~~~~~~ >> include/linux/string.h:379:2: note: in expansion of macro 'if' >> if (p_size < size || q_size < size) >> ^~ >> include/linux/compiler.h:162:4: warning: '______f' is static but declared in inline function 'memcmp' which is not static >> ______f = { \ >> ^ >> include/linux/compiler.h:154:23: note: in expansion of macro '__trace_if' >> #define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) ) >> ^~~~~~~~~~ >> include/linux/string.h:376:3: note: in expansion of macro 'if' >> if (q_size < size) >> ^~ >> include/linux/compiler.h:162:4: warning: '______f' is static but declared in inline function 'memcmp' which is not static >> ______f = { \ >> ^ >> include/linux/compiler.h:154:23: note: in expansion of macro '__trace_if' >> >> vim +/f2fs_wait_discard_bios +830 fs//f2fs/super.c >> >> 3c62be17 Jaegeuk Kim 2016-10-06 805 >> aff063e2 Jaegeuk Kim 2012-11-02 806 static void f2fs_put_super(struct super_block *sb) >> aff063e2 Jaegeuk Kim 2012-11-02 807 { >> aff063e2 Jaegeuk Kim 2012-11-02 808 struct f2fs_sb_info *sbi = F2FS_SB(sb); >> a398101a Chao Yu 2017-06-14 809 int i; >> aff063e2 Jaegeuk Kim 2012-11-02 810 >> 0abd675e Chao Yu 2017-07-09 811 f2fs_quota_off_umount(sb); >> aff063e2 Jaegeuk Kim 2012-11-02 812 >> 2658e50d Jaegeuk Kim 2015-06-19 813 /* prevent remaining shrinker jobs */ >> 2658e50d Jaegeuk Kim 2015-06-19 814 mutex_lock(&sbi->umount_mutex); >> 2658e50d Jaegeuk Kim 2015-06-19 815 >> 85dc2f2c Jaegeuk Kim 2015-01-14 816 /* >> 85dc2f2c Jaegeuk Kim 2015-01-14 817 * We don't need to do checkpoint when superblock is clean. >> 85dc2f2c Jaegeuk Kim 2015-01-14 818 * But, the previous checkpoint was not done by umount, it needs to do >> 85dc2f2c Jaegeuk Kim 2015-01-14 819 * clean checkpoint again. >> 85dc2f2c Jaegeuk Kim 2015-01-14 820 */ >> caf0047e Chao Yu 2015-01-28 821 if (is_sbi_flag_set(sbi, SBI_IS_DIRTY) || >> aaec2b1d Chao Yu 2016-09-20 822 !is_set_ckpt_flags(sbi, CP_UMOUNT_FLAG)) { >> 75ab4cb8 Jaegeuk Kim 2014-09-20 823 struct cp_control cpc = { >> 75ab4cb8 Jaegeuk Kim 2014-09-20 824 .reason = CP_UMOUNT, >> 75ab4cb8 Jaegeuk Kim 2014-09-20 825 }; >> 75ab4cb8 Jaegeuk Kim 2014-09-20 826 write_checkpoint(sbi, &cpc); >> 75ab4cb8 Jaegeuk Kim 2014-09-20 827 } >> aff063e2 Jaegeuk Kim 2012-11-02 828 >> 4e6a8d9b Jaegeuk Kim 2016-12-29 829 /* be sure to wait for any on-going discard commands */ >> 638164a2 Chao Yu 2017-10-02 @830 f2fs_wait_discard_bios(sbi, true); >> 4e6a8d9b Jaegeuk Kim 2016-12-29 831 >> 663f387b Chao Yu 2017-06-14 832 if (f2fs_discard_en(sbi) && !sbi->discard_blks) { >> 1f43e2ad Chao Yu 2017-04-28 833 struct cp_control cpc = { >> 1f43e2ad Chao Yu 2017-04-28 834 .reason = CP_UMOUNT | CP_TRIMMED, >> 1f43e2ad Chao Yu 2017-04-28 835 }; >> 1f43e2ad Chao Yu 2017-04-28 836 write_checkpoint(sbi, &cpc); >> 1f43e2ad Chao Yu 2017-04-28 837 } >> 1f43e2ad Chao Yu 2017-04-28 838 >> eca616f8 Jaegeuk Kim 2015-06-15 839 /* write_checkpoint can update stat informaion */ >> eca616f8 Jaegeuk Kim 2015-06-15 840 f2fs_destroy_stats(sbi); >> eca616f8 Jaegeuk Kim 2015-06-15 841 >> cf779cab Jaegeuk Kim 2014-08-11 842 /* >> cf779cab Jaegeuk Kim 2014-08-11 843 * normally superblock is clean, so we need to release this. >> cf779cab Jaegeuk Kim 2014-08-11 844 * In addition, EIO will skip do checkpoint, we need this as well. >> cf779cab Jaegeuk Kim 2014-08-11 845 */ >> 74ef9241 Jaegeuk Kim 2016-05-02 846 release_ino_entry(sbi, true); >> 6f12ac25 Jaegeuk Kim 2014-08-19 847 >> 2658e50d Jaegeuk Kim 2015-06-19 848 f2fs_leave_shrinker(sbi); >> 2658e50d Jaegeuk Kim 2015-06-19 849 mutex_unlock(&sbi->umount_mutex); >> 2658e50d Jaegeuk Kim 2015-06-19 850 >> 17c19120 Jaegeuk Kim 2016-01-29 851 /* our cp_error case, we can wait for any writeback page */ >> b9109b0e Jaegeuk Kim 2017-05-10 852 f2fs_flush_merged_writes(sbi); >> 17c19120 Jaegeuk Kim 2016-01-29 853 >> aff063e2 Jaegeuk Kim 2012-11-02 854 iput(sbi->node_inode); >> aff063e2 Jaegeuk Kim 2012-11-02 855 iput(sbi->meta_inode); >> aff063e2 Jaegeuk Kim 2012-11-02 856 >> aff063e2 Jaegeuk Kim 2012-11-02 857 /* destroy f2fs internal modules */ >> aff063e2 Jaegeuk Kim 2012-11-02 858 destroy_node_manager(sbi); >> aff063e2 Jaegeuk Kim 2012-11-02 859 destroy_segment_manager(sbi); >> aff063e2 Jaegeuk Kim 2012-11-02 860 >> aff063e2 Jaegeuk Kim 2012-11-02 861 kfree(sbi->ckpt); >> a398101a Chao Yu 2017-06-14 862 >> dc6b2055 Jaegeuk Kim 2017-07-26 863 f2fs_unregister_sysfs(sbi); >> aff063e2 Jaegeuk Kim 2012-11-02 864 >> aff063e2 Jaegeuk Kim 2012-11-02 865 sb->s_fs_info = NULL; >> 43b6573b Keith Mok 2016-03-02 866 if (sbi->s_chksum_driver) >> 43b6573b Keith Mok 2016-03-02 867 crypto_free_shash(sbi->s_chksum_driver); >> b39f0de2 Yunlei He 2015-12-15 868 kfree(sbi->raw_super); >> 523be8a6 Jaegeuk Kim 2016-05-13 869 >> 3c62be17 Jaegeuk Kim 2016-10-06 870 destroy_device_list(sbi); >> b6895e8f Chao Yu 2017-02-27 871 mempool_destroy(sbi->write_io_dummy); >> 4b2414d0 Chao Yu 2017-08-08 872 #ifdef CONFIG_QUOTA >> 4b2414d0 Chao Yu 2017-08-08 873 for (i = 0; i < MAXQUOTAS; i++) >> 4b2414d0 Chao Yu 2017-08-08 874 kfree(sbi->s_qf_names[i]); >> 4b2414d0 Chao Yu 2017-08-08 875 #endif >> 523be8a6 Jaegeuk Kim 2016-05-13 876 destroy_percpu_info(sbi); >> a912b54d Jaegeuk Kim 2017-05-10 877 for (i = 0; i < NR_PAGE_TYPE; i++) >> a912b54d Jaegeuk Kim 2017-05-10 878 kfree(sbi->write_io[i]); >> aff063e2 Jaegeuk Kim 2012-11-02 879 kfree(sbi); >> aff063e2 Jaegeuk Kim 2012-11-02 880 } >> aff063e2 Jaegeuk Kim 2012-11-02 881 >> >> :::::: The code at line 830 was first introduced by commit >> :::::: 638164a2718f337ea224b747cf5977ef143166a4 f2fs: fix potential panic during fstrim >> >> :::::: TO: Chao Yu <yuchao0@huawei.com> >> :::::: CC: Jaegeuk Kim <jaegeuk@kernel.org> >> >> --- >> 0-DAY kernel test infrastructure Open Source Technology Center >> https://lists.01.org/pipermail/kbuild-all Intel Corporation > > > > . > ------------------------------------------------------------------------------ Check out the vibrant tech community on one of the world's most engaging tech sites, Slashdot.org! http://sdm.link/slashdot ^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2017-10-11 1:24 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <201710110240.fDBixdgs%fengguang.wu@intel.com>
2017-10-10 18:32 ` [f2fs:dev-test 15/23] fs//f2fs/super.c:830:2: error: too many arguments to function 'f2fs_wait_discard_bios' Jaegeuk Kim
2017-10-11 1:22 ` Chao Yu
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).