From: Ruslan Valiyev <linuxoid@gmail.com>
To: bpf@vger.kernel.org, ast@kernel.org, daniel@iogearbox.net,
andrii@kernel.org, eddyz87@gmail.com, martin.lau@linux.dev,
memxor@gmail.com
Cc: song@kernel.org, yonghong.song@linux.dev, jolsa@kernel.org,
emil@etsalapatis.com
Subject: [PATCH v2 bpf-next] selftests/bpf: add arena split and fork tests
Date: Fri, 5 Jun 2026 13:57:13 +0200 [thread overview]
Message-ID: <20260605115713.3224051-1-linuxoid@gmail.com> (raw)
In-Reply-To: <DJ0F0WFEP78A.1VV9Z7T71JJX5@etsalapatis.com>
Add two tests for an arena map's mmap()ed region: arena_split checks
that a partial munmap() that would split the mapping fails with -EINVAL,
and arena_fork checks that a forked child does not inherit the mapping
(mincore() reports the range as unmapped).
Signed-off-by: Ruslan Valiyev <linuxoid@gmail.com>
---
v2:
- Drop the regression-test framing and per-callback details; use
short one-line comments.
- Split into two separate tests, arena_split (partial unmap) and
arena_fork.
- Unpack the mincore() child check into explicit statements.
.../selftests/bpf/prog_tests/arena_fork.c | 53 +++++++++++++++++++
.../selftests/bpf/prog_tests/arena_split.c | 34 ++++++++++++
2 files changed, 87 insertions(+)
create mode 100644 tools/testing/selftests/bpf/prog_tests/arena_fork.c
create mode 100644 tools/testing/selftests/bpf/prog_tests/arena_split.c
diff --git a/tools/testing/selftests/bpf/prog_tests/arena_fork.c b/tools/testing/selftests/bpf/prog_tests/arena_fork.c
new file mode 100644
index 0000000000000..bb134e9adb607
--- /dev/null
+++ b/tools/testing/selftests/bpf/prog_tests/arena_fork.c
@@ -0,0 +1,53 @@
+// SPDX-License-Identifier: GPL-2.0
+#include <test_progs.h>
+#include <errno.h>
+#include <sys/mman.h>
+#include <sys/wait.h>
+
+/* Make sure an arena mapping is not inherited across fork(). */
+
+#define NR_PAGES 3
+
+void test_arena_fork(void)
+{
+ LIBBPF_OPTS(bpf_map_create_opts, opts, .map_flags = BPF_F_MMAPABLE);
+ long ps = sysconf(_SC_PAGESIZE);
+ size_t sz = (size_t)NR_PAGES * ps;
+ void *area;
+ int fd, ret, status;
+ pid_t pid;
+
+ fd = bpf_map_create(BPF_MAP_TYPE_ARENA, "arena_fork", 0, 0, NR_PAGES, &opts);
+ if (!ASSERT_OK_FD(fd, "arena map create"))
+ return;
+
+ area = mmap(NULL, sz, PROT_READ | PROT_WRITE, MAP_SHARED, fd, 0);
+ if (!ASSERT_NEQ(area, MAP_FAILED, "mmap arena"))
+ goto close_fd;
+
+ pid = fork();
+ if (pid == 0) {
+ unsigned char vec;
+ int rc;
+
+ /* If the mapping was not inherited the range is unmapped in
+ * the child, so mincore() fails with ENOMEM. A success means
+ * the child wrongly inherited the mapping.
+ */
+ rc = mincore(area, ps, &vec);
+ if (rc == 0)
+ _exit(1);
+ _exit(errno == ENOMEM ? 0 : 2);
+ }
+ if (ASSERT_GE(pid, 0, "fork")) {
+ while ((ret = waitpid(pid, &status, 0)) < 0 && errno == EINTR)
+ ;
+ if (ASSERT_EQ(ret, pid, "waitpid"))
+ ASSERT_TRUE(WIFEXITED(status) && WEXITSTATUS(status) == 0,
+ "child must not inherit arena mapping");
+ }
+
+ munmap(area, sz);
+close_fd:
+ close(fd);
+}
diff --git a/tools/testing/selftests/bpf/prog_tests/arena_split.c b/tools/testing/selftests/bpf/prog_tests/arena_split.c
new file mode 100644
index 0000000000000..8a59cfdb2ef89
--- /dev/null
+++ b/tools/testing/selftests/bpf/prog_tests/arena_split.c
@@ -0,0 +1,34 @@
+// SPDX-License-Identifier: GPL-2.0
+#include <test_progs.h>
+#include <errno.h>
+#include <sys/mman.h>
+
+/* Make sure an arena mapping cannot be split by a partial munmap(). */
+
+#define NR_PAGES 3
+
+void test_arena_split(void)
+{
+ LIBBPF_OPTS(bpf_map_create_opts, opts, .map_flags = BPF_F_MMAPABLE);
+ long ps = sysconf(_SC_PAGESIZE);
+ size_t sz = (size_t)NR_PAGES * ps;
+ void *area;
+ int fd, ret, err;
+
+ fd = bpf_map_create(BPF_MAP_TYPE_ARENA, "arena_split", 0, 0, NR_PAGES, &opts);
+ if (!ASSERT_OK_FD(fd, "arena map create"))
+ return;
+
+ area = mmap(NULL, sz, PROT_READ | PROT_WRITE, MAP_SHARED, fd, 0);
+ if (!ASSERT_NEQ(area, MAP_FAILED, "mmap arena"))
+ goto close_fd;
+
+ ret = munmap((char *)area + ps, ps);
+ err = errno;
+ if (ASSERT_ERR(ret, "split munmap"))
+ ASSERT_EQ(err, EINVAL, "split munmap errno");
+
+ munmap(area, sz);
+close_fd:
+ close(fd);
+}
base-commit: 174914ea551314c52a61713b9c4bde9e42d48073
--
2.43.0
next prev parent reply other threads:[~2026-06-05 11:57 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-06-03 9:44 [PATCH bpf-next] selftests/bpf: add test for arena vma split and fork rejection Ruslan Valiyev
2026-06-03 9:52 ` sashiko-bot
2026-06-04 16:45 ` Emil Tsalapatis
2026-06-05 11:57 ` Ruslan Valiyev [this message]
2026-06-05 12:07 ` [PATCH v2 bpf-next] selftests/bpf: add arena split and fork tests sashiko-bot
2026-06-05 12:22 ` bot+bpf-ci
2026-06-05 14:31 ` Mykyta Yatsenko
2026-06-08 13:25 ` Ruslan Valiyev
2026-06-09 8:37 ` [PATCH v3 bpf-next] selftests/bpf: add tests for arena vma split and fork Ruslan Valiyev
2026-06-09 18:40 ` Emil Tsalapatis
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=20260605115713.3224051-1-linuxoid@gmail.com \
--to=linuxoid@gmail.com \
--cc=andrii@kernel.org \
--cc=ast@kernel.org \
--cc=bpf@vger.kernel.org \
--cc=daniel@iogearbox.net \
--cc=eddyz87@gmail.com \
--cc=emil@etsalapatis.com \
--cc=jolsa@kernel.org \
--cc=martin.lau@linux.dev \
--cc=memxor@gmail.com \
--cc=song@kernel.org \
--cc=yonghong.song@linux.dev \
/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