I tried to split it a bit, maybe I could even go further and split by TRACE_EVENT_CLASS() changes, but not sure if it adds any value. But at least all preparation patches are separate.
I wasn't sure if I should just remove tcp_hash_fail() as I did in this version, or rather put it under CONFIG_TCP_..., making it disabled by default and with a warning of deprecated, scheduled for removal. Maybe this won't cause any problems for anybody and I'm just too cautious of breaking others.
Anyways, version 1, thanks for any reviews!
Signed-off-by: Dmitry Safonov dima@arista.com --- Dmitry Safonov (10): net/tcp: Use static_branch_tcp_{md5,ao} to drop ifdefs net/tcp: Add a helper tcp_ao_hdr_maclen() net/tcp: Move tcp_inbound_hash() from headers net/tcp: Add tcp-md5 and tcp-ao tracepoints net/tcp: Remove tcp_hash_fail() selftests/net: Clean-up double assignment selftests/net: Provide test_snprintf() helper selftests/net: Be consistnat in kconfig checks selftests/net: Don't forget to close nsfd after switch_save_ns() selftest/net: Add trace events matching to tcp_ao
include/net/tcp.h | 79 +- include/net/tcp_ao.h | 42 +- include/trace/events/tcp.h | 317 ++++++++ net/ipv4/tcp.c | 86 ++- net/ipv4/tcp_ao.c | 24 +- net/ipv4/tcp_input.c | 8 +- net/ipv4/tcp_ipv4.c | 8 +- net/ipv4/tcp_output.c | 2 + tools/testing/selftests/net/tcp_ao/Makefile | 2 +- tools/testing/selftests/net/tcp_ao/bench-lookups.c | 2 +- tools/testing/selftests/net/tcp_ao/connect-deny.c | 18 +- tools/testing/selftests/net/tcp_ao/connect.c | 2 +- tools/testing/selftests/net/tcp_ao/icmps-discard.c | 2 +- .../testing/selftests/net/tcp_ao/key-management.c | 18 +- tools/testing/selftests/net/tcp_ao/lib/aolib.h | 150 +++- tools/testing/selftests/net/tcp_ao/lib/ftrace.c | 846 +++++++++++++++++++++ tools/testing/selftests/net/tcp_ao/lib/kconfig.c | 31 +- tools/testing/selftests/net/tcp_ao/lib/setup.c | 15 +- tools/testing/selftests/net/tcp_ao/lib/sock.c | 1 - tools/testing/selftests/net/tcp_ao/lib/utils.c | 26 + tools/testing/selftests/net/tcp_ao/restore.c | 18 +- tools/testing/selftests/net/tcp_ao/rst.c | 2 +- tools/testing/selftests/net/tcp_ao/self-connect.c | 19 +- tools/testing/selftests/net/tcp_ao/seq-ext.c | 10 +- .../selftests/net/tcp_ao/setsockopt-closed.c | 2 +- tools/testing/selftests/net/tcp_ao/unsigned-md5.c | 28 +- 26 files changed, 1576 insertions(+), 182 deletions(-) --- base-commit: d662c5b3ce6dbed9d0991bc83001bbcc4a9bc2f8 change-id: 20240224-tcp-ao-tracepoints-0ea8ba11467a
Best regards,