* [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?
@ 2025-12-17 1:36 kernel test robot
0 siblings, 0 replies; 2+ messages in thread
From: kernel test robot @ 2025-12-17 1:36 UTC (permalink / raw)
To: cros-kernel-buildreports; +Cc: oe-kbuild-all
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/202512170907.0KS5mX6u-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/202512170907.0KS5mX6u-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/202512170907.0KS5mX6u-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
^ permalink raw reply [flat|nested] 2+ messages in thread
* [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?
@ 2025-12-17 11:11 kernel test robot
0 siblings, 0 replies; 2+ messages in thread
From: kernel test robot @ 2025-12-17 11:11 UTC (permalink / raw)
To: cros-kernel-buildreports; +Cc: oe-kbuild-all
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
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2025-12-17 11:12 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-12-17 11:11 [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
-- strict thread matches above, loose matches on Subject: below --
2025-12-17 1:36 kernel test robot
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.