[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <202206152345.By6X3FNj-lkp@intel.com>
Date: Wed, 15 Jun 2022 23:02:55 +0800
From: kernel test robot <lkp@...el.com>
To: Ren Zhijie <renzhijie2@...wei.com>, jaegeuk@...nel.org,
chao@...nel.org, daehojeong@...gle.com
Cc: kbuild-all@...ts.01.org, linux-f2fs-devel@...ts.sourceforge.net,
linux-kernel@...r.kernel.org, Ren Zhijie <renzhijie2@...wei.com>
Subject: Re: [PATCH -next] f2fs: fix build error too many arguments to
functions
Hi Ren,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on next-20220615]
url: https://github.com/intel-lab-lkp/linux/commits/Ren-Zhijie/f2fs-fix-build-error-too-many-arguments-to-functions/20220615-150808
base: 6012273897fefb12566580efedee10bb06e5e6ed
config: x86_64-rhel-8.3 (https://download.01.org/0day-ci/archive/20220615/202206152345.By6X3FNj-lkp@intel.com/config)
compiler: gcc-11 (Debian 11.3.0-3) 11.3.0
reproduce (this is a W=1 build):
# https://github.com/intel-lab-lkp/linux/commit/1832104c73270fad342bf40c089542083dca7585
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review Ren-Zhijie/f2fs-fix-build-error-too-many-arguments-to-functions/20220615-150808
git checkout 1832104c73270fad342bf40c089542083dca7585
# 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 where applicable
Reported-by: kernel test robot <lkp@...el.com>
All errors (new ones prefixed by >>):
fs/f2fs/data.c: In function 'f2fs_finish_read_bio':
>> fs/f2fs/data.c:136:33: error: too few arguments to function 'f2fs_end_read_compressed_page'
136 | f2fs_end_read_compressed_page(page, true, 0);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from fs/f2fs/data.c:25:
fs/f2fs/f2fs.h:4226:20: note: declared here
4226 | static inline void f2fs_end_read_compressed_page(struct page *page, bool failed,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>> fs/f2fs/data.c:137:25: error: too few arguments to function 'f2fs_put_page_dic'
137 | f2fs_put_page_dic(page);
| ^~~~~~~~~~~~~~~~~
In file included from fs/f2fs/data.c:25:
fs/f2fs/f2fs.h:4231:20: note: declared here
4231 | static inline void f2fs_put_page_dic(struct page *page, bool in_softirq)
| ^~~~~~~~~~~~~~~~~
fs/f2fs/data.c: In function 'f2fs_handle_step_decompress':
fs/f2fs/data.c:239:25: error: too few arguments to function 'f2fs_end_read_compressed_page'
239 | f2fs_end_read_compressed_page(page, PageError(page),
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from fs/f2fs/data.c:25:
fs/f2fs/f2fs.h:4226:20: note: declared here
4226 | static inline void f2fs_end_read_compressed_page(struct page *page, bool failed,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
vim +/f2fs_end_read_compressed_page +136 fs/f2fs/data.c
6dbb17961f46b2 Eric Biggers 2018-04-18 121
7f59b277f79e8a Eric Biggers 2021-01-04 122 static void f2fs_finish_read_bio(struct bio *bio)
93dfe2ac516250 Jaegeuk Kim 2013-11-30 123 {
6dbb17961f46b2 Eric Biggers 2018-04-18 124 struct bio_vec *bv;
6dc4f100c175dd Ming Lei 2019-02-15 125 struct bvec_iter_all iter_all;
93dfe2ac516250 Jaegeuk Kim 2013-11-30 126
7f59b277f79e8a Eric Biggers 2021-01-04 127 /*
7f59b277f79e8a Eric Biggers 2021-01-04 128 * Update and unlock the bio's pagecache pages, and put the
7f59b277f79e8a Eric Biggers 2021-01-04 129 * decompression context for any compressed pages.
7f59b277f79e8a Eric Biggers 2021-01-04 130 */
2b070cfe582b8e Christoph Hellwig 2019-04-25 131 bio_for_each_segment_all(bv, bio, iter_all) {
7f59b277f79e8a Eric Biggers 2021-01-04 132 struct page *page = bv->bv_page;
6dbb17961f46b2 Eric Biggers 2018-04-18 133
7f59b277f79e8a Eric Biggers 2021-01-04 134 if (f2fs_is_compressed_page(page)) {
7f59b277f79e8a Eric Biggers 2021-01-04 135 if (bio->bi_status)
6ce19aff0b8cd3 Chao Yu 2021-05-20 @136 f2fs_end_read_compressed_page(page, true, 0);
7f59b277f79e8a Eric Biggers 2021-01-04 @137 f2fs_put_page_dic(page);
4c8ff7095bef64 Chao Yu 2019-11-01 138 continue;
4c8ff7095bef64 Chao Yu 2019-11-01 139 }
4c8ff7095bef64 Chao Yu 2019-11-01 140
7f59b277f79e8a Eric Biggers 2021-01-04 141 /* PG_error was set if decryption or verity failed. */
6dbb17961f46b2 Eric Biggers 2018-04-18 142 if (bio->bi_status || PageError(page)) {
6dbb17961f46b2 Eric Biggers 2018-04-18 143 ClearPageUptodate(page);
fb7d70db305a14 Jaegeuk Kim 2018-09-25 144 /* will re-read again later */
fb7d70db305a14 Jaegeuk Kim 2018-09-25 145 ClearPageError(page);
6dbb17961f46b2 Eric Biggers 2018-04-18 146 } else {
6dbb17961f46b2 Eric Biggers 2018-04-18 147 SetPageUptodate(page);
6dbb17961f46b2 Eric Biggers 2018-04-18 148 }
5f9abab42b60e6 Jaegeuk Kim 2018-10-16 149 dec_page_count(F2FS_P_SB(page), __read_io_type(page));
6dbb17961f46b2 Eric Biggers 2018-04-18 150 unlock_page(page);
6dbb17961f46b2 Eric Biggers 2018-04-18 151 }
79bbefb19f1359 Chao Yu 2020-03-23 152
7f59b277f79e8a Eric Biggers 2021-01-04 153 if (bio->bi_private)
7f59b277f79e8a Eric Biggers 2021-01-04 154 mempool_free(bio->bi_private, bio_post_read_ctx_pool);
7f59b277f79e8a Eric Biggers 2021-01-04 155 bio_put(bio);
4c8ff7095bef64 Chao Yu 2019-11-01 156 }
4c8ff7095bef64 Chao Yu 2019-11-01 157
--
0-DAY CI Kernel Test Service
https://01.org/lkp
Powered by blists - more mailing lists