From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pf1-x42b.google.com (mail-pf1-x42b.google.com [IPv6:2607:f8b0:4864:20::42b]) by mail.toke.dk (Postfix) with ESMTPS id 8BB629D3D39 for ; Fri, 23 Dec 2022 05:07:22 +0100 (CET) Authentication-Results: mail.toke.dk; dkim=pass (2048-bit key; unprotected) header.d=google.com header.i=@google.com header.a=rsa-sha256 header.s=20210112 header.b=aiUbit8n Received: by mail-pf1-x42b.google.com with SMTP id a14so2565486pfa.1 for ; Thu, 22 Dec 2022 20:07:22 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20210112; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:from:to:cc:subject:date:message-id:reply-to; bh=uuRoS5f+F8qdvavLV7QeHdX9SfeSfZjDpTq44MsniBI=; b=aiUbit8nI0Qh4akK1B2FcpZRewpESB9clKE6W9Ij7/1o+mfp2SHvkXg77KT1mSyuyR C3d0IUJBgzATy99SRRcCEA4+DjOx7b3yzdg29oMjrzXrp+d0akP9AX3JaGw64u60ACzf Ps1PT5MW7U0LG3VlHVEn/6ivGo144GHPab2/g5qLFTBN8HHn+W65o4WkIWqSWZoCKQFA VzN9jWczmvBIZjaPBNFRzHytLmNrG5qKIZvcYSPhFHt89r0QNRmFAiQHlewyeUlwp+zg I8NvByhaiA1xUwxmFUs1oY7seKIMEGfGiccVn/Oh8HhBbT1MH8j1Miqu2Mu//5Zq48YM foQg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=uuRoS5f+F8qdvavLV7QeHdX9SfeSfZjDpTq44MsniBI=; b=u3I4eKvHneXB2nNBetQa9dkwo/v26xQgPFaGokpfAJMPkOgH7X8SOi938TwrEclonS 648UUSGO93RO7JD3jjYBhqrGCSYtneJtOk3wCV65Bdv/CErLwf8gLzWIYfAVWFD18gup 7tI0dAQkBgLOtOduY+UUsX2wBJUTWACWT46BBPkp7msUdEbZPgGgz65Eq8tzWFo7MUVE xNOx1JQlkB8hyWW4CofSLw05aGHfvWiqACb2EVZ0RXMtgnmHxavrbyrcqmCyT8h4DF4o 2DXFRnlFtmyLTliWOMe9Xxxuv2LmGg69KqTobvOrqNQicUbierZvKzNDC7CSCnxAkYJp 64Yg== X-Gm-Message-State: AFqh2krKwfIrS62CI+kIuY3IeAyTMVIZ1HjwaVH85nImNbqNSZ/zcAiS lKb9Xvp4+HZDHqU7qYhgGHs0TJkA7gUj3zuWCwAYcQ== X-Google-Smtp-Source: AMrXdXtHF3+xazC2Hrx5AjudWDyWXqDOqdnnxfEKkohBiAQlDSqR35HrJ5Gn7v/a+YtZ6xbiMno6mXfQshK/aEWCwjw= X-Received: by 2002:a63:1e11:0:b0:490:afd1:55b2 with SMTP id e17-20020a631e11000000b00490afd155b2mr411193pge.112.1671768440589; Thu, 22 Dec 2022 20:07:20 -0800 (PST) MIME-Version: 1.0 References: <20221220222043.3348718-1-sdf@google.com> <20221220222043.3348718-18-sdf@google.com> <95e79329-b7c9-550b-290e-e5e4ea6e7a01@linux.dev> In-Reply-To: <95e79329-b7c9-550b-290e-e5e4ea6e7a01@linux.dev> From: Stanislav Fomichev Date: Thu, 22 Dec 2022 20:07:08 -0800 Message-ID: To: Martin KaFai Lau Content-Type: text/plain; charset="UTF-8" Message-ID-Hash: OR22UBYLOWVJ3YZO7WIVLXUJ7Z6AB3CR X-Message-ID-Hash: OR22UBYLOWVJ3YZO7WIVLXUJ7Z6AB3CR X-MailFrom: sdf@google.com X-Mailman-Rule-Misses: dmarc-mitigation; no-senders; approved; emergency; loop; banned-address; member-moderation; nonmember-moderation; administrivia; implicit-dest; max-recipients; max-size; news-moderation; no-subject; digests; suspicious-header CC: ast@kernel.org, daniel@iogearbox.net, andrii@kernel.org, song@kernel.org, yhs@fb.com, john.fastabend@gmail.com, kpsingh@kernel.org, haoluo@google.com, jolsa@kernel.org, David Ahern , Jakub Kicinski , Willem de Bruijn , Jesper Dangaard Brouer , Anatoly Burakov , Alexander Lobakin , Magnus Karlsson , Maryam Tahhan , xdp-hints@xdp-project.net, netdev@vger.kernel.org, bpf@vger.kernel.org X-Mailman-Version: 3.3.7 Precedence: list Subject: [xdp-hints] Re: [PATCH bpf-next v5 17/17] selftests/bpf: Simple program to dump XDP RX metadata List-Id: XDP hardware hints design discussion Archived-At: List-Archive: List-Help: List-Owner: List-Post: List-Subscribe: List-Unsubscribe: On Thu, Dec 22, 2022 at 4:53 PM Martin KaFai Lau wrote: > > On 12/20/22 2:20 PM, Stanislav Fomichev wrote: > > To be used for verification of driver implementations. Note that > > the skb path is gone from the series, but I'm still keeping the > > implementation for any possible future work. > > > > $ xdp_hw_metadata > > > > On the other machine: > > > > $ echo -n xdp | nc -u -q1 9091 # for AF_XDP > > $ echo -n skb | nc -u -q1 9092 # for skb > > > > Sample output: > > > > # xdp > > xsk_ring_cons__peek: 1 > > 0x19f9090: rx_desc[0]->addr=100000000008000 addr=8100 comp_addr=8000 > > rx_timestamp_supported: 1 > > rx_timestamp: 1667850075063948829 > > 0x19f9090: complete idx=8 addr=8000 > > > > # skb > > found skb hwtstamp = 1668314052.854274681 > > > > Decoding: > > # xdp > > rx_timestamp=1667850075.063948829 > > > > $ date -d @1667850075 > > Mon Nov 7 11:41:15 AM PST 2022 > > $ date > > Mon Nov 7 11:42:05 AM PST 2022 > > > > # skb > > $ date -d @1668314052 > > Sat Nov 12 08:34:12 PM PST 2022 > > $ date > > Sat Nov 12 08:37:06 PM PST 2022 > > > > Cc: John Fastabend > > Cc: David Ahern > > Cc: Martin KaFai Lau > > Cc: Jakub Kicinski > > Cc: Willem de Bruijn > > Cc: Jesper Dangaard Brouer > > Cc: Anatoly Burakov > > Cc: Alexander Lobakin > > Cc: Magnus Karlsson > > Cc: Maryam Tahhan > > Cc: xdp-hints@xdp-project.net > > Cc: netdev@vger.kernel.org > > Signed-off-by: Stanislav Fomichev > > --- > > tools/testing/selftests/bpf/.gitignore | 1 + > > tools/testing/selftests/bpf/Makefile | 6 +- > > .../selftests/bpf/progs/xdp_hw_metadata.c | 81 ++++ > > tools/testing/selftests/bpf/xdp_hw_metadata.c | 405 ++++++++++++++++++ > > 4 files changed, 492 insertions(+), 1 deletion(-) > > create mode 100644 tools/testing/selftests/bpf/progs/xdp_hw_metadata.c > > create mode 100644 tools/testing/selftests/bpf/xdp_hw_metadata.c > > > > diff --git a/tools/testing/selftests/bpf/.gitignore b/tools/testing/selftests/bpf/.gitignore > > index 07d2d0a8c5cb..01e3baeefd4f 100644 > > --- a/tools/testing/selftests/bpf/.gitignore > > +++ b/tools/testing/selftests/bpf/.gitignore > > @@ -46,3 +46,4 @@ test_cpp > > xskxceiver > > xdp_redirect_multi > > xdp_synproxy > > +xdp_hw_metadata > > diff --git a/tools/testing/selftests/bpf/Makefile b/tools/testing/selftests/bpf/Makefile > > index e6cbc04a7920..b7d5d3aa554e 100644 > > --- a/tools/testing/selftests/bpf/Makefile > > +++ b/tools/testing/selftests/bpf/Makefile > > @@ -83,7 +83,7 @@ TEST_PROGS_EXTENDED := with_addr.sh \ > > TEST_GEN_PROGS_EXTENDED = test_sock_addr test_skb_cgroup_id_user \ > > flow_dissector_load test_flow_dissector test_tcp_check_syncookie_user \ > > test_lirc_mode2_user xdping test_cpp runqslower bench bpf_testmod.ko \ > > - xskxceiver xdp_redirect_multi xdp_synproxy veristat > > + xskxceiver xdp_redirect_multi xdp_synproxy veristat xdp_hw_metadata > > > > TEST_CUSTOM_PROGS = $(OUTPUT)/urandom_read $(OUTPUT)/sign-file > > TEST_GEN_FILES += liburandom_read.so > > @@ -241,6 +241,9 @@ $(OUTPUT)/test_maps: $(TESTING_HELPERS) > > $(OUTPUT)/test_verifier: $(TESTING_HELPERS) $(CAP_HELPERS) > > $(OUTPUT)/xsk.o: $(BPFOBJ) > > $(OUTPUT)/xskxceiver: $(OUTPUT)/xsk.o > > +$(OUTPUT)/xdp_hw_metadata: $(OUTPUT)/xsk.o $(OUTPUT)/xdp_hw_metadata.skel.h > > +$(OUTPUT)/xdp_hw_metadata: $(OUTPUT)/network_helpers.o > > +$(OUTPUT)/xdp_hw_metadata: LDFLAGS += -static > > > This test binary fails to build for llvm. gcc looks fine though. The CI tests > cannot be run on this set because of this. Please take a look: > > https://github.com/kernel-patches/bpf/actions/runs/3745257032/jobs/6359527599#step:11:2202 Ugh, again, I was hoping I fixed it :-( clang: error: cannot specify -o when generating multiple output files Will give it another try; for some reason can't reproduce locally (LLVM version 16.0.0git) > I only have minor comments on the set. Looking forward to v6. Thank you for another round of reviews! I'll probably keep quiet next week to give us some family time during holidays and will publish v6 early January. Happy holidays!