bpf: Convert PTR_TO_MEM_OR_NULL to composable types.

Remove PTR_TO_MEM_OR_NULL and replace it with PTR_TO_MEM combined with
flag PTR_MAYBE_NULL.

Signed-off-by: Hao Luo <haoluo@google.com>
Signed-off-by: Alexei Starovoitov <ast@kernel.org>
Link: https://lore.kernel.org/bpf/20211217003152.48334-7-haoluo@google.com

diff --git a/include/linux/bpf.h b/include/linux/bpf.h
index 126048110bdbdecce2bcd1fe469fbd3d1c90573f..567d83bf28f90bfa997ce65fc89818e961b0f5f8 100644 (file)
--- a/include/linux/bpf.h
+++ b/include/linux/bpf.h
@@ -506,7 +506,6 @@ enum bpf_reg_type {
        PTR_TO_SOCK_COMMON_OR_NULL      = PTR_MAYBE_NULL | PTR_TO_SOCK_COMMON,
        PTR_TO_TCP_SOCK_OR_NULL         = PTR_MAYBE_NULL | PTR_TO_TCP_SOCK,
        PTR_TO_BTF_ID_OR_NULL           = PTR_MAYBE_NULL | PTR_TO_BTF_ID,
-       PTR_TO_MEM_OR_NULL              = PTR_MAYBE_NULL | PTR_TO_MEM,
 
        /* This must be the last entry. Its purpose is to ensure the enum is
         * wide enough to hold the higher bits reserved for bpf_type_flag.

diff --git a/kernel/bpf/btf.c b/kernel/bpf/btf.c
index d1447b075c736ce9be90b11cb074227856d13967..d948b5be3bb84608bef854be7fa94543c97cbc77 100644 (file)
--- a/kernel/bpf/btf.c
+++ b/kernel/bpf/btf.c
@@ -5859,7 +5859,7 @@ int btf_prepare_func_args(struct bpf_verifier_env *env, int subprog,
                                return -EINVAL;
                        }
 
-                       reg->type = PTR_TO_MEM_OR_NULL;
+                       reg->type = PTR_TO_MEM | PTR_MAYBE_NULL;
                        reg->id = ++env->id_gen;
 
                        continue;

diff --git a/kernel/bpf/verifier.c b/kernel/bpf/verifier.c
index e0a8a55ea3df99a7ed37ddafee2a52ab5636fa31..9073337ac66fcdf07a4ea778ae0c77636896bd97 100644 (file)
--- a/kernel/bpf/verifier.c
+++ b/kernel/bpf/verifier.c
@@ -13578,7 +13578,7 @@ static int do_check_common(struct bpf_verifier_env *env, int subprog)
                                mark_reg_known_zero(env, regs, i);
                        else if (regs[i].type == SCALAR_VALUE)
                                mark_reg_unknown(env, regs, i);
-                       else if (regs[i].type == PTR_TO_MEM_OR_NULL) {
+                       else if (base_type(regs[i].type) == PTR_TO_MEM) {
                                const u32 mem_size = regs[i].mem_size;
 
                                mark_reg_known_zero(env, regs, i);