[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PULL 13/31] tests/tcg: Add the PROT_NONE gdbstub test
|
From: |
Richard Henderson |
|
Subject: |
[PULL 13/31] tests/tcg: Add the PROT_NONE gdbstub test |
|
Date: |
Tue, 30 Jan 2024 09:01:03 +1000 |
From: Ilya Leoshkevich <iii@linux.ibm.com>
Make sure that qemu gdbstub, like gdbserver, allows reading from and
writing to PROT_NONE pages.
Signed-off-by: Ilya Leoshkevich <iii@linux.ibm.com>
Message-Id: <20240129093410.3151-4-iii@linux.ibm.com>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
---
tests/tcg/multiarch/prot-none.c | 40 ++++++++++++++++++++++++
tests/tcg/multiarch/Makefile.target | 9 +++++-
tests/tcg/multiarch/gdbstub/prot-none.py | 36 +++++++++++++++++++++
3 files changed, 84 insertions(+), 1 deletion(-)
create mode 100644 tests/tcg/multiarch/prot-none.c
create mode 100644 tests/tcg/multiarch/gdbstub/prot-none.py
diff --git a/tests/tcg/multiarch/prot-none.c b/tests/tcg/multiarch/prot-none.c
new file mode 100644
index 0000000000..dc56aadb3c
--- /dev/null
+++ b/tests/tcg/multiarch/prot-none.c
@@ -0,0 +1,40 @@
+/*
+ * Test that GDB can access PROT_NONE pages.
+ *
+ * SPDX-License-Identifier: GPL-2.0-or-later
+ */
+#include <assert.h>
+#include <stdlib.h>
+#include <string.h>
+#include <sys/mman.h>
+#include <unistd.h>
+
+void break_here(void *q)
+{
+}
+
+int main(void)
+{
+ long pagesize = sysconf(_SC_PAGESIZE);
+ void *p, *q;
+ int err;
+
+ p = mmap(NULL, pagesize * 2, PROT_READ | PROT_WRITE,
+ MAP_PRIVATE | MAP_ANONYMOUS, -1, 0);
+ assert(p != MAP_FAILED);
+ q = p + pagesize - 1;
+ strcpy(q, "42");
+
+ err = mprotect(p, pagesize * 2, PROT_NONE);
+ assert(err == 0);
+
+ break_here(q);
+
+ err = mprotect(p, pagesize * 2, PROT_READ);
+ assert(err == 0);
+ if (getenv("PROT_NONE_PY")) {
+ assert(strcmp(q, "24") == 0);
+ }
+
+ return EXIT_SUCCESS;
+}
diff --git a/tests/tcg/multiarch/Makefile.target
b/tests/tcg/multiarch/Makefile.target
index d31ba8d6ae..315a2e1358 100644
--- a/tests/tcg/multiarch/Makefile.target
+++ b/tests/tcg/multiarch/Makefile.target
@@ -101,13 +101,20 @@ run-gdbstub-registers: sha512
--bin $< --test $(MULTIARCH_SRC)/gdbstub/registers.py, \
checking register enumeration)
+run-gdbstub-prot-none: prot-none
+ $(call run-test, $@, env PROT_NONE_PY=1 $(GDB_SCRIPT) \
+ --gdb $(GDB) \
+ --qemu $(QEMU) --qargs "$(QEMU_OPTS)" \
+ --bin $< --test $(MULTIARCH_SRC)/gdbstub/prot-none.py, \
+ accessing PROT_NONE memory)
+
else
run-gdbstub-%:
$(call skip-test, "gdbstub test $*", "need working gdb with $(patsubst
-%,,$(TARGET_NAME)) support")
endif
EXTRA_RUNS += run-gdbstub-sha1 run-gdbstub-qxfer-auxv-read \
run-gdbstub-proc-mappings run-gdbstub-thread-breakpoint \
- run-gdbstub-registers
+ run-gdbstub-registers run-gdbstub-prot-none
# ARM Compatible Semi Hosting Tests
#
diff --git a/tests/tcg/multiarch/gdbstub/prot-none.py
b/tests/tcg/multiarch/gdbstub/prot-none.py
new file mode 100644
index 0000000000..e829d3ebc5
--- /dev/null
+++ b/tests/tcg/multiarch/gdbstub/prot-none.py
@@ -0,0 +1,36 @@
+"""Test that GDB can access PROT_NONE pages.
+
+This runs as a sourced script (via -x, via run-test.py).
+
+SPDX-License-Identifier: GPL-2.0-or-later
+"""
+import ctypes
+from test_gdbstub import main, report
+
+
+def probe_proc_self_mem():
+ buf = ctypes.create_string_buffer(b'aaa')
+ try:
+ with open("/proc/self/mem", "rb") as fp:
+ fp.seek(ctypes.addressof(buf))
+ return fp.read(3) == b'aaa'
+ except OSError:
+ return False
+
+
+def run_test():
+ """Run through the tests one by one"""
+ if not probe_proc_self_mem:
+ print("SKIP: /proc/self/mem is not usable")
+ exit(0)
+ gdb.Breakpoint("break_here")
+ gdb.execute("continue")
+ val = gdb.parse_and_eval("*(char[2] *)q").string()
+ report(val == "42", "{} == 42".format(val))
+ gdb.execute("set *(char[3] *)q = \"24\"")
+ gdb.execute("continue")
+ exitcode = int(gdb.parse_and_eval("$_exitcode"))
+ report(exitcode == 0, "{} == 0".format(exitcode))
+
+
+main(run_test)
--
2.34.1
- [PULL 04/31] target: Use vaddr in gen_intermediate_code, (continued)
- [PULL 04/31] target: Use vaddr in gen_intermediate_code, Richard Henderson, 2024/01/29
- [PULL 02/31] include/exec: Move vaddr defines to separate file, Richard Henderson, 2024/01/29
- [PULL 01/31] cpu-exec: simplify jump cache management, Richard Henderson, 2024/01/29
- [PULL 06/31] include/exec: typedef abi_ptr to vaddr, Richard Henderson, 2024/01/29
- [PULL 07/31] include/exec: Move PAGE_* macros to common header, Richard Henderson, 2024/01/29
- [PULL 08/31] include/exec: Move cpu_*()/cpu_env() to common header, Richard Henderson, 2024/01/29
- [PULL 05/31] include/exec: Use vaddr in DisasContextBase for virtual addresses, Richard Henderson, 2024/01/29
- [PULL 09/31] include/hw/core: Move do_interrupt in TCGCPUOps, Richard Henderson, 2024/01/29
- [PULL 10/31] include/hw/core: Remove i386 conditional on fake_user_interrupt, Richard Henderson, 2024/01/29
- [PULL 12/31] tests/tcg: Factor out gdbstub test functions, Richard Henderson, 2024/01/29
- [PULL 13/31] tests/tcg: Add the PROT_NONE gdbstub test,
Richard Henderson <=
- [PULL 11/31] linux-user: Allow gdbstub to ignore page protection, Richard Henderson, 2024/01/29
- [PULL 14/31] accel/tcg/cpu-exec: Use RCU_READ_LOCK_GUARD, Richard Henderson, 2024/01/29
- [PULL 15/31] target: Make qemu_target_page_mask() available for *-user, Richard Henderson, 2024/01/29
- [PULL 16/31] accel/tcg: Make use of qemu_target_page_mask() in perf.c, Richard Henderson, 2024/01/29
- [PULL 17/31] tcg: Make tb_cflags() usable from target-agnostic code, Richard Henderson, 2024/01/29
- [PULL 19/31] accel/tcg: Move perf and debuginfo support to tcg/, Richard Henderson, 2024/01/29
- [PULL 21/31] accel/tcg: Rename tcg_cpus_destroy() -> tcg_cpu_destroy(), Richard Henderson, 2024/01/29
- [PULL 18/31] accel/tcg: Remove #ifdef TARGET_I386 from perf.c, Richard Henderson, 2024/01/29
- [PULL 20/31] accel/tcg: Rename tcg_ss[] -> tcg_specific_ss[] in meson, Richard Henderson, 2024/01/29