[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Thu, 10 Oct 2019 12:57:57 -0700
From: Stanislav Fomichev <sdf@...ichev.me>
To: Jakub Sitnicki <jakub@...udflare.com>
Cc: bpf@...r.kernel.org, netdev@...r.kernel.org,
kernel-team@...udflare.com
Subject: Re: [PATCH bpf-next v2 0/2] Atomic flow dissector updates
On 10/10, Jakub Sitnicki wrote:
> This patch set changes how bpf(BPF_PROG_ATTACH) operates on flow dissector
> hook when there is already a program attached. After this change the user
> is allowed to update the program in a single syscall. Please see the first
> patch for rationale.
>
> v1 -> v2:
>
> - Don't use CHECK macro which expects BPF program run duration, which we
> don't track in attach/detach tests. Suggested by Stanislav Fomichev.
>
> - Test re-attaching flow dissector in both root and non-root network
> namespace. Suggested by Stanislav Fomichev.
For the series:
Reviewed-by: Stanislav Fomichev <sdf@...gle.com>
Thanks!
> Jakub Sitnicki (2):
> flow_dissector: Allow updating the flow dissector program atomically
> selftests/bpf: Check that flow dissector can be re-attached
>
> net/core/flow_dissector.c | 10 +-
> .../bpf/prog_tests/flow_dissector_reattach.c | 127 ++++++++++++++++++
> 2 files changed, 134 insertions(+), 3 deletions(-)
> create mode 100644 tools/testing/selftests/bpf/prog_tests/flow_dissector_reattach.c
>
> --
> 2.20.1
>
Powered by blists - more mailing lists