Re: [PATCH bpf-next v4 3/6] selftests/bpf: Use post_socket_cb in connect_to_fd_opts

[Martin KaFai Lau]

On 5/23/24 8:21 PM, Geliang Tang wrote:
+static int cc_cb(int fd, void *opts)
+{
+       struct cb_opts *cb_opts = (struct cb_opts *)opts;
+
+       return setsockopt(fd, SOL_TCP, TCP_CONGESTION, cb_opts->cc,
+                         strlen(cb_opts->cc) + 1);

Since this cc_cb is in the test itself, it should use ASSERT_* such that the 
error output will be more useful. The same (use ASSERT_* whenever makes sense) 
goes for all cb functions in this set. In particular for the cb that does more 
than setsockopt in patch 5.