From: Jason Xing The client sends data, and the server reads it via read(), which triggers tcp_recvmsg() internally. And then the computation is done in the BPF_SOCK_OPS_TSTAMP_RCV_CB callback. Extend test_tcp() to cover the new BPF RX timestamping path. One more crucial thing is to add usleep to allow the delay of workqueue turning on global time record of each skb in recv path, or else the test might fail due to no timestamp in the skb. Verify that SO_TIMESTAMPING RX (software) and BPF RX timestamping can work simultaneously on the same socket without conflict. When enable_socket_timestamping is set, enable SOF_TIMESTAMPING_SOFTWARE | SOF_TIMESTAMPING_RX_SOFTWARE on afd and use recvmsg() to verify the SCM_TIMESTAMPING cmsg contains a valid software RX timestamp. This exercises the mixed mode where all 4 timestamping paths (SO TX on cfd, SO RX on afd, BPF TX on cfd, BPF RX on afd) run together. The standalone mode (enable_socket_timestamping=false) keeps only BPF TX and BPF RX active using read(). Signed-off-by: Jason Xing --- .../bpf/prog_tests/net_timestamping.c | 71 ++++++++++++++++++- .../selftests/bpf/progs/net_timestamping.c | 35 +++++++++ 2 files changed, 105 insertions(+), 1 deletion(-) diff --git a/tools/testing/selftests/bpf/prog_tests/net_timestamping.c b/tools/testing/selftests/bpf/prog_tests/net_timestamping.c index dbfd87499b6b..e3fe850e4cc7 100644 --- a/tools/testing/selftests/bpf/prog_tests/net_timestamping.c +++ b/tools/testing/selftests/bpf/prog_tests/net_timestamping.c @@ -143,11 +143,47 @@ static void test_socket_timestamping(int fd) SK_TS_ACK = 0; } +static bool recv_verify_rx_timestamp(int fd, char *buf, int len) +{ + struct scm_timestamping *tss = NULL; + char ctrl[1024]; + struct msghdr msg = {}; + struct iovec iov; + struct cmsghdr *cm; + int ret; + + iov.iov_base = buf; + iov.iov_len = len; + msg.msg_iov = &iov; + msg.msg_iovlen = 1; + msg.msg_control = ctrl; + msg.msg_controllen = sizeof(ctrl); + + ret = recvmsg(fd, &msg, 0); + if (!ASSERT_EQ(ret, len, "recvmsg from client")) + return false; + + for (cm = CMSG_FIRSTHDR(&msg); cm; cm = CMSG_NXTHDR(&msg, cm)) { + if (cm->cmsg_level == SOL_SOCKET && + cm->cmsg_type == SCM_TIMESTAMPING) { + tss = (void *)CMSG_DATA(cm); + break; + } + } + + if (!ASSERT_TRUE(tss != NULL, "SCM_TIMESTAMPING cmsg present") || + !ASSERT_TRUE(tss->ts[0].tv_sec || tss->ts[0].tv_nsec, "rx sw + timestamp non-zero")) + return false; + + return true; +} + static void test_tcp(int family, bool enable_socket_timestamping) { struct net_timestamping__bss *bss; char buf[cfg_payload_len]; - int sfd = -1, cfd = -1; + int sfd = -1, cfd = -1, afd = -1; unsigned int sock_opt; struct netns_obj *ns; int cg_fd; @@ -187,7 +223,27 @@ static void test_tcp(int family, bool enable_socket_timestamping) if (!ASSERT_OK_FD(cfd, "connect_to_fd_server")) goto out; + afd = accept(sfd, NULL, NULL); + if (!ASSERT_OK_FD(afd, "accept")) + goto out; + + /* net_enable_timestamp() defers the static key update via + * schedule_work() when CONFIG_JUMP_LABEL is set. Give the + * workqueue a chance to run so that netstamp_needed_key is + * active and skb->tstamp gets populated in the receive path. + */ + usleep(10000); + if (enable_socket_timestamping) { + unsigned int rx_opt; + + rx_opt = SOF_TIMESTAMPING_SOFTWARE | + SOF_TIMESTAMPING_RX_SOFTWARE; + ret = setsockopt(afd, SOL_SOCKET, SO_TIMESTAMPING, + (char *)&rx_opt, sizeof(rx_opt)); + if (!ASSERT_OK(ret, "setsockopt SO_TIMESTAMPING RX on afd")) + goto out; + sock_opt = SOF_TIMESTAMPING_SOFTWARE | SOF_TIMESTAMPING_OPT_ID | SOF_TIMESTAMPING_TX_SCHED | @@ -207,6 +263,15 @@ static void test_tcp(int family, bool enable_socket_timestamping) if (!ASSERT_EQ(ret, sizeof(buf), "send to server")) goto out; + if (enable_socket_timestamping) { + if (!recv_verify_rx_timestamp(afd, buf, sizeof(buf))) + goto out; + } else { + ret = read(afd, buf, sizeof(buf)); + if (!ASSERT_EQ(ret, sizeof(buf), "recv from client")) + goto out; + } + if (enable_socket_timestamping) test_socket_timestamping(cfd); @@ -215,8 +280,12 @@ static void test_tcp(int family, bool enable_socket_timestamping) ASSERT_EQ(bss->nr_sched, 1, "nr_sched"); ASSERT_EQ(bss->nr_txsw, 1, "nr_txsw"); ASSERT_EQ(bss->nr_ack, 1, "nr_ack"); + ASSERT_EQ(bss->nr_passive, 1, "nr_passive"); + ASSERT_GE(bss->nr_rcv, 1, "nr_rcv"); out: + if (afd >= 0) + close(afd); if (sfd >= 0) close(sfd); if (cfd >= 0) diff --git a/tools/testing/selftests/bpf/progs/net_timestamping.c b/tools/testing/selftests/bpf/progs/net_timestamping.c index b4c2f0f2be11..4bbff09db55c 100644 --- a/tools/testing/selftests/bpf/progs/net_timestamping.c +++ b/tools/testing/selftests/bpf/progs/net_timestamping.c @@ -6,6 +6,8 @@ #include "bpf_kfuncs.h" #include +extern u64 bpf_ktime_get_real_ns(void) __ksym; + __u32 monitored_pid = 0; int nr_active; @@ -14,6 +16,7 @@ int nr_passive; int nr_sched; int nr_txsw; int nr_ack; +int nr_rcv; struct sk_stg { __u64 sendmsg_ns; /* record ts when sendmsg is called */ @@ -65,6 +68,22 @@ static int bpf_test_sockopt(void *ctx, const struct sock *sk, int expected) return 0; } +static int bpf_test_rx_sockopt(void *ctx, const struct sock *sk, int expected) +{ + int tmp, new = SK_BPF_CB_RX_TIMESTAMPING; + int opt = SK_BPF_CB_FLAGS; + int level = SOL_SOCKET; + + if (bpf_setsockopt(ctx, level, opt, &new, sizeof(new)) != expected) + return 1; + + if (bpf_getsockopt(ctx, level, opt, &tmp, sizeof(tmp)) != expected || + (!expected && tmp != new)) + return 1; + + return 0; +} + static bool bpf_test_access_sockopt(void *ctx, const struct sock *sk) { if (bpf_test_sockopt(ctx, sk, -EOPNOTSUPP)) @@ -224,6 +243,9 @@ int skops_sockopt(struct bpf_sock_ops *skops) case BPF_SOCK_OPS_ACTIVE_ESTABLISHED_CB: nr_active += !bpf_test_sockopt(skops, sk, 0); break; + case BPF_SOCK_OPS_PASSIVE_ESTABLISHED_CB: + nr_passive += !bpf_test_rx_sockopt(skops, sk, 0); + break; case BPF_SOCK_OPS_TSTAMP_SENDMSG_CB: if (bpf_test_delay(skops, sk)) nr_snd += 1; @@ -240,6 +262,19 @@ int skops_sockopt(struct bpf_sock_ops *skops) if (bpf_test_delay(skops, sk)) nr_ack += 1; break; + case BPF_SOCK_OPS_TSTAMP_RCV_CB: { + u64 sw_tstamp, now, delay; + + sw_tstamp = (u64)skops->args[0] | ((u64)skops->args[1] << 32); + if (!sw_tstamp) + break; + + now = bpf_ktime_get_real_ns(); + delay = now - sw_tstamp; + if (delay < delay_tolerance_nsec) + nr_rcv += 1; + break; + } } return 1; -- 2.43.7