Re: [PATCH v2 3/4] kbuild: re-implement CONFIG_TRIM_UNUSED_KSYMS to make it work in one-pass

From: kernel test robot <lkp@intel.com>
To: Masahiro Yamada <masahiroy@kernel.org>, linux-kbuild@vger.kernel.org
Cc: kbuild-all@lists.01.org, clang-built-linux@googlegroups.com,
	Christoph Hellwig <hch@lst.de>, Jessica Yu <jeyu@kernel.org>,
	Nicolas Pitre <nico@fluxnic.net>,
	linux-kernel@vger.kernel.org, linux-arch@vger.kernel.org,
	Masahiro Yamada <masahiroy@kernel.org>
Subject: Re: [PATCH v2 3/4] kbuild: re-implement CONFIG_TRIM_UNUSED_KSYMS to make it work in one-pass
Date: Wed, 17 Mar 2021 23:48:01 +0800	[thread overview]
Message-ID: <202103172304.lbqDHM7c-lkp@intel.com> (raw)
In-Reply-To: <20210309151737.345722-4-masahiroy@kernel.org>

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

Hi Masahiro,

I love your patch! Perhaps something to improve:

[auto build test WARNING on powerpc/next]
[also build test WARNING on linus/master v5.12-rc3]
[cannot apply to kbuild/for-next asm-generic/master next-20210317]
[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/Masahiro-Yamada/kbuild-build-speed-improvement-of-CONFIG_TRIM_UNUSED_KSYMS/20210309-232117
base:   https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git next
config: x86_64-randconfig-a015-20210317 (attached as .config)
compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project 8ef111222a3dd12a9175f69c3bff598c46e8bdf7)
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 x86_64 cross compiling tool for clang build
        # apt-get install binutils-x86-64-linux-gnu
        # https://github.com/0day-ci/linux/commit/331032950fb793dce926a30d68897756d504c4a9
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review Masahiro-Yamada/kbuild-build-speed-improvement-of-CONFIG_TRIM_UNUSED_KSYMS/20210309-232117
        git checkout 331032950fb793dce926a30d68897756d504c4a9
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=x86_64 

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

All warnings (new ones prefixed by >>):

>> drivers/staging/comedi/drivers/cb_pcidas64.c:232:19: warning: unused function 'analog_trig_low_threshold_bits' [-Wunused-function]
   static inline u16 analog_trig_low_threshold_bits(u16 threshold)
                     ^
>> drivers/staging/comedi/drivers/cb_pcidas64.c:383:28: warning: unused function 'dma_chain_flag_bits' [-Wunused-function]
   static inline unsigned int dma_chain_flag_bits(u16 prepost_bits)
                              ^
   2 warnings generated.
--
>> drivers/staging/rts5208/xd.c:34:19: warning: unused function 'xd_check_err_code' [-Wunused-function]
   static inline int xd_check_err_code(struct rtsx_chip *chip, u8 err_code)
                     ^
   1 warning generated.
--
>> drivers/video/fbdev/tridentfb.c:1127:20: warning: unused function 'shadowmode_off' [-Wunused-function]
   static inline void shadowmode_off(struct tridentfb_par *par)
                      ^
   1 warning generated.
--
>> drivers/video/fbdev/via/via-core.c:62:19: warning: unused function 'viafb_mmio_read' [-Wunused-function]
   static inline int viafb_mmio_read(int reg)
                     ^
   1 warning generated.
--
   mm/compaction.c:56:27: warning: unused variable 'HPAGE_FRAG_CHECK_INTERVAL_MSEC' [-Wunused-const-variable]
   static const unsigned int HPAGE_FRAG_CHECK_INTERVAL_MSEC = 500;
                             ^
>> mm/compaction.c:462:20: warning: unused function 'isolation_suitable' [-Wunused-function]
   static inline bool isolation_suitable(struct compact_control *cc,
                      ^
>> mm/compaction.c:468:20: warning: unused function 'pageblock_skip_persistent' [-Wunused-function]
   static inline bool pageblock_skip_persistent(struct page *page)
                      ^
>> mm/compaction.c:473:20: warning: unused function 'update_pageblock_skip' [-Wunused-function]
   static inline void update_pageblock_skip(struct compact_control *cc,
                      ^
   4 warnings generated.
--
>> mm/z3fold.c:287:37: warning: unused function 'handle_to_z3fold_header' [-Wunused-function]
   static inline struct z3fold_header *handle_to_z3fold_header(unsigned long h)
                                       ^
   1 warning generated.
--
>> security/apparmor/file.c:150:20: warning: unused function 'is_deleted' [-Wunused-function]
   static inline bool is_deleted(struct dentry *dentry)
                      ^
   1 warning generated.
--
>> security/apparmor/label.c:1258:20: warning: unused function 'label_is_visible' [-Wunused-function]
   static inline bool label_is_visible(struct aa_profile *profile,
                      ^
   1 warning generated.
--
>> drivers/hwmon/sis5595.c:158:18: warning: unused function 'DIV_TO_REG' [-Wunused-function]
   static inline u8 DIV_TO_REG(int val)
                    ^
   1 warning generated.
--
>> drivers/mfd/max8925-core.c:472:40: warning: unused function 'irq_to_max8925' [-Wunused-function]
   static inline struct max8925_irq_data *irq_to_max8925(struct max8925_chip *chip,
                                          ^
   1 warning generated.
--
>> drivers/misc/hpilo.c:395:19: warning: unused function 'is_device_reset' [-Wunused-function]
   static inline int is_device_reset(struct ilo_hwinfo *hw)
                     ^
   1 warning generated.
..

vim +/is_deleted +150 security/apparmor/file.c

6380bd8ddf613b John Johansen 2010-07-29  143  
aebd873e8d3e34 John Johansen 2017-06-09  144  /**
aebd873e8d3e34 John Johansen 2017-06-09  145   * is_deleted - test if a file has been completely unlinked
aebd873e8d3e34 John Johansen 2017-06-09  146   * @dentry: dentry of file to test for deletion  (NOT NULL)
aebd873e8d3e34 John Johansen 2017-06-09  147   *
e37986097ba63c Zou Wei       2020-04-28  148   * Returns: true if deleted else false
aebd873e8d3e34 John Johansen 2017-06-09  149   */
aebd873e8d3e34 John Johansen 2017-06-09 @150  static inline bool is_deleted(struct dentry *dentry)
aebd873e8d3e34 John Johansen 2017-06-09  151  {
aebd873e8d3e34 John Johansen 2017-06-09  152  	if (d_unlinked(dentry) && d_backing_inode(dentry)->i_nlink == 0)
e37986097ba63c Zou Wei       2020-04-28  153  		return true;
e37986097ba63c Zou Wei       2020-04-28  154  	return false;
aebd873e8d3e34 John Johansen 2017-06-09  155  }
aebd873e8d3e34 John Johansen 2017-06-09  156  

---
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: 38529 bytes --]