All of lore.kernel.org
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: cros-kernel-buildreports@googlegroups.com
Cc: oe-kbuild-all@lists.linux.dev
Subject: [android-common:android13-5.15 0/2] drivers/android/android_debug_symbols.c:27:6: sparse: sparse: symbol 'ads_page_owner' was not declared. Should it be static?
Date: Wed, 17 Dec 2025 19:11:38 +0800	[thread overview]
Message-ID: <202512171930.FWNm3OS7-lkp@intel.com> (raw)

tree:   https://android.googlesource.com/kernel/common android13-5.15
head:   d36711b0a31fb7c6682f53e306fa330cc0b125a2
commit: b2df67a932b5cbd6535de9f1a6e63004992ad014 [0/2] ANDROID: android: Create debug_symbols driver
config: i386-randconfig-061-20251217 (https://download.01.org/0day-ci/archive/20251217/202512171930.FWNm3OS7-lkp@intel.com/config)
compiler: gcc-13 (Debian 13.3.0-16) 13.3.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20251217/202512171930.FWNm3OS7-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@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202512171930.FWNm3OS7-lkp@intel.com/

sparse warnings: (new ones prefixed by >>)
>> drivers/android/android_debug_symbols.c:27:6: sparse: sparse: symbol 'ads_page_owner' was not declared. Should it be static?
>> drivers/android/android_debug_symbols.c:28:6: sparse: sparse: symbol 'ads_slub_debug' was not declared. Should it be static?
>> drivers/android/android_debug_symbols.c:29:15: sparse: sparse: symbol 'ads_vmalloc_nr_pages' was not declared. Should it be static?
>> drivers/android/android_debug_symbols.c:30:15: sparse: sparse: symbol 'ads_pcpu_nr_pages' was not declared. Should it be static?
>> drivers/android/android_debug_symbols.c:88:9: sparse: sparse: cast removes address space '__percpu' of expression

vim +/ads_page_owner +27 drivers/android/android_debug_symbols.c

    26	
  > 27	bool ads_page_owner;
  > 28	bool ads_slub_debug;
  > 29	unsigned long ads_vmalloc_nr_pages;
  > 30	unsigned long ads_pcpu_nr_pages;
    31	
    32	#define _ADS_ENTRY(index, symbol)			\
    33		[index] = { .name = #symbol, .addr = (void *)symbol }
    34	#define ADS_ENTRY(index, symbol) _ADS_ENTRY(index, symbol)
    35	
    36	#define _ADS_PER_CPU_ENTRY(index, symbol)			\
    37		[index] = { .name = #symbol, .addr = (void *)&symbol }
    38	#define ADS_PER_CPU_ENTRY(index, symbol) _ADS_PER_CPU_ENTRY(index, symbol)
    39	
    40	/*
    41	 * This module maintains static array of symbol and address information.
    42	 * Add all required core kernel symbols and their addresses into ads_entries[] array,
    43	 * so that vendor modules can query and to find address of non-exported symbol.
    44	 */
    45	static const struct ads_entry ads_entries[ADS_END] = {
    46		ADS_ENTRY(ADS_SDATA, _sdata),
    47		ADS_ENTRY(ADS_BSS_END, __bss_stop),
    48		ADS_ENTRY(ADS_PER_CPU_START, __per_cpu_start),
    49		ADS_ENTRY(ADS_PER_CPU_END, __per_cpu_end),
    50		ADS_ENTRY(ADS_START_RO_AFTER_INIT, __start_ro_after_init),
    51		ADS_ENTRY(ADS_END_RO_AFTER_INIT, __end_ro_after_init),
    52		ADS_ENTRY(ADS_LINUX_BANNER, linux_banner),
    53	#ifdef CONFIG_CMA
    54		ADS_ENTRY(ADS_TOTAL_CMA, &totalcma_pages),
    55	#endif
    56		ADS_ENTRY(ADS_SLAB_CACHES, &slab_caches),
    57		ADS_ENTRY(ADS_SLAB_MUTEX, &slab_mutex),
    58		ADS_ENTRY(ADS_MIN_LOW_PFN, &min_low_pfn),
    59		ADS_ENTRY(ADS_MAX_PFN, &max_pfn),
    60		ADS_ENTRY(ADS_VMALLOC_NR_PAGES, &ads_vmalloc_nr_pages),
    61		ADS_ENTRY(ADS_PCPU_NR_PAGES, &ads_pcpu_nr_pages),
    62	#ifdef CONFIG_PAGE_OWNER
    63		ADS_ENTRY(ADS_PAGE_OWNER_ENABLED, &ads_page_owner),
    64	#endif
    65	#ifdef CONFIG_SLUB_DEBUG
    66		ADS_ENTRY(ADS_SLUB_DEBUG, &ads_slub_debug),
    67	#endif
    68	#ifdef CONFIG_SWAP
    69		ADS_ENTRY(ADS_NR_SWAP_PAGES, &nr_swap_pages),
    70	#endif
    71	#ifdef CONFIG_MMU
    72		ADS_ENTRY(ADS_MMAP_MIN_ADDR, &mmap_min_addr),
    73	#endif
    74		ADS_ENTRY(ADS_STACK_GUARD_GAP, &stack_guard_gap),
    75	#ifdef CONFIG_SYSCTL
    76		ADS_ENTRY(ADS_SYSCTL_LEGACY_VA_LAYOUT, &sysctl_legacy_va_layout),
    77	#endif
    78	};
    79	
    80	/*
    81	 * ads_per_cpu_entries array contains all the per_cpu variable address information.
    82	 */
    83	static const struct ads_entry ads_per_cpu_entries[ADS_DEBUG_PER_CPU_END] = {
    84	#ifdef CONFIG_ARM64
    85		ADS_PER_CPU_ENTRY(ADS_IRQ_STACK_PTR, irq_stack_ptr),
    86	#endif
    87	#ifdef CONFIG_X86
  > 88		ADS_PER_CPU_ENTRY(ADS_IRQ_STACK_PTR, hardirq_stack_ptr),
    89	#endif
    90	};
    91	

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

             reply	other threads:[~2025-12-17 11:12 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-12-17 11:11 kernel test robot [this message]
  -- strict thread matches above, loose matches on Subject: below --
2025-12-17  1:36 [android-common:android13-5.15 0/2] drivers/android/android_debug_symbols.c:27:6: sparse: sparse: symbol 'ads_page_owner' was not declared. Should it be static? kernel test robot

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=202512171930.FWNm3OS7-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=cros-kernel-buildreports@googlegroups.com \
    --cc=oe-kbuild-all@lists.linux.dev \
    /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 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.