Re: [PATCH v4 2/3] mm, page_owner: Add page_owner_stacks file to print out only stacks and their counte

From: kernel test robot
Date: Fri Apr 21 2023 - 09:26:27 EST


Hi Oscar,

kernel test robot noticed the following build warnings:

[auto build test WARNING on linus/master]
[also build test WARNING on v6.3-rc7]
[cannot apply to akpm-mm/mm-everything next-20230420]
[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#_base_tree_information]

url: https://github.com/intel-lab-lkp/linux/commits/Oscar-Salvador/lib-stackdepot-Add-a-refcount-field-in-stack_record/20230421-181709
patch link: https://lore.kernel.org/r/20230421101415.5734-3-osalvador%40suse.de
patch subject: [PATCH v4 2/3] mm, page_owner: Add page_owner_stacks file to print out only stacks and their counte
config: ia64-allyesconfig (https://download.01.org/0day-ci/archive/20230421/202304212153.jLLmROz6-lkp@xxxxxxxxx/config)
compiler: ia64-linux-gcc (GCC) 12.1.0
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
# https://github.com/intel-lab-lkp/linux/commit/6c6dfc43015e1939f433f4371d33418ab4d03411
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review Oscar-Salvador/lib-stackdepot-Add-a-refcount-field-in-stack_record/20230421-181709
git checkout 6c6dfc43015e1939f433f4371d33418ab4d03411
# save the config file
mkdir build_dir && cp config build_dir/.config
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 O=build_dir ARCH=ia64 olddefconfig
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 O=build_dir ARCH=ia64 SHELL=/bin/bash M=lib/

If you fix the issue, kindly add following tag where applicable
| Reported-by: kernel test robot <lkp@xxxxxxxxx>
| Link: https://lore.kernel.org/oe-kbuild-all/202304212153.jLLmROz6-lkp@xxxxxxxxx/

All warnings (new ones prefixed by >>):

In file included from include/linux/page_ext.h:7,
from include/linux/mm.h:22,
from include/linux/mman.h:5,
from lib/test_user_copy.c:13:
>> include/linux/stackdepot.h:120:26: warning: 'struct seq_file' declared inside parameter list will not be visible outside of this definition or declaration
120 | void *stack_start(struct seq_file *m, loff_t *ppos);
| ^~~~~~~~
include/linux/stackdepot.h:121:25: warning: 'struct seq_file' declared inside parameter list will not be visible outside of this definition or declaration
121 | void *stack_next(struct seq_file *m, void *v, loff_t *ppos);
| ^~~~~~~~
include/linux/stackdepot.h:122:24: warning: 'struct seq_file' declared inside parameter list will not be visible outside of this definition or declaration
122 | int stack_print(struct seq_file *m, void *v);
| ^~~~~~~~


vim +120 include/linux/stackdepot.h

99
100 /**
101 * stack_depot_save - Save a stack trace to stack depot
102 *
103 * @entries: Pointer to the stack trace
104 * @nr_entries: Number of frames in the stack
105 * @alloc_flags: Allocation GFP flags
106 *
107 * Context: Contexts where allocations via alloc_pages() are allowed.
108 * See __stack_depot_save() for more details.
109 *
110 * Return: Handle of the stack trace stored in depot, 0 on failure
111 */
112 depot_stack_handle_t stack_depot_save(unsigned long *entries,
113 unsigned int nr_entries, gfp_t gfp_flags);
114 depot_stack_handle_t stack_depot_save_action(unsigned long *entries,
115 unsigned int nr_entries,
116 gfp_t gfp_flags,
117 bool counter);
118
119 #ifdef CONFIG_PAGE_OWNER
> 120 void *stack_start(struct seq_file *m, loff_t *ppos);
121 void *stack_next(struct seq_file *m, void *v, loff_t *ppos);
122 int stack_print(struct seq_file *m, void *v);
123 #endif
124

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