The line in process_accept() checking sockfd is a leftover from times there was just one static socket. Now it's allocated, and we shouldn't be checking an uninitialised value, which fails the benchmark from time to time. Signed-off-by: Pavel Begunkov --- examples/zcrx.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/examples/zcrx.c b/examples/zcrx.c index 4b9f078e..b55b07ce 100644 --- a/examples/zcrx.c +++ b/examples/zcrx.c @@ -356,8 +356,6 @@ static void process_accept(struct io_uring *ring, struct io_uring_cqe *cqe) conn = aligned_alloc(64, sizeof(*conn)); if (!conn) t_error(1, 0, "can't allocate conn structure"); - if (conn->sockfd) - t_error(1, 0, "Unexpected second connection"); memset(conn, 0, sizeof(*conn)); conn->sockfd = cqe->res; -- 2.53.0