]> git.ipfire.org Git - thirdparty/kernel/stable.git/commitdiff
tools/bpf: selftests: add map lookup to test_map_in_map bpf prog
authorYonghong Song <yhs@fb.com>
Wed, 27 Feb 2019 21:22:57 +0000 (13:22 -0800)
committerAlexei Starovoitov <ast@kernel.org>
Thu, 28 Feb 2019 01:03:13 +0000 (17:03 -0800)
The bpf_map_lookup_elem is added in the bpf program.
Without previous patch, the test change will trigger the
following error:
  $ ./test_maps
  ...
  ; value_p = bpf_map_lookup_elem(map, &key);
  20: (bf) r1 = r7
  21: (bf) r2 = r8
  22: (85) call bpf_map_lookup_elem#1
  ; if (!value_p || *value_p != 123)
  23: (15) if r0 == 0x0 goto pc+16
   R0=map_value(id=2,off=0,ks=4,vs=4,imm=0) R6=inv1 R7=map_ptr(id=0,off=0,ks=4,vs=4,imm=0)
   R8=fp-8,call_-1 R10=fp0,call_-1 fp-8=mmmmmmmm
  ; if (!value_p || *value_p != 123)
  24: (61) r1 = *(u32 *)(r0 +0)
   R0=map_value(id=2,off=0,ks=4,vs=4,imm=0) R6=inv1 R7=map_ptr(id=0,off=0,ks=4,vs=4,imm=0)
   R8=fp-8,call_-1 R10=fp0,call_-1 fp-8=mmmmmmmm
  bpf_spin_lock cannot be accessed directly by load/store

With the kernel fix in the previous commit, the error goes away.

Signed-off-by: Yonghong Song <yhs@fb.com>
Acked-by: Andrii Nakryiko <andriin@fb.com>
Signed-off-by: Alexei Starovoitov <ast@kernel.org>
tools/testing/selftests/bpf/progs/test_map_in_map.c

index ce923e67e08e167e9ebb2642b5c57e5616e0799e..2985f262846e0b77bd90e083f535e95f4e880637 100644 (file)
@@ -27,6 +27,7 @@ SEC("xdp_mimtest")
 int xdp_mimtest0(struct xdp_md *ctx)
 {
        int value = 123;
+       int *value_p;
        int key = 0;
        void *map;
 
@@ -35,6 +36,9 @@ int xdp_mimtest0(struct xdp_md *ctx)
                return XDP_DROP;
 
        bpf_map_update_elem(map, &key, &value, 0);
+       value_p = bpf_map_lookup_elem(map, &key);
+       if (!value_p || *value_p != 123)
+               return XDP_DROP;
 
        map = bpf_map_lookup_elem(&mim_hash, &key);
        if (!map)