test_bpf: Rename second ALU64_SMOD_X to ALU64_SMOD_K

seehearfeel · Kernel Patches Daemon · commit fdb6c3c77c43 · 2023-12-09T19:16:33.000-08:00
Currently, there are two test cases with same name "ALU64_SMOD_X: -7 % 2 = -1", the first one is right, the second one should be ALU64_SMOD_K because its code is BPF_ALU64 | BPF_MOD | BPF_K. Before: test_bpf: #170 ALU64_SMOD_X: -7 % 2 = -1 jited:1 4 PASS test_bpf: #171 ALU64_SMOD_X: -7 % 2 = -1 jited:1 4 PASS After: test_bpf: #170 ALU64_SMOD_X: -7 % 2 = -1 jited:1 4 PASS test_bpf: #171 ALU64_SMOD_K: -7 % 2 = -1 jited:1 4 PASS Fixes: daabb2b ("bpf/tests: add tests for cpuv4 instructions") Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn> Acked-by: Yonghong Song <yonghong.song@linux.dev>
diff --git a/lib/test_bpf.c b/lib/test_bpf.c
@@ -6277,7 +6277,7 @@ static struct bpf_test tests[] = {
 	},
 	/* BPF_ALU64 | BPF_MOD | BPF_K off=1 (SMOD64) */
 	{
-		"ALU64_SMOD_X: -7 % 2 = -1",
+		"ALU64_SMOD_K: -7 % 2 = -1",
 		.u.insns_int = {
 			BPF_LD_IMM64(R0, -7),
 			BPF_ALU64_IMM_OFF(BPF_MOD, R0, 2, 1),

Original file line number	Diff line number	Diff line change
`@@ -6277,7 +6277,7 @@ static struct bpf_test tests[] = {`
`6277`	`6277`	`},`
`6278`	`6278`	`/* BPF_ALU64 \| BPF_MOD \| BPF_K off=1 (SMOD64) */`
`6279`	`6279`	`{`
`6280`		`- "ALU64_SMOD_X: -7 % 2 = -1",`
	`6280`	`+ "ALU64_SMOD_K: -7 % 2 = -1",`
`6281`	`6281`	`.u.insns_int = {`
`6282`	`6282`	`BPF_LD_IMM64(R0, -7),`
`6283`	`6283`	`BPF_ALU64_IMM_OFF(BPF_MOD, R0, 2, 1),`