Hi Viktor, Thank you for the patch! Yet something to improve: [auto build test ERROR on bpf-next/master] url: https://github.com/intel-lab-lkp/linux/commits/Viktor-Malik/bpf-Fix-attaching-fentry-fexit-fmod_ret-lsm-to-modules/20230310-154848 base: https://git.kernel.org/pub/scm/linux/kernel/git/bpf/bpf-next.git master patch link: https://lore.kernel.org/r/3f6a9d8ae850532b5ef864ef16327b0f7a669063.1678432753.git.vmalik%40redhat.com patch subject: [PATCH bpf-next v10 1/2] bpf: Fix attaching fentry/fexit/fmod_ret/lsm to modules config: arm-buildonly-randconfig-r005-20230312 (https://download.01.org/0day-ci/archive/20230316/202303160919.SGyfD0uE-lkp@xxxxxxxxx/config) compiler: clang version 17.0.0 (https://github.com/llvm/llvm-project 67409911353323ca5edf2049ef0df54132fa1ca7) 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 # install arm cross compiling tool for clang build # apt-get install binutils-arm-linux-gnueabi # https://github.com/intel-lab-lkp/linux/commit/a6713fb8bbf7954ee98fec48f2a1f1e33814d92a git remote add linux-review https://github.com/intel-lab-lkp/linux git fetch --no-tags linux-review Viktor-Malik/bpf-Fix-attaching-fentry-fexit-fmod_ret-lsm-to-modules/20230310-154848 git checkout a6713fb8bbf7954ee98fec48f2a1f1e33814d92a # save the config file mkdir build_dir && cp config build_dir/.config COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=arm olddefconfig COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=arm SHELL=/bin/bash kernel/ 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/202303160919.SGyfD0uE-lkp@xxxxxxxxx/ All errors (new ones prefixed by >>): In file included from kernel/bpf/verifier.c:27: >> kernel/bpf/../module/internal.h:260:14: error: expected ')' const char *name) ^ kernel/bpf/../module/internal.h:259:55: note: to match this '(' static inline unsigned long find_kallsyms_symbol_value(struct module *mod ^ >> kernel/bpf/verifier.c:18440:45: error: too many arguments to function call, expected single argument 'mod', have 2 arguments addr = find_kallsyms_symbol_value(mod, tname); ~~~~~~~~~~~~~~~~~~~~~~~~~~ ^~~~~ kernel/bpf/../module/internal.h:259:29: note: 'find_kallsyms_symbol_value' declared here static inline unsigned long find_kallsyms_symbol_value(struct module *mod ^ 2 errors generated. vim +260 kernel/bpf/../module/internal.h 250 251 static inline bool sect_empty(const Elf_Shdr *sect) 252 { 253 return !(sect->sh_flags & SHF_ALLOC) || sect->sh_size == 0; 254 } 255 #else /* !CONFIG_KALLSYMS */ 256 static inline void init_build_id(struct module *mod, const struct load_info *info) { } 257 static inline void layout_symtab(struct module *mod, struct load_info *info) { } 258 static inline void add_kallsyms(struct module *mod, const struct load_info *info) { } 259 static inline unsigned long find_kallsyms_symbol_value(struct module *mod > 260 const char *name) 261 { 262 return 0; 263 } 264 #endif /* CONFIG_KALLSYMS */ 265 -- 0-DAY CI Kernel Test Service https://github.com/intel/lkp-tests
