bpf, arm64: Remove redundant switch case about BPF_DIV and BPF_MOD

After commit 96a71005bdcb ("bpf, arm64: remove obsolete exception handling
from div/mod"), there is no need to check twice about BPF_DIV and BPF_MOD,
remove the redundant switch case.

Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn>
Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
Link: https://lore.kernel.org/bpf/1621328170-17583-1-git-send-email-yangtiezhu@loongson.cn

diff --git a/arch/arm64/net/bpf_jit_comp.c b/arch/arm64/net/bpf_jit_comp.c
index 978502629f4e645434648de6e7b0c17a90a75104..be873a7da62b5a7bd26759aa677108b98fa112f1 100644 (file)
--- a/arch/arm64/net/bpf_jit_comp.c
+++ b/arch/arm64/net/bpf_jit_comp.c
@@ -485,17 +485,12 @@ static int build_insn(const struct bpf_insn *insn, struct jit_ctx *ctx,
                break;
        case BPF_ALU | BPF_DIV | BPF_X:
        case BPF_ALU64 | BPF_DIV | BPF_X:
+               emit(A64_UDIV(is64, dst, dst, src), ctx);
+               break;
        case BPF_ALU | BPF_MOD | BPF_X:
        case BPF_ALU64 | BPF_MOD | BPF_X:
-               switch (BPF_OP(code)) {
-               case BPF_DIV:
-                       emit(A64_UDIV(is64, dst, dst, src), ctx);
-                       break;
-               case BPF_MOD:
-                       emit(A64_UDIV(is64, tmp, dst, src), ctx);
-                       emit(A64_MSUB(is64, dst, dst, tmp, src), ctx);
-                       break;
-               }
+               emit(A64_UDIV(is64, tmp, dst, src), ctx);
+               emit(A64_MSUB(is64, dst, dst, tmp, src), ctx);
                break;
        case BPF_ALU | BPF_LSH | BPF_X:
        case BPF_ALU64 | BPF_LSH | BPF_X: