From: kernel test robot <lkp@intel.com> To: Dave Chinner <david@fromorbit.com>, linux-xfs@vger.kernel.org Cc: kbuild-all@lists.01.org, clang-built-linux@googlegroups.com Subject: Re: [PATCH 29/39] xfs: introduce per-cpu CIL tracking structure Date: Thu, 3 Jun 2021 17:18:00 +0800 [thread overview] Message-ID: <202106031736.pHZbABAz-lkp@intel.com> (raw) In-Reply-To: <20210603052240.171998-30-david@fromorbit.com> [-- Attachment #1: Type: text/plain, Size: 2329 bytes --] Hi Dave, Thank you for the patch! Yet something to improve: [auto build test ERROR on xfs-linux/for-next] [also build test ERROR on linus/master v5.13-rc4 next-20210602] [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/Dave-Chinner/xfs-CIL-and-log-optimisations/20210603-134113 base: https://git.kernel.org/pub/scm/fs/xfs/xfs-linux.git for-next config: arm-randconfig-r023-20210603 (attached as .config) compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project d8e0ae9a76a62bdc6117630d59bf9967ac9bb4ea) 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 # install arm cross compiling tool for clang build # apt-get install binutils-arm-linux-gnueabi # https://github.com/0day-ci/linux/commit/7832a61eefffdb48fdd863bbb0da4b8b7a4e2bb9 git remote add linux-review https://github.com/0day-ci/linux git fetch --no-tags linux-review Dave-Chinner/xfs-CIL-and-log-optimisations/20210603-134113 git checkout 7832a61eefffdb48fdd863bbb0da4b8b7a4e2bb9 # save the attached .config to linux build tree COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=arm 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/xfs/xfs_log_cil.c:1454:30: error: invalid operands to binary expression ('void' and 'int') if (xlog_cil_pcp_hpadd(cil) < 0) { ~~~~~~~~~~~~~~~~~~~~~~~ ^ ~ 1 error generated. vim +1454 fs/xfs/xfs_log_cil.c 1443 1444 static void __percpu * 1445 xlog_cil_pcp_alloc( 1446 struct xfs_cil *cil) 1447 { 1448 void __percpu *pcp; 1449 1450 pcp = alloc_percpu(struct xlog_cil_pcp); 1451 if (!pcp) 1452 return NULL; 1453 > 1454 if (xlog_cil_pcp_hpadd(cil) < 0) { 1455 free_percpu(pcp); 1456 return NULL; 1457 } 1458 return pcp; 1459 } 1460 --- 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: 31120 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: kernel test robot <lkp@intel.com> To: kbuild-all@lists.01.org Subject: Re: [PATCH 29/39] xfs: introduce per-cpu CIL tracking structure Date: Thu, 03 Jun 2021 17:18:00 +0800 [thread overview] Message-ID: <202106031736.pHZbABAz-lkp@intel.com> (raw) In-Reply-To: <20210603052240.171998-30-david@fromorbit.com> [-- Attachment #1: Type: text/plain, Size: 2390 bytes --] Hi Dave, Thank you for the patch! Yet something to improve: [auto build test ERROR on xfs-linux/for-next] [also build test ERROR on linus/master v5.13-rc4 next-20210602] [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/Dave-Chinner/xfs-CIL-and-log-optimisations/20210603-134113 base: https://git.kernel.org/pub/scm/fs/xfs/xfs-linux.git for-next config: arm-randconfig-r023-20210603 (attached as .config) compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project d8e0ae9a76a62bdc6117630d59bf9967ac9bb4ea) 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 # install arm cross compiling tool for clang build # apt-get install binutils-arm-linux-gnueabi # https://github.com/0day-ci/linux/commit/7832a61eefffdb48fdd863bbb0da4b8b7a4e2bb9 git remote add linux-review https://github.com/0day-ci/linux git fetch --no-tags linux-review Dave-Chinner/xfs-CIL-and-log-optimisations/20210603-134113 git checkout 7832a61eefffdb48fdd863bbb0da4b8b7a4e2bb9 # save the attached .config to linux build tree COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=arm 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/xfs/xfs_log_cil.c:1454:30: error: invalid operands to binary expression ('void' and 'int') if (xlog_cil_pcp_hpadd(cil) < 0) { ~~~~~~~~~~~~~~~~~~~~~~~ ^ ~ 1 error generated. vim +1454 fs/xfs/xfs_log_cil.c 1443 1444 static void __percpu * 1445 xlog_cil_pcp_alloc( 1446 struct xfs_cil *cil) 1447 { 1448 void __percpu *pcp; 1449 1450 pcp = alloc_percpu(struct xlog_cil_pcp); 1451 if (!pcp) 1452 return NULL; 1453 > 1454 if (xlog_cil_pcp_hpadd(cil) < 0) { 1455 free_percpu(pcp); 1456 return NULL; 1457 } 1458 return pcp; 1459 } 1460 --- 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: 31120 bytes --]
next prev parent reply other threads:[~2021-06-03 9:18 UTC|newest] Thread overview: 60+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-06-03 5:22 [PATCH 00/39 v5] xfs: CIL and log optimisations Dave Chinner 2021-06-03 5:22 ` [PATCH 01/39] xfs: log stripe roundoff is a property of the log Dave Chinner 2021-06-03 5:22 ` [PATCH 02/39] xfs: separate CIL commit record IO Dave Chinner 2021-06-03 5:22 ` [PATCH 03/39] xfs: remove xfs_blkdev_issue_flush Dave Chinner 2021-06-03 5:22 ` [PATCH 04/39] xfs: async blkdev cache flush Dave Chinner 2021-06-03 5:22 ` [PATCH 05/39] xfs: CIL checkpoint flushes caches unconditionally Dave Chinner 2021-06-03 5:22 ` [PATCH 06/39] xfs: remove need_start_rec parameter from xlog_write() Dave Chinner 2021-06-03 22:29 ` Allison Henderson 2021-06-03 5:22 ` [PATCH 07/39] xfs: journal IO cache flush reductions Dave Chinner 2021-06-03 22:29 ` Allison Henderson 2021-06-04 5:15 ` [xfs] 80d287de7b: stress-ng.dir.ops_per_sec 80.1% improvement kernel test robot 2021-06-04 5:15 ` kernel test robot 2021-06-03 5:22 ` [PATCH 08/39] xfs: Fix CIL throttle hang when CIL space used going backwards Dave Chinner 2021-06-03 22:29 ` Allison Henderson 2021-06-03 5:22 ` [PATCH 09/39] xfs: xfs_log_force_lsn isn't passed a LSN Dave Chinner 2021-06-03 22:29 ` Allison Henderson 2021-06-03 5:22 ` [PATCH 10/39] xfs: AIL needs asynchronous CIL forcing Dave Chinner 2021-06-03 22:29 ` Allison Henderson 2021-06-03 5:22 ` [PATCH 11/39] xfs: CIL work is serialised, not pipelined Dave Chinner 2021-06-03 5:22 ` [PATCH 12/39] xfs: factor out the CIL transaction header building Dave Chinner 2021-06-03 5:22 ` [PATCH 13/39] xfs: only CIL pushes require a start record Dave Chinner 2021-06-03 5:22 ` [PATCH 14/39] xfs: embed the xlog_op_header in the unmount record Dave Chinner 2021-06-03 5:22 ` [PATCH 15/39] xfs: embed the xlog_op_header in the commit record Dave Chinner 2021-06-03 5:22 ` [PATCH 16/39] xfs: log tickets don't need log client id Dave Chinner 2021-06-03 5:22 ` [PATCH 17/39] xfs: move log iovec alignment to preparation function Dave Chinner 2021-06-03 5:22 ` [PATCH 18/39] xfs: reserve space and initialise xlog_op_header in item formatting Dave Chinner 2021-06-03 5:22 ` [PATCH 19/39] xfs: log ticket region debug is largely useless Dave Chinner 2021-06-03 5:22 ` [PATCH 20/39] xfs: pass lv chain length into xlog_write() Dave Chinner 2021-06-03 5:22 ` [PATCH 21/39] xfs: introduce xlog_write_single() Dave Chinner 2021-06-03 5:22 ` [PATCH 22/39] xfs:_introduce xlog_write_partial() Dave Chinner 2021-06-03 5:22 ` [PATCH 23/39] xfs: xlog_write() no longer needs contwr state Dave Chinner 2021-06-03 5:22 ` [PATCH 24/39] xfs: xlog_write() doesn't need optype anymore Dave Chinner 2021-06-03 5:22 ` [PATCH 25/39] xfs: CIL context doesn't need to count iovecs Dave Chinner 2021-06-03 5:22 ` [PATCH 26/39] xfs: use the CIL space used counter for emptiness checks Dave Chinner 2021-06-03 5:22 ` [PATCH 27/39] xfs: lift init CIL reservation out of xc_cil_lock Dave Chinner 2021-06-03 5:22 ` [PATCH 28/39] xfs: rework per-iclog header CIL reservation Dave Chinner 2021-06-03 5:22 ` [PATCH 29/39] xfs: introduce per-cpu CIL tracking structure Dave Chinner 2021-06-03 9:18 ` kernel test robot [this message] 2021-06-03 9:18 ` kernel test robot 2021-06-03 10:08 ` kernel test robot 2021-06-03 10:08 ` kernel test robot 2021-06-03 16:49 ` Darrick J. Wong 2021-06-03 5:22 ` [PATCH 30/39] xfs: implement percpu cil space used calculation Dave Chinner 2021-06-03 16:44 ` Darrick J. Wong 2021-06-03 5:22 ` [PATCH 31/39] xfs: track CIL ticket reservation in percpu structure Dave Chinner 2021-06-03 16:43 ` Darrick J. Wong 2021-06-03 5:22 ` [PATCH 32/39] xfs: convert CIL busy extents to per-cpu Dave Chinner 2021-06-03 5:22 ` [PATCH 33/39] xfs: Add order IDs to log items in CIL Dave Chinner 2021-06-03 5:22 ` [PATCH 34/39] xfs: convert CIL to unordered per cpu lists Dave Chinner 2021-06-03 5:22 ` [PATCH 35/39] xfs: convert log vector chain to use list heads Dave Chinner 2021-06-03 10:16 ` kernel test robot 2021-06-03 10:16 ` kernel test robot 2021-06-03 5:22 ` [PATCH 36/39] xfs: move CIL ordering to the logvec chain Dave Chinner 2021-06-03 5:22 ` [PATCH 37/39] xfs: avoid cil push lock if possible Dave Chinner 2021-06-03 5:22 ` [PATCH 38/39] xfs: xlog_sync() manually adjusts grant head space Dave Chinner 2021-06-03 5:22 ` [PATCH 39/39] xfs: expanding delayed logging design with background material Dave Chinner 2021-06-03 17:05 ` [PATCH 00/39 v5] xfs: CIL and log optimisations Darrick J. Wong 2021-06-03 22:43 ` Dave Chinner -- strict thread matches above, loose matches on Subject: below -- 2021-05-19 12:12 [PATCH 00/39 v4] " Dave Chinner 2021-05-19 12:13 ` [PATCH 29/39] xfs: introduce per-cpu CIL tracking structure Dave Chinner 2021-05-27 18:31 ` Darrick J. Wong
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=202106031736.pHZbABAz-lkp@intel.com \ --to=lkp@intel.com \ --cc=clang-built-linux@googlegroups.com \ --cc=david@fromorbit.com \ --cc=kbuild-all@lists.01.org \ --cc=linux-xfs@vger.kernel.org \ /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: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.