lkml.org 
[lkml]   [2022]   [Jul]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
Subject[xiang:erofs/folios 1/11] fs/erofs/zdata.c:1475:26: warning: the address of 'pagepool' will always evaluate as 'true'
tree:   https://git.kernel.org/pub/scm/linux/kernel/git/xiang/linux.git erofs/folios
head: c94ebb8be9f7900ed0aa53a0638d119c4121ed1f
commit: af9c47cccaca8f1f3eb1de21328804b391edaaca [1/11] erofs: get rid of unneeded `inode', `map' and `sb'
config: m68k-allyesconfig (https://download.01.org/0day-ci/archive/20220704/202207040757.5bj5YTel-lkp@intel.com/config)
compiler: m68k-linux-gcc (GCC) 11.3.0
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# https://git.kernel.org/pub/scm/linux/kernel/git/xiang/linux.git/commit/?id=af9c47cccaca8f1f3eb1de21328804b391edaaca
git remote add xiang https://git.kernel.org/pub/scm/linux/kernel/git/xiang/linux.git
git fetch --no-tags xiang erofs/folios
git checkout af9c47cccaca8f1f3eb1de21328804b391edaaca
# save the config file
mkdir build_dir && cp config build_dir/.config
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.3.0 make.cross W=1 O=build_dir ARCH=m68k SHELL=/bin/bash fs/erofs/

If you fix the issue, kindly add following tag where applicable
Reported-by: kernel test robot <lkp@intel.com>

All warnings (new ones prefixed by >>):

fs/erofs/zdata.c: In function 'z_erofs_read_folio':
fs/erofs/zdata.c:1474:31: error: passing argument 1 of 'z_erofs_runqueue' from incompatible pointer type [-Werror=incompatible-pointer-types]
1474 | z_erofs_runqueue(inode->i_sb, &f, &pagepool,
| ~~~~~^~~~~~
| |
| struct super_block *
fs/erofs/zdata.c:1369:66: note: expected 'struct z_erofs_decompress_frontend *' but argument is of type 'struct super_block *'
1369 | static void z_erofs_runqueue(struct z_erofs_decompress_frontend *f,
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^
fs/erofs/zdata.c:1474:39: error: passing argument 2 of 'z_erofs_runqueue' from incompatible pointer type [-Werror=incompatible-pointer-types]
1474 | z_erofs_runqueue(inode->i_sb, &f, &pagepool,
| ^~
| |
| struct z_erofs_decompress_frontend *
fs/erofs/zdata.c:1370:44: note: expected 'struct page **' but argument is of type 'struct z_erofs_decompress_frontend *'
1370 | struct page **pagepool, bool force_fg)
| ~~~~~~~~~~~~~~^~~~~~~~
>> fs/erofs/zdata.c:1475:26: warning: the address of 'pagepool' will always evaluate as 'true' [-Waddress]
1475 | z_erofs_get_sync_decompress_policy(sbi, 0));
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
fs/erofs/zdata.c:1474:9: error: too many arguments to function 'z_erofs_runqueue'
1474 | z_erofs_runqueue(inode->i_sb, &f, &pagepool,
| ^~~~~~~~~~~~~~~~
fs/erofs/zdata.c:1369:13: note: declared here
1369 | static void z_erofs_runqueue(struct z_erofs_decompress_frontend *f,
| ^~~~~~~~~~~~~~~~
cc1: some warnings being treated as errors


vim +1475 fs/erofs/zdata.c

386292919c255d fs/erofs/zdata.c Gao Xiang 2021-10-09 1453
a2e20a25a1470e fs/erofs/zdata.c Matthew Wilcox (Oracle 2022-04-29 1454) static int z_erofs_read_folio(struct file *file, struct folio *folio)
3883a79abd0227 drivers/staging/erofs/unzip_vle.c Gao Xiang 2018-07-26 1455 {
a2e20a25a1470e fs/erofs/zdata.c Matthew Wilcox (Oracle 2022-04-29 1456) struct page *page = &folio->page;
3883a79abd0227 drivers/staging/erofs/unzip_vle.c Gao Xiang 2018-07-26 1457 struct inode *const inode = page->mapping->host;
40452ffca3c1a0 fs/erofs/zdata.c Huang Jianan 2021-12-06 1458 struct erofs_sb_info *const sbi = EROFS_I_SB(inode);
97e86a858bc360 drivers/staging/erofs/zdata.c Gao Xiang 2019-07-31 1459 struct z_erofs_decompress_frontend f = DECOMPRESS_FRONTEND_INIT(inode);
eaa9172ad988b3 fs/erofs/zdata.c Gao Xiang 2021-10-22 1460 struct page *pagepool = NULL;
3883a79abd0227 drivers/staging/erofs/unzip_vle.c Gao Xiang 2018-07-26 1461 int err;
3883a79abd0227 drivers/staging/erofs/unzip_vle.c Gao Xiang 2018-07-26 1462
ba9ce771b018c5 drivers/staging/erofs/unzip_vle.c Gao Xiang 2018-11-23 1463 trace_erofs_readpage(page, false);
f0c519fc2679ca drivers/staging/erofs/unzip_vle.c Gao Xiang 2018-11-23 1464 f.headoffset = (erofs_off_t)page->index << PAGE_SHIFT;
f0c519fc2679ca drivers/staging/erofs/unzip_vle.c Gao Xiang 2018-11-23 1465
386292919c255d fs/erofs/zdata.c Gao Xiang 2021-10-09 1466 z_erofs_pcluster_readmore(&f, NULL, f.headoffset + PAGE_SIZE - 1,
386292919c255d fs/erofs/zdata.c Gao Xiang 2021-10-09 1467 &pagepool, true);
1825c8d7ce93c4 fs/erofs/zdata.c Gao Xiang 2020-12-09 1468 err = z_erofs_do_read_page(&f, page, &pagepool);
386292919c255d fs/erofs/zdata.c Gao Xiang 2021-10-09 1469 z_erofs_pcluster_readmore(&f, NULL, 0, &pagepool, false);
386292919c255d fs/erofs/zdata.c Gao Xiang 2021-10-09 1470
5c6dcc57e2e505 fs/erofs/zdata.c Gao Xiang 2022-03-02 1471 (void)z_erofs_collector_end(&f);
3883a79abd0227 drivers/staging/erofs/unzip_vle.c Gao Xiang 2018-07-26 1472
ee45197c807895 drivers/staging/erofs/zdata.c Gao Xiang 2019-08-19 1473 /* if some compressed cluster ready, need submit them anyway */
40452ffca3c1a0 fs/erofs/zdata.c Huang Jianan 2021-12-06 1474 z_erofs_runqueue(inode->i_sb, &f, &pagepool,
40452ffca3c1a0 fs/erofs/zdata.c Huang Jianan 2021-12-06 @1475 z_erofs_get_sync_decompress_policy(sbi, 0));
ee45197c807895 drivers/staging/erofs/zdata.c Gao Xiang 2019-08-19 1476
ee45197c807895 drivers/staging/erofs/zdata.c Gao Xiang 2019-08-19 1477 if (err)
4f761fa253b49f fs/erofs/zdata.c Gao Xiang 2019-09-04 1478 erofs_err(inode->i_sb, "failed to read, err [%d]", err);
3883a79abd0227 drivers/staging/erofs/unzip_vle.c Gao Xiang 2018-07-26 1479
09c543798c3cde fs/erofs/zdata.c Gao Xiang 2022-01-02 1480 erofs_put_metabuf(&f.map.buf);
eaa9172ad988b3 fs/erofs/zdata.c Gao Xiang 2021-10-22 1481 erofs_release_pages(&pagepool);
ee45197c807895 drivers/staging/erofs/zdata.c Gao Xiang 2019-08-19 1482 return err;
3883a79abd0227 drivers/staging/erofs/unzip_vle.c Gao Xiang 2018-07-26 1483 }
3883a79abd0227 drivers/staging/erofs/unzip_vle.c Gao Xiang 2018-07-26 1484

:::::: The code at line 1475 was first introduced by commit
:::::: 40452ffca3c1a0f2994e826f9fa213b107f1a2d4 erofs: add sysfs node to control sync decompression strategy

:::::: TO: Huang Jianan <huangjianan@oppo.com>
:::::: CC: Gao Xiang <hsiangkao@linux.alibaba.com>

--
0-DAY CI Kernel Test Service
https://01.org/lkp

\
 
 \ /
  Last update: 2022-07-04 01:19    [W:7.607 / U:0.016 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site