[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-Id: <20220130115518.213259-1-jakub@cloudflare.com>
Date: Sun, 30 Jan 2022 12:55:16 +0100
From: Jakub Sitnicki <jakub@...udflare.com>
To: bpf@...r.kernel.org
Cc: netdev@...r.kernel.org, Alexei Starovoitov <ast@...nel.org>,
Daniel Borkmann <daniel@...earbox.net>,
Andrii Nakryiko <andrii@...nel.org>,
Menglong Dong <imagedong@...cent.com>,
Martin KaFai Lau <kafai@...com>
Subject: [PATCH bpf-next v2 0/2] Split bpf_sock dst_port field
This is a follow-up to discussion around the idea of making dst_port in struct
bpf_sock a 16-bit field that happened in [1].
v2:
- use an anonymous field for zero padding (Alexei)
v1:
- keep dst_field offset unchanged to prevent existing BPF program breakage
(Martin)
- allow 8-bit loads from dst_port[0] and [1]
- add test coverage for the verifier and the context access converter
[1] https://lore.kernel.org/bpf/87sftbobys.fsf@cloudflare.com/
Jakub Sitnicki (2):
bpf: Make dst_port field in struct bpf_sock 16-bit wide
selftests/bpf: Extend verifier and bpf_sock tests for dst_port loads
include/uapi/linux/bpf.h | 3 +-
net/core/filter.c | 10 ++-
tools/include/uapi/linux/bpf.h | 3 +-
.../selftests/bpf/prog_tests/sock_fields.c | 58 +++++++++----
.../selftests/bpf/progs/test_sock_fields.c | 41 ++++++++++
tools/testing/selftests/bpf/verifier/sock.c | 81 ++++++++++++++++++-
6 files changed, 173 insertions(+), 23 deletions(-)
--
2.31.1
Powered by blists - more mailing lists