diff options
author | Edward Cree <ecree@solarflare.com> | 2017-08-23 15:10:50 +0100 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2017-08-23 22:38:08 -0700 |
commit | 1b688a19a92223cf2d1892c9d05d64dc397b33e3 (patch) | |
tree | 1e388805090774a172afdca917e657dad8b300ff /fs/freevxfs | |
parent | df20cb7ec17577c94ef93fa86c7c80958046a01e (diff) |
bpf/verifier: remove varlen_map_value_access flag
The optimisation it does is broken when the 'new' register value has a
variable offset and the 'old' was constant. I broke it with my pointer
types unification (see Fixes tag below), before which the 'new' value
would have type PTR_TO_MAP_VALUE_ADJ and would thus not compare equal;
other changes in that patch mean that its original behaviour (ignore
min/max values) cannot be restored.
Tests on a sample set of cilium programs show no change in count of
processed instructions.
Fixes: f1174f77b50c ("bpf/verifier: rework value tracking")
Signed-off-by: Edward Cree <ecree@solarflare.com>
Acked-by: Alexei Starovoitov <ast@kernel.org>
Acked-by: Daniel Borkmann <daniel@iogearbox.net>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'fs/freevxfs')
0 files changed, 0 insertions, 0 deletions