From: kernel test robot <lkp@intel.com>
To: Tiezhu Yang <yangtiezhu@loongson.cn>,
Alexei Starovoitov <ast@kernel.org>,
Daniel Borkmann <daniel@iogearbox.net>,
Andrii Nakryiko <andrii@kernel.org>
Cc: llvm@lists.linux.dev, oe-kbuild-all@lists.linux.dev,
bpf@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH bpf-next 2/4] bpf: treewide: Clean up BPF_ALU and BPF_JMP
Date: Thu, 2 Feb 2023 12:01:50 +0800 [thread overview]
Message-ID: <202302021105.HU4McFMJ-lkp@intel.com> (raw)
In-Reply-To: <1675254998-4951-3-git-send-email-yangtiezhu@loongson.cn>
Hi Tiezhu,
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/Tiezhu-Yang/bpf-Add-new-macro-BPF_ALU32-and-BPF_JMP64/20230201-203836
base: https://git.kernel.org/pub/scm/linux/kernel/git/bpf/bpf-next.git master
patch link: https://lore.kernel.org/r/1675254998-4951-3-git-send-email-yangtiezhu%40loongson.cn
patch subject: [PATCH bpf-next 2/4] bpf: treewide: Clean up BPF_ALU and BPF_JMP
config: i386-randconfig-a016-20230130 (https://download.01.org/0day-ci/archive/20230202/202302021105.HU4McFMJ-lkp@intel.com/config)
compiler: clang version 14.0.6 (https://github.com/llvm/llvm-project f28c006a5895fc0e329fe15fead81e37457cb1d1)
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/9955f957cb218f711161ac710656be1164eaa3a3
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review Tiezhu-Yang/bpf-Add-new-macro-BPF_ALU32-and-BPF_JMP64/20230201-203836
git checkout 9955f957cb218f711161ac710656be1164eaa3a3
# 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=i386 olddefconfig
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=i386 SHELL=/bin/bash samples/
If you fix the issue, kindly add following tag where applicable
| Reported-by: kernel test robot <lkp@intel.com>
All errors (new ones prefixed by >>):
>> samples/seccomp/bpf-fancy.c:38:3: error: use of undeclared identifier 'BPF_JMP64'
SYSCALL(__NR_exit, ALLOW),
^
samples/seccomp/bpf-helper.h:56:11: note: expanded from macro 'SYSCALL'
BPF_JUMP(BPF_JMP64+BPF_JEQ+BPF_K, (nr), 0, 1), \
^
samples/seccomp/bpf-fancy.c:39:3: error: use of undeclared identifier 'BPF_JMP64'
SYSCALL(__NR_exit_group, ALLOW),
^
samples/seccomp/bpf-helper.h:56:11: note: expanded from macro 'SYSCALL'
BPF_JUMP(BPF_JMP64+BPF_JEQ+BPF_K, (nr), 0, 1), \
^
samples/seccomp/bpf-fancy.c:40:3: error: use of undeclared identifier 'BPF_JMP64'
SYSCALL(__NR_write, JUMP(&l, write_fd)),
^
samples/seccomp/bpf-helper.h:56:11: note: expanded from macro 'SYSCALL'
BPF_JUMP(BPF_JMP64+BPF_JEQ+BPF_K, (nr), 0, 1), \
^
samples/seccomp/bpf-fancy.c:40:23: error: use of undeclared identifier 'BPF_JMP64'
SYSCALL(__NR_write, JUMP(&l, write_fd)),
^
samples/seccomp/bpf-helper.h:50:11: note: expanded from macro 'JUMP'
BPF_JUMP(BPF_JMP64+BPF_JA, FIND_LABEL((labels), (label)), \
^
samples/seccomp/bpf-fancy.c:41:3: error: use of undeclared identifier 'BPF_JMP64'
SYSCALL(__NR_read, JUMP(&l, read)),
^
samples/seccomp/bpf-helper.h:56:11: note: expanded from macro 'SYSCALL'
BPF_JUMP(BPF_JMP64+BPF_JEQ+BPF_K, (nr), 0, 1), \
^
samples/seccomp/bpf-fancy.c:41:22: error: use of undeclared identifier 'BPF_JMP64'
SYSCALL(__NR_read, JUMP(&l, read)),
^
samples/seccomp/bpf-helper.h:50:11: note: expanded from macro 'JUMP'
BPF_JUMP(BPF_JMP64+BPF_JA, FIND_LABEL((labels), (label)), \
^
samples/seccomp/bpf-fancy.c:44:3: error: use of undeclared identifier 'BPF_JMP64'
LABEL(&l, read),
^
samples/seccomp/bpf-helper.h:53:11: note: expanded from macro 'LABEL'
BPF_JUMP(BPF_JMP64+BPF_JA, FIND_LABEL((labels), (label)), \
^
samples/seccomp/bpf-fancy.c:46:3: error: use of undeclared identifier 'BPF_JMP64'
JNE(STDIN_FILENO, DENY),
^
samples/seccomp/bpf-helper.h:77:20: note: expanded from macro 'JNE'
#define JNE(x, jt) JNE32(x, EXPAND(jt))
^
samples/seccomp/bpf-helper.h:154:11: note: expanded from macro 'JNE32'
BPF_JUMP(BPF_JMP64+BPF_JEQ+BPF_K, (value), 1, 0), \
^
samples/seccomp/bpf-fancy.c:48:3: error: use of undeclared identifier 'BPF_JMP64'
JNE((unsigned long)buf, DENY),
^
samples/seccomp/bpf-helper.h:77:20: note: expanded from macro 'JNE'
#define JNE(x, jt) JNE32(x, EXPAND(jt))
^
samples/seccomp/bpf-helper.h:154:11: note: expanded from macro 'JNE32'
BPF_JUMP(BPF_JMP64+BPF_JEQ+BPF_K, (value), 1, 0), \
^
samples/seccomp/bpf-fancy.c:50:3: error: use of undeclared identifier 'BPF_JMP64'
JGE(sizeof(buf), DENY),
^
samples/seccomp/bpf-helper.h:80:20: note: expanded from macro 'JGE'
#define JGE(x, jt) JGE32(x, EXPAND(jt))
^
samples/seccomp/bpf-helper.h:162:11: note: expanded from macro 'JGE32'
BPF_JUMP(BPF_JMP64+BPF_JGE+BPF_K, (value), 0, 1), \
^
samples/seccomp/bpf-fancy.c:53:3: error: use of undeclared identifier 'BPF_JMP64'
LABEL(&l, write_fd),
^
samples/seccomp/bpf-helper.h:53:11: note: expanded from macro 'LABEL'
BPF_JUMP(BPF_JMP64+BPF_JA, FIND_LABEL((labels), (label)), \
^
samples/seccomp/bpf-fancy.c:55:3: error: use of undeclared identifier 'BPF_JMP64'
JEQ(STDOUT_FILENO, JUMP(&l, write_buf)),
^
samples/seccomp/bpf-helper.h:76:20: note: expanded from macro 'JEQ'
#define JEQ(x, jt) JEQ32(x, EXPAND(jt))
^
samples/seccomp/bpf-helper.h:150:11: note: expanded from macro 'JEQ32'
BPF_JUMP(BPF_JMP64+BPF_JEQ+BPF_K, (value), 0, 1), \
^
samples/seccomp/bpf-fancy.c:55:22: error: use of undeclared identifier 'BPF_JMP64'
JEQ(STDOUT_FILENO, JUMP(&l, write_buf)),
^
samples/seccomp/bpf-helper.h:50:11: note: expanded from macro 'JUMP'
BPF_JUMP(BPF_JMP64+BPF_JA, FIND_LABEL((labels), (label)), \
^
samples/seccomp/bpf-fancy.c:56:3: error: use of undeclared identifier 'BPF_JMP64'
JEQ(STDERR_FILENO, JUMP(&l, write_buf)),
^
samples/seccomp/bpf-helper.h:76:20: note: expanded from macro 'JEQ'
#define JEQ(x, jt) JEQ32(x, EXPAND(jt))
^
samples/seccomp/bpf-helper.h:150:11: note: expanded from macro 'JEQ32'
BPF_JUMP(BPF_JMP64+BPF_JEQ+BPF_K, (value), 0, 1), \
^
samples/seccomp/bpf-fancy.c:56:22: error: use of undeclared identifier 'BPF_JMP64'
JEQ(STDERR_FILENO, JUMP(&l, write_buf)),
--
>> samples/seccomp/bpf-helper.c:33:23: error: use of undeclared identifier 'BPF_JMP64'
if (instr->code != (BPF_JMP64+BPF_JA))
^
1 error generated.
--
>> samples/seccomp/dropper.c:33:12: error: use of undeclared identifier 'BPF_JMP64'
BPF_JUMP(BPF_JMP64+BPF_JEQ+BPF_K, arch, 0, 3),
^
samples/seccomp/dropper.c:36:12: error: use of undeclared identifier 'BPF_JMP64'
BPF_JUMP(BPF_JMP64+BPF_JEQ+BPF_K, nr, 0, 1),
^
>> samples/seccomp/dropper.c:42:33: error: invalid application of 'sizeof' to an incomplete type 'struct sock_filter[]'
.len = (unsigned short)(sizeof(filter)/sizeof(filter[0])),
^~~~~~~~
3 errors generated.
--
>> samples/seccomp/bpf-direct.c:117:12: error: use of undeclared identifier 'BPF_JMP64'
BPF_JUMP(BPF_JMP64+BPF_JEQ+BPF_K, __NR_rt_sigreturn, 0, 1),
^
samples/seccomp/bpf-direct.c:120:12: error: use of undeclared identifier 'BPF_JMP64'
BPF_JUMP(BPF_JMP64+BPF_JEQ+BPF_K, __NR_sigreturn, 0, 1),
^
samples/seccomp/bpf-direct.c:123:12: error: use of undeclared identifier 'BPF_JMP64'
BPF_JUMP(BPF_JMP64+BPF_JEQ+BPF_K, __NR_exit_group, 0, 1),
^
samples/seccomp/bpf-direct.c:125:12: error: use of undeclared identifier 'BPF_JMP64'
BPF_JUMP(BPF_JMP64+BPF_JEQ+BPF_K, __NR_exit, 0, 1),
^
samples/seccomp/bpf-direct.c:127:12: error: use of undeclared identifier 'BPF_JMP64'
BPF_JUMP(BPF_JMP64+BPF_JEQ+BPF_K, __NR_read, 1, 0),
^
samples/seccomp/bpf-direct.c:128:12: error: use of undeclared identifier 'BPF_JMP64'
BPF_JUMP(BPF_JMP64+BPF_JEQ+BPF_K, __NR_write, 3, 2),
^
samples/seccomp/bpf-direct.c:132:12: error: use of undeclared identifier 'BPF_JMP64'
BPF_JUMP(BPF_JMP64+BPF_JEQ+BPF_K, STDIN_FILENO, 4, 0),
^
samples/seccomp/bpf-direct.c:137:12: error: use of undeclared identifier 'BPF_JMP64'
BPF_JUMP(BPF_JMP64+BPF_JEQ+BPF_K, STDOUT_FILENO, 1, 0),
^
samples/seccomp/bpf-direct.c:139:12: error: use of undeclared identifier 'BPF_JMP64'
BPF_JUMP(BPF_JMP64+BPF_JEQ+BPF_K, STDERR_FILENO, 1, 2),
^
>> samples/seccomp/bpf-direct.c:146:33: error: invalid application of 'sizeof' to an incomplete type 'struct sock_filter[]'
.len = (unsigned short)(sizeof(filter)/sizeof(filter[0])),
^~~~~~~~
10 errors generated.
--
>> samples/seccomp/user-trap.c:91:12: error: use of undeclared identifier 'BPF_JMP64'
BPF_JUMP(BPF_JMP64+BPF_JEQ+BPF_K, nr, 0, 1),
^
>> samples/seccomp/user-trap.c:97:26: error: invalid application of 'sizeof' to an incomplete type 'struct sock_filter[]'
.len = (unsigned short)ARRAY_SIZE(filter),
^~~~~~~~~~~~~~~~~~
samples/seccomp/user-trap.c:24:30: note: expanded from macro 'ARRAY_SIZE'
#define ARRAY_SIZE(x) (sizeof(x) / sizeof(*(x)))
^~~
2 errors generated.
vim +/BPF_JMP64 +38 samples/seccomp/bpf-fancy.c
8ac270d1e29f042 Will Drewry 2012-04-12 26
8ac270d1e29f042 Will Drewry 2012-04-12 27 int main(int argc, char **argv)
8ac270d1e29f042 Will Drewry 2012-04-12 28 {
3a9af0bd34410a2 Kees Cook 2015-02-17 29 struct bpf_labels l = {
3a9af0bd34410a2 Kees Cook 2015-02-17 30 .count = 0,
3a9af0bd34410a2 Kees Cook 2015-02-17 31 };
8ac270d1e29f042 Will Drewry 2012-04-12 32 static const char msg1[] = "Please type something: ";
8ac270d1e29f042 Will Drewry 2012-04-12 33 static const char msg2[] = "You typed: ";
8ac270d1e29f042 Will Drewry 2012-04-12 34 char buf[256];
8ac270d1e29f042 Will Drewry 2012-04-12 35 struct sock_filter filter[] = {
8ac270d1e29f042 Will Drewry 2012-04-12 36 /* TODO: LOAD_SYSCALL_NR(arch) and enforce an arch */
8ac270d1e29f042 Will Drewry 2012-04-12 37 LOAD_SYSCALL_NR,
8ac270d1e29f042 Will Drewry 2012-04-12 @38 SYSCALL(__NR_exit, ALLOW),
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests
next prev parent reply other threads:[~2023-02-02 4:02 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-02-01 12:36 [PATCH bpf-next 0/4] bpf: Replace BPF_ALU and BPF_JMP with BPF_ALU32 and BPF_JMP64 Tiezhu Yang
2023-02-01 12:36 ` [PATCH bpf-next 1/4] bpf: Add new macro " Tiezhu Yang
2023-02-01 14:59 ` Dave Thaler
2023-02-01 12:36 ` [PATCH bpf-next 2/4] bpf: treewide: Clean up BPF_ALU and BPF_JMP Tiezhu Yang
2023-02-02 4:01 ` kernel test robot [this message]
2023-02-01 12:36 ` [PATCH bpf-next 3/4] bpf: treewide: Clean up BPF_ALU_* and BPF_JMP_* Tiezhu Yang
2023-02-01 12:36 ` [PATCH bpf-next 4/4] bpf: Mark BPF_ALU and BPF_JMP as deprecated Tiezhu Yang
2023-02-02 11:26 ` [PATCH bpf-next 0/4] bpf: Replace BPF_ALU and BPF_JMP with BPF_ALU32 and BPF_JMP64 Alexei Starovoitov
2023-02-02 12:55 ` Tiezhu Yang
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=202302021105.HU4McFMJ-lkp@intel.com \
--to=lkp@intel.com \
--cc=andrii@kernel.org \
--cc=ast@kernel.org \
--cc=bpf@vger.kernel.org \
--cc=daniel@iogearbox.net \
--cc=linux-kernel@vger.kernel.org \
--cc=llvm@lists.linux.dev \
--cc=oe-kbuild-all@lists.linux.dev \
--cc=yangtiezhu@loongson.cn \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®