On Wed, Jun 05, 2019 at 08:57:56AM -0700, Jonathan Lemon wrote: > Currently, the AF_XDP code uses a separate map in order to > determine if an xsk is bound to a queue. Instead of doing this, > have bpf_map_lookup_elem() return a xdp_sock. > > Rearrange some xdp_sock members to eliminate structure holes. > > Signed-off-by: Jonathan Lemon <jonathan.le...@gmail.com> > --- > include/linux/bpf.h | 8 ++++ > include/net/xdp_sock.h | 4 +- > include/uapi/linux/bpf.h | 4 ++ > kernel/bpf/verifier.c | 26 +++++++++++- > kernel/bpf/xskmap.c | 7 ++++ > net/core/filter.c | 40 +++++++++++++++++++ > tools/include/uapi/linux/bpf.h | 4 ++P The convention is to submit this uapi's bpf.h sync to tools/ in a separate patch to make conflicts easier to be dealt with. It will be the 2nd patch.
> .../bpf/verifier/prevent_map_lookup.c | 15 ------- > tools/testing/selftests/bpf/verifier/sock.c | 18 +++++++++ Hence, the selftest changes will be in the 3rd patch. Others LGTM, Acked-by: Martin KaFai Lau <ka...@fb.com>
