From: kernel test robot <lkp@intel.com> To: Song Liu <songliubraving@fb.com>, bpf@vger.kernel.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org Cc: kbuild-all@lists.01.org, ast@kernel.org, daniel@iogearbox.net, kernel-team@fb.com, peterz@infradead.org, Song Liu <songliubraving@fb.com>, KP Singh <kpsingh@kernel.org> Subject: Re: [PATCH v4 bpf-next 1/6] bpf: enable task local storage for tracing programs Date: Tue, 23 Feb 2021 12:04:29 +0800 Message-ID: <202102231150.M7FKVKFw-lkp@intel.com> (raw) In-Reply-To: <20210223012014.2087583-2-songliubraving@fb.com> [-- Attachment #1: Type: text/plain, Size: 4376 bytes --] Hi Song, I love your patch! Yet something to improve: [auto build test ERROR on bpf-next/master] url: https://github.com/0day-ci/linux/commits/Song-Liu/bpf-enable-task-local-storage-for-tracing-programs/20210223-092439 base: https://git.kernel.org/pub/scm/linux/kernel/git/bpf/bpf-next.git master config: xtensa-randconfig-r034-20210222 (attached as .config) compiler: xtensa-linux-gcc (GCC) 9.3.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/0day-ci/linux/commit/a1afc383ea0cda5b0221b111f208140114752782 git remote add linux-review https://github.com/0day-ci/linux git fetch --no-tags linux-review Song-Liu/bpf-enable-task-local-storage-for-tracing-programs/20210223-092439 git checkout a1afc383ea0cda5b0221b111f208140114752782 # save the attached .config to linux build tree COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=xtensa If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <lkp@intel.com> All errors (new ones prefixed by >>): xtensa-linux-ld: kernel/bpf/bpf_task_storage.o: in function `notsupp_get_next_key': bpf_task_storage.c:(.text+0x14): undefined reference to `bpf_local_storage_lookup' >> xtensa-linux-ld: bpf_task_storage.c:(.text+0x18): undefined reference to `bpf_selem_unlink' xtensa-linux-ld: kernel/bpf/bpf_task_storage.o: in function `task_storage_delete': bpf_task_storage.c:(.text+0x32): undefined reference to `bpf_local_storage_lookup' xtensa-linux-ld: bpf_task_storage.c:(.text+0x3e): undefined reference to `bpf_selem_unlink' xtensa-linux-ld: kernel/bpf/bpf_task_storage.o: in function `bpf_pid_task_storage_delete_elem': bpf_task_storage.c:(.text+0x94): undefined reference to `bpf_local_storage_update' xtensa-linux-ld: kernel/bpf/bpf_task_storage.o: in function `bpf_pid_task_storage_update_elem': bpf_task_storage.c:(.text+0xce): undefined reference to `bpf_local_storage_update' >> xtensa-linux-ld: bpf_task_storage.c:(.text+0x12c): undefined reference to `bpf_local_storage_lookup' xtensa-linux-ld: kernel/bpf/bpf_task_storage.o: in function `bpf_pid_task_storage_lookup_elem': bpf_task_storage.c:(.text+0x148): undefined reference to `bpf_local_storage_cache_idx_free' >> xtensa-linux-ld: bpf_task_storage.c:(.text+0x14c): undefined reference to `bpf_local_storage_map_free' >> xtensa-linux-ld: bpf_task_storage.c:(.text+0x159): undefined reference to `bpf_local_storage_cache_idx_free' xtensa-linux-ld: bpf_task_storage.c:(.text+0x162): undefined reference to `bpf_local_storage_map_free' >> xtensa-linux-ld: bpf_task_storage.c:(.text+0x16c): undefined reference to `bpf_local_storage_map_alloc' >> xtensa-linux-ld: bpf_task_storage.c:(.text+0x170): undefined reference to `bpf_local_storage_cache_idx_get' xtensa-linux-ld: bpf_task_storage.c:(.text+0x17a): undefined reference to `bpf_local_storage_map_alloc' xtensa-linux-ld: bpf_task_storage.c:(.text+0x18b): undefined reference to `bpf_local_storage_cache_idx_get' xtensa-linux-ld: kernel/bpf/bpf_task_storage.o: in function `task_storage_map_free': bpf_task_storage.c:(.text+0x1d3): undefined reference to `bpf_local_storage_lookup' xtensa-linux-ld: kernel/bpf/bpf_task_storage.o: in function `task_storage_map_alloc': bpf_task_storage.c:(.text+0x1f6): undefined reference to `bpf_local_storage_update' xtensa-linux-ld: bpf_task_storage.c:(.text+0x208): undefined reference to `bpf_selem_unlink_map' >> xtensa-linux-ld: bpf_task_storage.c:(.text+0x20c): undefined reference to `bpf_selem_unlink_storage_nolock' xtensa-linux-ld: kernel/bpf/bpf_task_storage.o: in function `bpf_task_storage_get': bpf_task_storage.c:(.text+0x252): undefined reference to `bpf_selem_unlink_map' xtensa-linux-ld: bpf_task_storage.c:(.text+0x25e): undefined reference to `bpf_selem_unlink_storage_nolock' >> xtensa-linux-ld: kernel/bpf/bpf_task_storage.o:(.rodata+0x78): undefined reference to `bpf_local_storage_map_alloc_check' >> xtensa-linux-ld: kernel/bpf/bpf_task_storage.o:(.rodata+0xd0): undefined reference to `bpf_local_storage_map_check_btf' --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org [-- Attachment #2: .config.gz --] [-- Type: application/gzip, Size: 22009 bytes --]
next prev parent reply index Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-02-23 1:20 [PATCH v4 bpf-next 0/6] " Song Liu 2021-02-23 1:20 ` [PATCH v4 bpf-next 1/6] " Song Liu 2021-02-23 3:08 ` kernel test robot 2021-02-23 4:04 ` kernel test robot [this message] 2021-02-23 19:23 ` Martin KaFai Lau 2021-02-23 20:51 ` Song Liu 2021-02-23 1:20 ` [PATCH v4 bpf-next 2/6] bpf: prevent deadlock from recursive bpf_task_storage_[get|delete] Song Liu 2021-02-23 6:21 ` Andrii Nakryiko 2021-02-23 7:16 ` Song Liu 2021-02-23 7:19 ` Andrii Nakryiko 2021-02-23 16:44 ` Alexei Starovoitov 2021-02-23 11:06 ` Peter Zijlstra 2021-02-23 20:49 ` Song Liu 2021-02-23 1:20 ` [PATCH v4 bpf-next 3/6] selftests/bpf: add non-BPF_LSM test for task local storage Song Liu 2021-02-23 1:20 ` [PATCH v4 bpf-next 4/6] selftests/bpf: test deadlock from recursive bpf_task_storage_[get|delete] Song Liu 2021-02-23 1:20 ` [PATCH v4 bpf-next 5/6] bpf: runqslower: prefer using local vmlimux to generate vmlinux.h Song Liu 2021-02-23 6:26 ` Andrii Nakryiko 2021-02-23 21:24 ` Martin KaFai Lau 2021-02-23 1:20 ` [PATCH v4 bpf-next 6/6] bpf: runqslower: use task local storage Song Liu 2021-02-23 21:33 ` Martin KaFai Lau
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=202102231150.M7FKVKFw-lkp@intel.com \ --to=lkp@intel.com \ --cc=ast@kernel.org \ --cc=bpf@vger.kernel.org \ --cc=daniel@iogearbox.net \ --cc=kbuild-all@lists.01.org \ --cc=kernel-team@fb.com \ --cc=kpsingh@kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=netdev@vger.kernel.org \ --cc=peterz@infradead.org \ --cc=songliubraving@fb.com \ /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
BPF Archive on lore.kernel.org Archives are clonable: git clone --mirror https://lore.kernel.org/bpf/0 bpf/git/0.git # If you have public-inbox 1.1+ installed, you may # initialize and index your mirror using the following commands: public-inbox-init -V2 bpf bpf/ https://lore.kernel.org/bpf \ bpf@vger.kernel.org public-inbox-index bpf Example config snippet for mirrors Newsgroup available over NNTP: nntp://nntp.lore.kernel.org/org.kernel.vger.bpf AGPL code for this site: git clone https://public-inbox.org/public-inbox.git