linux-kernel - drivers/platform/x86/socwatch/sw_pmt.c:232:8: sparse: sparse: symbol 's_endpoint_index' was not declared. Should it be static?

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [day] [month] [year] [list]

Date:   Wed, 27 Jul 2022 07:23:06 +0800
From:   kernel test robot <lkp@...el.com>
To:     Faycal Benmlih <faycal.benmlih@...el.com>
Cc:     kbuild-all@...ts.01.org, linux-kernel@...r.kernel.org
Subject: drivers/platform/x86/socwatch/sw_pmt.c:232:8: sparse: sparse: symbol
 's_endpoint_index' was not declared. Should it be static?

tree:   https://github.com/mchinth/linux sep_socwatch_linux_5_10
head:   c55df4a2b60fc7c6c4d00fd9f9fb447087833513
commit: cd0ae8e45c3cc66d1cfb9bfc5046d6a1e9b04d19 SEP/SOCWATCH Update SoCWatch driver after rebasing to linux v5.7
date:   1 year, 10 months ago
config: x86_64-randconfig-s022 (https://download.01.org/0day-ci/archive/20220727/202207270752.rRUYdUH1-lkp@intel.com/config)
compiler: gcc-11 (Debian 11.3.0-3) 11.3.0
reproduce:
        # apt-get install sparse
        # sparse version: v0.6.4-39-gce1a6720-dirty
        # https://github.com/mchinth/linux/commit/cd0ae8e45c3cc66d1cfb9bfc5046d6a1e9b04d19
        git remote add mchinth https://github.com/mchinth/linux
        git fetch --no-tags mchinth sep_socwatch_linux_5_10
        git checkout cd0ae8e45c3cc66d1cfb9bfc5046d6a1e9b04d19
        # save the config file
        mkdir build_dir && cp config build_dir/.config
        make W=1 C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' O=build_dir ARCH=x86_64 SHELL=/bin/bash

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

sparse warnings: (new ones prefixed by >>)
>> drivers/platform/x86/socwatch/sw_pmt.c:232:8: sparse: sparse: symbol 's_endpoint_index' was not declared. Should it be static?
--
>> drivers/platform/x86/socwatch/sw_pci.c:64:6: sparse: sparse: symbol 'sw_print_pci_devices_i' was not declared. Should it be static?

vim +/s_endpoint_index +232 drivers/platform/x86/socwatch/sw_pmt.c

4f5db8d9f7a900 drivers/platform/x86/socwatch/sw_cta.c Faycal Benmlih 2019-10-15  224  
4f5db8d9f7a900 drivers/platform/x86/socwatch/sw_cta.c Faycal Benmlih 2019-10-15  225  /* *********************************
cd0ae8e45c3cc6 drivers/platform/x86/socwatch/sw_pmt.c Faycal Benmlih 2020-05-01  226   * End PMT driver import
4f5db8d9f7a900 drivers/platform/x86/socwatch/sw_cta.c Faycal Benmlih 2019-10-15  227   * *********************************
4f5db8d9f7a900 drivers/platform/x86/socwatch/sw_cta.c Faycal Benmlih 2019-10-15  228   */
4f5db8d9f7a900 drivers/platform/x86/socwatch/sw_cta.c Faycal Benmlih 2019-10-15  229  
cd0ae8e45c3cc6 drivers/platform/x86/socwatch/sw_pmt.c Faycal Benmlih 2020-05-01  230  #define MAX_TELEM_ENDPOINTS MAX_AGGR_TELEM_ENDPOINTS /* For now */
4f5db8d9f7a900 drivers/platform/x86/socwatch/sw_cta.c Faycal Benmlih 2019-10-15  231  static struct telem_endpoint *s_telem_endpoints[MAX_TELEM_ENDPOINTS]; /* TODO: make this a linked list instead */
4f5db8d9f7a900 drivers/platform/x86/socwatch/sw_cta.c Faycal Benmlih 2019-10-15 @232  size_t s_endpoint_index = 0;
4f5db8d9f7a900 drivers/platform/x86/socwatch/sw_cta.c Faycal Benmlih 2019-10-15  233  

:::::: The code at line 232 was first introduced by commit
:::::: 4f5db8d9f7a900abd8cc574d4852045bca86c31f platform/x86: Update SoCWatch driver for 5.4 pull

:::::: TO: Faycal Benmlih <faycal.benmlih@...el.com>
:::::: CC: Faycal Benmlih <faycal.benmlih@...el.com>

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