linux-erofs.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: Huang Jianan <huangjianan@oppo.com>, linux-erofs@lists.ozlabs.org
Cc: zhangshiming@oppo.com, guoweichao@oppo.com, kbuild-all@01.org,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH 2/2] erofs: decompress in endio if possible
Date: Fri, 5 Mar 2021 16:21:35 +0800	[thread overview]
Message-ID: <202103051610.dZK8qRsp-lkp@intel.com> (raw)
In-Reply-To: <20210305062219.557128-2-huangjianan@oppo.com>

[-- Attachment #1: Type: text/plain, Size: 4347 bytes --]

Hi Huang,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on xiang-erofs/dev-test]
[cannot apply to v5.12-rc1 next-20210305]
[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]

url:    https://github.com/0day-ci/linux/commits/Huang-Jianan/erofs-avoid-memory-allocation-failure-during-rolling-decompression/20210305-142329
base:   https://git.kernel.org/pub/scm/linux/kernel/git/xiang/erofs.git dev-test
config: parisc-randconfig-r012-20210305 (attached as .config)
compiler: hppa64-linux-gcc (GCC) 9.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://github.com/0day-ci/linux/commit/ecb1173ba28392f322cbb4c9cb1a66524b10dd78
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review Huang-Jianan/erofs-avoid-memory-allocation-failure-during-rolling-decompression/20210305-142329
        git checkout ecb1173ba28392f322cbb4c9cb1a66524b10dd78
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=parisc 

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

All errors (new ones prefixed by >>):

   fs/erofs/super.c: In function 'erofs_read_superblock':
>> fs/erofs/super.c:191:5: error: 'struct erofs_sb_info' has no member named 'sync_decompress'
     191 |  sbi->sync_decompress = false;
         |     ^~


vim +191 fs/erofs/super.c

   124	
   125	static int erofs_read_superblock(struct super_block *sb)
   126	{
   127		struct erofs_sb_info *sbi;
   128		struct page *page;
   129		struct erofs_super_block *dsb;
   130		unsigned int blkszbits;
   131		void *data;
   132		int ret;
   133	
   134		page = read_mapping_page(sb->s_bdev->bd_inode->i_mapping, 0, NULL);
   135		if (IS_ERR(page)) {
   136			erofs_err(sb, "cannot read erofs superblock");
   137			return PTR_ERR(page);
   138		}
   139	
   140		sbi = EROFS_SB(sb);
   141	
   142		data = kmap(page);
   143		dsb = (struct erofs_super_block *)(data + EROFS_SUPER_OFFSET);
   144	
   145		ret = -EINVAL;
   146		if (le32_to_cpu(dsb->magic) != EROFS_SUPER_MAGIC_V1) {
   147			erofs_err(sb, "cannot find valid erofs superblock");
   148			goto out;
   149		}
   150	
   151		sbi->feature_compat = le32_to_cpu(dsb->feature_compat);
   152		if (sbi->feature_compat & EROFS_FEATURE_COMPAT_SB_CHKSUM) {
   153			ret = erofs_superblock_csum_verify(sb, data);
   154			if (ret)
   155				goto out;
   156		}
   157	
   158		blkszbits = dsb->blkszbits;
   159		/* 9(512 bytes) + LOG_SECTORS_PER_BLOCK == LOG_BLOCK_SIZE */
   160		if (blkszbits != LOG_BLOCK_SIZE) {
   161			erofs_err(sb, "blkszbits %u isn't supported on this platform",
   162				  blkszbits);
   163			goto out;
   164		}
   165	
   166		if (!check_layout_compatibility(sb, dsb))
   167			goto out;
   168	
   169		sbi->blocks = le32_to_cpu(dsb->blocks);
   170		sbi->meta_blkaddr = le32_to_cpu(dsb->meta_blkaddr);
   171	#ifdef CONFIG_EROFS_FS_XATTR
   172		sbi->xattr_blkaddr = le32_to_cpu(dsb->xattr_blkaddr);
   173	#endif
   174		sbi->islotbits = ilog2(sizeof(struct erofs_inode_compact));
   175		sbi->root_nid = le16_to_cpu(dsb->root_nid);
   176		sbi->inos = le64_to_cpu(dsb->inos);
   177	
   178		sbi->build_time = le64_to_cpu(dsb->build_time);
   179		sbi->build_time_nsec = le32_to_cpu(dsb->build_time_nsec);
   180	
   181		memcpy(&sb->s_uuid, dsb->uuid, sizeof(dsb->uuid));
   182	
   183		ret = strscpy(sbi->volume_name, dsb->volume_name,
   184			      sizeof(dsb->volume_name));
   185		if (ret < 0) {	/* -E2BIG */
   186			erofs_err(sb, "bad volume name without NIL terminator");
   187			ret = -EFSCORRUPTED;
   188			goto out;
   189		}
   190	
 > 191		sbi->sync_decompress = false;
   192		/* parse on-disk compression configurations */
   193		z_erofs_load_lz4_config(sbi, dsb);
   194		ret = 0;
   195	out:
   196		kunmap(page);
   197		put_page(page);
   198		return ret;
   199	}
   200	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 20631 bytes --]

  parent reply	other threads:[~2021-03-05  8:23 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-03-05  6:22 [PATCH 1/2] erofs: avoid memory allocation failure during rolling decompression Huang Jianan via Linux-erofs
2021-03-05  6:22 ` [PATCH 2/2] erofs: decompress in endio if possible Huang Jianan via Linux-erofs
2021-03-05  6:41   ` Gao Xiang
2021-03-05  8:21   ` kernel test robot [this message]
2021-03-05  6:45 ` [PATCH 1/2] erofs: avoid memory allocation failure during rolling decompression Gao Xiang
2021-03-05  9:58 ` [PATCH v5 " Huang Jianan via Linux-erofs
2021-03-05  9:58   ` [PATCH v5 2/2] erofs: decompress in endio if possible Huang Jianan via Linux-erofs
2021-03-15 11:28     ` Gao Xiang
2021-03-15 11:24   ` [PATCH v5 1/2] erofs: avoid memory allocation failure during rolling decompression Gao Xiang
2021-03-16  1:11   ` Chao Yu
2021-03-16  1:29     ` Gao Xiang

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=202103051610.dZK8qRsp-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=guoweichao@oppo.com \
    --cc=huangjianan@oppo.com \
    --cc=kbuild-all@01.org \
    --cc=linux-erofs@lists.ozlabs.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=zhangshiming@oppo.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).