Merge branch 'unsupported-map-lookup'
authorAlexei Starovoitov <ast@kernel.org>
Wed, 10 Oct 2018 04:52:21 +0000 (21:52 -0700)
committerAlexei Starovoitov <ast@kernel.org>
Wed, 10 Oct 2018 04:52:21 +0000 (21:52 -0700)
commite0aa5cf5919ee272732fce9576222e58e48622ab
tree498d230771d811b1153e5a9eae801ecc7a675dc8
parent8af03d1ae2e154a8be3631e8694b87007e1bdbc2
parent7c85c448e7d74c4ddd759440a2141eab663567cf
Merge branch 'unsupported-map-lookup'

Prashant Bhole says:

====================
Currently when map a lookup fails, user space API can not make any
distinction whether given key was not found or lookup is not supported
by particular map.

In this series we modify return value of maps which do not support
lookup. Lookup on such map implementation will return -EOPNOTSUPP.
bpf() syscall with BPF_MAP_LOOKUP_ELEM command will set EOPNOTSUPP
errno. We also handle this error in bpftool to print appropriate
message.

Patch 1: adds handling of BPF_MAP_LOOKUP ELEM command of bpf syscall
such that errno will set to EOPNOTSUPP when map doesn't support lookup

Patch 2: Modifies the return value of map_lookup_elem() to EOPNOTSUPP
for maps which do not support lookup

Patch 3: Splits do_dump() in bpftool/map.c. Element printing code is
moved out into new function dump_map_elem(). This was done in order to
reduce deep indentation and accomodate further changes.

Patch 4: Changes in bpftool to print strerror() message when lookup
error is occured. This will result in appropriate message like
"Operation not supported" when map doesn't support lookup.

Patch 5: test_verifier: change fixup map naming convention as
suggested by Alexei

Patch 6: Added verifier tests to check whether verifier rejects call
to bpf_map_lookup_elem from bpf program. For all map types those
do not support map lookup.
====================

Signed-off-by: Alexei Starovoitov <ast@kernel.org>