diff options
| author | Alexei Starovoitov <ast@plumgrid.com> | 2015-01-23 02:11:09 +0100 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2015-01-27 02:20:40 +0100 |
| commit | ba1a68bf1308a71a07d2462f4c38c242e08f92ba (patch) | |
| tree | 05f6de4541734b82240629d9a21cb3df1b83d7ef | |
| parent | bpf: rcu lock must not be held when calling copy_to_user() (diff) | |
| download | linux-ba1a68bf1308a71a07d2462f4c38c242e08f92ba.tar.xz linux-ba1a68bf1308a71a07d2462f4c38c242e08f92ba.zip | |
samples: bpf: relax test_maps check
hash map is unordered, so get_next_key() iterator shouldn't
rely on particular order of elements. So relax this test.
Fixes: ffb65f27a155 ("bpf: add a testsuite for eBPF maps")
Reported-by: Michael Holzheu <holzheu@linux.vnet.ibm.com>
Signed-off-by: Alexei Starovoitov <ast@plumgrid.com>
Acked-by: Daniel Borkmann <dborkman@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
| -rw-r--r-- | samples/bpf/test_maps.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/samples/bpf/test_maps.c b/samples/bpf/test_maps.c index e286b42307f3..6299ee95cd11 100644 --- a/samples/bpf/test_maps.c +++ b/samples/bpf/test_maps.c @@ -69,9 +69,9 @@ static void test_hashmap_sanity(int i, void *data) /* iterate over two elements */ assert(bpf_get_next_key(map_fd, &key, &next_key) == 0 && - next_key == 2); + (next_key == 1 || next_key == 2)); assert(bpf_get_next_key(map_fd, &next_key, &next_key) == 0 && - next_key == 1); + (next_key == 1 || next_key == 2)); assert(bpf_get_next_key(map_fd, &next_key, &next_key) == -1 && errno == ENOENT); |