From: Ming Lei <tom.leiming@gmail.com>
To: Jens Axboe <axboe@kernel.dk>, io-uring@vger.kernel.org
Cc: Ming Lei <tom.leiming@gmail.com>
Subject: [PATCH 2/2] liburing test: add fixed-buf-send-recv for registered buffer send/recv
Date: Mon, 1 Jun 2026 04:58:46 -0500 [thread overview]
Message-ID: <20260601095853.3670199-3-ming.lei@redhat.com> (raw)
In-Reply-To: <20260601095853.3670199-1-ming.lei@redhat.com>
From: Ming Lei <tom.leiming@gmail.com>
Exercise IORING_RECVSEND_FIXED_BUF on plain IORING_OP_SEND and
IORING_OP_RECV: send-fixed, recv-fixed and both-fixed roundtrips with
non-zero offsets into distinct registered buffers, a large MSG_WAITALL
transfer to cover the persisted bvec iter across partial retries, plus
negative cases (sendmsg/bundle/recv-multishot rejected with -EINVAL and
a bad buf_index returning -EFAULT).
Signed-off-by: Ming Lei <tom.leiming@gmail.com>
---
test/Makefile | 1 +
test/fixed-buf-send-recv.c | 300 +++++++++++++++++++++++++++++++++++++
2 files changed, 301 insertions(+)
create mode 100644 test/fixed-buf-send-recv.c
diff --git a/test/Makefile b/test/Makefile
index d1cd2470..effb3bae 100644
--- a/test/Makefile
+++ b/test/Makefile
@@ -132,6 +132,7 @@ test_srcs := \
file-verify.c \
fixed-buf-iter.c \
fixed-buf-merge.c \
+ fixed-buf-send-recv.c \
fixed-hugepage.c \
fixed-link.c \
fixed-reuse.c \
diff --git a/test/fixed-buf-send-recv.c b/test/fixed-buf-send-recv.c
new file mode 100644
index 00000000..5e84935b
--- /dev/null
+++ b/test/fixed-buf-send-recv.c
@@ -0,0 +1,300 @@
+/* SPDX-License-Identifier: MIT */
+/*
+ * Test IORING_RECVSEND_FIXED_BUF on plain IORING_OP_SEND / IORING_OP_RECV.
+ *
+ * A registered (fixed) buffer can be used as the send source and/or the recv
+ * destination over a TCP socket via IORING_RECVSEND_FIXED_BUF. Covers:
+ * - send fixed -> recv normal
+ * - send normal -> recv fixed
+ * - send fixed -> recv fixed (both ends registered, non-zero offsets)
+ * - large MSG_WAITALL transfer (exercises the persisted bvec iter across
+ * partial send/recv retries)
+ * - negative cases: FIXED_BUF rejected on sendmsg, on bundle, on recv
+ * multishot, and a bad buf_index -> -EFAULT.
+ */
+#include <errno.h>
+#include <limits.h>
+#include <stdio.h>
+#include <stdlib.h>
+#include <string.h>
+#include <unistd.h>
+#include <sys/socket.h>
+
+#include "liburing.h"
+#include "helpers.h"
+
+#define BUF_SIZE (128 * 1024)
+#define OFF 4096
+
+/* registered buffer indices */
+#define SBUF_IDX 0
+#define RBUF_IDX 1
+
+static int no_fixed_buf;
+
+static void fill_pattern(unsigned char *buf, size_t len, unsigned seed)
+{
+ size_t i;
+
+ for (i = 0; i < len; i++)
+ buf[i] = (unsigned char)((i + seed) & 0xff);
+}
+
+/*
+ * Submit a paired send (user_data 1) + recv (user_data 2) and wait for both.
+ * Either side may use a registered buffer. Returns 0 on success with the
+ * received data verified against the sent pattern, -EINVAL if the kernel
+ * doesn't support the flag, or 1 on hard failure.
+ */
+static int do_roundtrip(struct io_uring *ring, int sfd, int rfd,
+ unsigned char *sptr, unsigned char *rptr, size_t len,
+ int s_fixed, int r_fixed, int waitall)
+{
+ struct io_uring_sqe *sqe;
+ struct io_uring_cqe *cqe;
+ int ret, i, sflags = 0, rflags = 0;
+ int s_res = INT_MIN, r_res = INT_MIN;
+ static unsigned seed;
+
+ seed++;
+ if (waitall) {
+ sflags |= MSG_WAITALL;
+ rflags |= MSG_WAITALL;
+ }
+
+ fill_pattern(sptr, len, seed);
+ memset(rptr, 0, len);
+
+ sqe = io_uring_get_sqe(ring);
+ io_uring_prep_send(sqe, sfd, sptr, len, sflags);
+ if (s_fixed) {
+ sqe->ioprio |= IORING_RECVSEND_FIXED_BUF;
+ sqe->buf_index = SBUF_IDX;
+ }
+ sqe->user_data = 1;
+
+ sqe = io_uring_get_sqe(ring);
+ io_uring_prep_recv(sqe, rfd, rptr, len, rflags);
+ if (r_fixed) {
+ sqe->ioprio |= IORING_RECVSEND_FIXED_BUF;
+ sqe->buf_index = RBUF_IDX;
+ }
+ sqe->user_data = 2;
+
+ ret = io_uring_submit_and_wait(ring, 2);
+ if (ret != 2) {
+ fprintf(stderr, "submit_and_wait: %d\n", ret);
+ return 1;
+ }
+
+ for (i = 0; i < 2; i++) {
+ ret = io_uring_peek_cqe(ring, &cqe);
+ if (ret) {
+ fprintf(stderr, "peek_cqe: %d\n", ret);
+ return 1;
+ }
+ if (cqe->user_data == 1)
+ s_res = cqe->res;
+ else
+ r_res = cqe->res;
+ io_uring_cqe_seen(ring, cqe);
+ }
+
+ if (s_res == -EINVAL || r_res == -EINVAL) {
+ no_fixed_buf = 1;
+ return -EINVAL;
+ }
+ if (s_res != (int)len) {
+ fprintf(stderr, "send res %d, want %zu (s_fixed=%d)\n",
+ s_res, len, s_fixed);
+ return 1;
+ }
+ if (r_res != (int)len) {
+ fprintf(stderr, "recv res %d, want %zu (r_fixed=%d)\n",
+ r_res, len, r_fixed);
+ return 1;
+ }
+ if (memcmp(sptr, rptr, len)) {
+ fprintf(stderr, "data mismatch (s_fixed=%d r_fixed=%d len=%zu)\n",
+ s_fixed, r_fixed, len);
+ return 1;
+ }
+ return 0;
+}
+
+/* Submit one sqe (already prepared by caller) and expect a specific res. */
+static int expect_res(struct io_uring *ring, int expect)
+{
+ struct io_uring_cqe *cqe;
+ int ret, res;
+
+ ret = io_uring_submit(ring);
+ if (ret != 1) {
+ fprintf(stderr, "submit: %d\n", ret);
+ return 1;
+ }
+ ret = io_uring_wait_cqe(ring, &cqe);
+ if (ret) {
+ fprintf(stderr, "wait_cqe: %d\n", ret);
+ return 1;
+ }
+ res = cqe->res;
+ io_uring_cqe_seen(ring, cqe);
+ if (res != expect) {
+ fprintf(stderr, "got res %d, expected %d\n", res, expect);
+ return 1;
+ }
+ return 0;
+}
+
+static int test_negative(struct io_uring *ring, int sfd)
+{
+ struct io_uring_sqe *sqe;
+ struct msghdr msg = { };
+ struct iovec iov;
+ static char nbuf[64];
+
+ /* sendmsg + FIXED_BUF is only allowed for plain send -> -EINVAL */
+ iov.iov_base = nbuf;
+ iov.iov_len = sizeof(nbuf);
+ msg.msg_iov = &iov;
+ msg.msg_iovlen = 1;
+ sqe = io_uring_get_sqe(ring);
+ io_uring_prep_sendmsg(sqe, sfd, &msg, 0);
+ sqe->ioprio |= IORING_RECVSEND_FIXED_BUF;
+ sqe->buf_index = SBUF_IDX;
+ sqe->user_data = 10;
+ if (expect_res(ring, -EINVAL)) {
+ fprintf(stderr, "sendmsg+fixed_buf not rejected\n");
+ return 1;
+ }
+
+ /* send + bundle + FIXED_BUF -> -EINVAL */
+ sqe = io_uring_get_sqe(ring);
+ io_uring_prep_send(sqe, sfd, nbuf, sizeof(nbuf), 0);
+ sqe->ioprio |= IORING_RECVSEND_FIXED_BUF | IORING_RECVSEND_BUNDLE;
+ sqe->buf_index = SBUF_IDX;
+ sqe->user_data = 11;
+ if (expect_res(ring, -EINVAL)) {
+ fprintf(stderr, "send bundle+fixed_buf not rejected\n");
+ return 1;
+ }
+
+ /* recv multishot + FIXED_BUF -> -EINVAL */
+ sqe = io_uring_get_sqe(ring);
+ io_uring_prep_recv_multishot(sqe, sfd, nbuf, sizeof(nbuf), 0);
+ sqe->ioprio |= IORING_RECVSEND_FIXED_BUF;
+ sqe->buf_index = RBUF_IDX;
+ sqe->user_data = 12;
+ if (expect_res(ring, -EINVAL)) {
+ fprintf(stderr, "recv multishot+fixed_buf not rejected\n");
+ return 1;
+ }
+
+ /* send fixed with an unregistered buf_index -> -EFAULT at issue */
+ sqe = io_uring_get_sqe(ring);
+ io_uring_prep_send(sqe, sfd, nbuf, sizeof(nbuf), 0);
+ sqe->ioprio |= IORING_RECVSEND_FIXED_BUF;
+ sqe->buf_index = 42;
+ sqe->user_data = 13;
+ if (expect_res(ring, -EFAULT)) {
+ fprintf(stderr, "send fixed bad index not -EFAULT\n");
+ return 1;
+ }
+
+ return 0;
+}
+
+int main(int argc, char *argv[])
+{
+ struct io_uring ring;
+ struct iovec regvec[2];
+ unsigned char *sbuf, *rbuf, *hbuf;
+ int ret, fds[2];
+
+ if (argc > 1)
+ return T_EXIT_SKIP;
+
+ if (posix_memalign((void **)&sbuf, 4096, BUF_SIZE) ||
+ posix_memalign((void **)&rbuf, 4096, BUF_SIZE)) {
+ fprintf(stderr, "posix_memalign failed\n");
+ return T_EXIT_FAIL;
+ }
+ hbuf = malloc(BUF_SIZE);
+ if (!hbuf)
+ return T_EXIT_FAIL;
+
+ ret = io_uring_queue_init(8, &ring, 0);
+ if (ret) {
+ fprintf(stderr, "queue_init: %d\n", ret);
+ return T_EXIT_FAIL;
+ }
+
+ regvec[SBUF_IDX].iov_base = sbuf;
+ regvec[SBUF_IDX].iov_len = BUF_SIZE;
+ regvec[RBUF_IDX].iov_base = rbuf;
+ regvec[RBUF_IDX].iov_len = BUF_SIZE;
+ ret = io_uring_register_buffers(&ring, regvec, 2);
+ if (ret) {
+ fprintf(stderr, "register_buffers: %d\n", ret);
+ return T_EXIT_FAIL;
+ }
+
+ ret = t_create_socket_pair(fds, true);
+ if (ret) {
+ fprintf(stderr, "socket pair: %d\n", ret);
+ return T_EXIT_FAIL;
+ }
+
+ /* send fixed -> recv normal (also doubles as feature detection) */
+ ret = do_roundtrip(&ring, fds[1], fds[0], sbuf + OFF, hbuf, 4096,
+ 1, 0, 0);
+ if (ret == -EINVAL) {
+ fprintf(stderr, "IORING_RECVSEND_FIXED_BUF send unsupported, skip\n");
+ return T_EXIT_SKIP;
+ }
+ if (ret)
+ goto fail;
+
+ /* send normal -> recv fixed */
+ ret = do_roundtrip(&ring, fds[1], fds[0], hbuf, rbuf + OFF, 4096,
+ 0, 1, 0);
+ if (ret == -EINVAL) {
+ fprintf(stderr, "IORING_RECVSEND_FIXED_BUF recv unsupported, skip\n");
+ return T_EXIT_SKIP;
+ }
+ if (ret)
+ goto fail;
+
+ /* send fixed -> recv fixed, non-zero offsets on both ends */
+ ret = do_roundtrip(&ring, fds[1], fds[0], sbuf + OFF, rbuf + 2 * OFF,
+ 8192, 1, 1, 0);
+ if (ret)
+ goto fail;
+
+ /* large transfer with MSG_WAITALL: persisted bvec iter across retries */
+ ret = do_roundtrip(&ring, fds[1], fds[0], sbuf, rbuf, BUF_SIZE,
+ 1, 1, 1);
+ if (ret)
+ goto fail;
+
+ /* and the other direction */
+ ret = do_roundtrip(&ring, fds[0], fds[1], sbuf, rbuf, BUF_SIZE,
+ 1, 1, 1);
+ if (ret)
+ goto fail;
+
+ if (test_negative(&ring, fds[1]))
+ goto fail;
+
+ io_uring_queue_exit(&ring);
+ close(fds[0]);
+ close(fds[1]);
+ free(hbuf);
+ free(sbuf);
+ free(rbuf);
+ return T_EXIT_PASS;
+fail:
+ fprintf(stderr, "test failed\n");
+ return T_EXIT_FAIL;
+}
--
2.54.0
next prev parent reply other threads:[~2026-06-01 9:59 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-06-01 9:58 [PATCH 0/2] io_uring/net: support registered buffer for plain send and recv Ming Lei
2026-06-01 9:58 ` [PATCH 1/2] " Ming Lei
2026-06-08 3:08 ` Jens Axboe
2026-06-01 9:58 ` Ming Lei [this message]
2026-06-07 23:30 ` [PATCH 0/2] " Ming Lei
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20260601095853.3670199-3-ming.lei@redhat.com \
--to=tom.leiming@gmail.com \
--cc=axboe@kernel.dk \
--cc=io-uring@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox