linux-kernel - arch/powerpc/mm/nohash/tlb_64e.c:36:5: sparse: sparse: symbol 'mmu_pte_psize' 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]

Message-ID: <202408011256.1O99IB0s-lkp@intel.com>
Date: Thu, 1 Aug 2024 13:06:46 +0800
From: kernel test robot <lkp@...el.com>
To: Michael Ellerman <mpe@...erman.id.au>
Cc: oe-kbuild-all@...ts.linux.dev, linux-kernel@...r.kernel.org,
	Andrew Morton <akpm@...ux-foundation.org>,
	Linux Memory Management List <linux-mm@...ck.org>,
	Christophe Leroy <christophe.leroy@...roup.eu>
Subject: arch/powerpc/mm/nohash/tlb_64e.c:36:5: sparse: sparse: symbol
 'mmu_pte_psize' was not declared. Should it be static?

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head:   21b136cc63d2a9ddd60d4699552b69c214b32964
commit: a898530eea3d0ba08c17a60865995a3bb468d1bc powerpc/64e: split out nohash Book3E 64-bit code
date:   3 weeks ago
config: powerpc64-randconfig-r122-20240801 (https://download.01.org/0day-ci/archive/20240801/202408011256.1O99IB0s-lkp@intel.com/config)
compiler: powerpc64-linux-gcc (GCC) 14.1.0
reproduce: (https://download.01.org/0day-ci/archive/20240801/202408011256.1O99IB0s-lkp@intel.com/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@...el.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202408011256.1O99IB0s-lkp@intel.com/

sparse warnings: (new ones prefixed by >>)
>> arch/powerpc/mm/nohash/tlb_64e.c:36:5: sparse: sparse: symbol 'mmu_pte_psize' was not declared. Should it be static?
   arch/powerpc/mm/nohash/tlb_64e.c: note: in included file (through include/linux/mmzone.h, include/linux/gfp.h, include/linux/mm.h):
   include/linux/page-flags.h:235:46: sparse: sparse: self-comparison always evaluates to false
   include/linux/page-flags.h:235:46: sparse: sparse: self-comparison always evaluates to false

vim +/mmu_pte_psize +36 arch/powerpc/mm/nohash/tlb_64e.c

    31	
    32	/* The variables below are currently only used on 64-bit Book3E
    33	 * though this will probably be made common with other nohash
    34	 * implementations at some point
    35	 */
  > 36	int mmu_pte_psize;		/* Page size used for PTE pages */
    37	int mmu_vmemmap_psize;		/* Page size used for the virtual mem map */
    38	int book3e_htw_mode;		/* HW tablewalk?  Value is PPC_HTW_* */
    39	unsigned long linear_map_top;	/* Top of linear mapping */
    40	
    41	

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki