This patch adds a test case to test SME register exposure to a remote gdb debugging session. This test simply sets and reads SME registers.
Signed-off-by: Vacha Bhavsar <vacha.bhav...@oss.qualcomm.com> --- Changes since v4: - this patch was not present in v4, it was added based on suggestions during the review of v4 --- configure | 6 ++ tests/tcg/aarch64/Makefile.target | 23 ++++- tests/tcg/aarch64/gdbstub/test-sme.py | 119 ++++++++++++++++++++++++++ 3 files changed, 147 insertions(+), 1 deletion(-) create mode 100644 tests/tcg/aarch64/gdbstub/test-sme.py diff --git a/configure b/configure index 825057ebf1..177d3775ac 100755 --- a/configure +++ b/configure @@ -1837,6 +1837,12 @@ for target in $target_list; do echo "GDB=$gdb_bin" >> $config_target_mak fi + if test "${gdb_arches#*$arch}" != "$gdb_arches" && version_ge $gdb_version 14.1; then + echo "GDB_HAS_SME_TILES=y" >> $config_target_mak + else + echo "GDB_HAS_SME_TILES=n" >> $config_target_mak + fi + if test "${gdb_arches#*aarch64}" != "$gdb_arches" && version_ge $gdb_version 15.1; then echo "GDB_HAS_MTE=y" >> $config_target_mak fi diff --git a/tests/tcg/aarch64/Makefile.target b/tests/tcg/aarch64/Makefile.target index 16ddcf4f88..1a33ef17a0 100644 --- a/tests/tcg/aarch64/Makefile.target +++ b/tests/tcg/aarch64/Makefile.target @@ -132,7 +132,28 @@ run-gdbstub-sve-ioctls: sve-ioctls --bin $< --test $(AARCH64_SRC)/gdbstub/test-sve-ioctl.py, \ basic gdbstub SVE ZLEN support) -EXTRA_RUNS += run-gdbstub-sysregs run-gdbstub-sve-ioctls +ifneq ($(CROSS_AS_HAS_ARMV9_SME),) +# SME gdbstub test +ifeq ($(GDB_HAS_SME_TILES),y) +run-gdbstub-sysregs-sme: sysregs + $(call run-test, $@, $(GDB_SCRIPT) \ + --gdb $(GDB) \ + --qemu $(QEMU) --qargs "$(QEMU_OPTS)" \ + --bin $< --test $(AARCH64_SRC)/gdbstub/test-sme.py \ + -- test_sme --gdb_sme_tile_support, \ + basic gdbstub SME support) +else +run-gdbstub-sysregs-sme: sysregs + $(call run-test, $@, $(GDB_SCRIPT) \ + --gdb $(GDB) \ + --qemu $(QEMU) --qargs "$(QEMU_OPTS)" \ + --bin $< --test $(AARCH64_SRC)/gdbstub/test-sme.py, \ + basic gdbstub SME support) + +endif +endif + +EXTRA_RUNS += run-gdbstub-sysregs run-gdbstub-sve-ioctls run-gdbstub-sysregs-sme ifeq ($(GDB_HAS_MTE),y) run-gdbstub-mte: mte-8 diff --git a/tests/tcg/aarch64/gdbstub/test-sme.py b/tests/tcg/aarch64/gdbstub/test-sme.py new file mode 100644 index 0000000000..c2b9d774ae --- /dev/null +++ b/tests/tcg/aarch64/gdbstub/test-sme.py @@ -0,0 +1,119 @@ +from __future__ import print_function +# +# Test the SME registers are visible and changeable via gdbstub +# +# This is launched via tests/guest-debug/run-test.py +# + +import argparse +import gdb +from test_gdbstub import main, report + +MAGIC = 0x01020304 + +def run_test(): + "Run through the tests one by one" + + frame = gdb.selected_frame() + rname = "za" + za = frame.read_register(rname) + report(True, "Reading %s" % rname) + + for i in range(0, 16): + for j in range(0, 16): + cmd = "set $za[%d][%d] = 0x01" % (i, j) + gdb.execute(cmd) + report(True, "%s" % cmd) + for i in range(0, 16): + for j in range(0, 16): + reg = "$za[%d][%d]" % (i, j) + v = gdb.parse_and_eval(reg) + report(str(v.type) == "uint8_t", + "size of %s" % (reg)) + report(int(v) == 0x1, "%s is 0x%x" % (reg, 0x1)) + +def run_test_slices(): + "Run through the tests one by one" + + frame = gdb.selected_frame() + rname = "za" + za = frame.read_register(rname) + report(True, "Reading %s" % rname) + + for i in range(0, 16): + for j in range(0, 16): + cmd = "set $za[%d][%d] = 0x01" % (i, j) + gdb.execute(cmd) + report(True, "%s" % cmd) + for i in range(0, 16): + for j in range(0, 16): + reg = "$za[%d][%d]" % (i, j) + v = gdb.parse_and_eval(reg) + report(str(v.type) == "uint8_t", + "size of %s" % (reg)) + report(int(v) == 0x1, "%s is 0x%x" % (reg, 0x1)) + + for i in range(0, 4): + for j in range(0, 4): + for k in range(0, 4): + cmd = "set $za%dhq%d[%d] = 0x%x" % (i, j, k, MAGIC) + gdb.execute(cmd) + report(True, "%s" % cmd) + for j in range(0, 4): + for k in range(0, 4): + reg = "$za%dhq%d[%d]" % (i, j, k) + v = gdb.parse_and_eval(reg) + report(str(v.type) == "uint128_t", + "size of %s" % (reg)) + report(int(v) == MAGIC, "%s is 0x%x" % (reg, MAGIC)) + + for j in range(0, 4): + for k in range(0, 4): + cmd = "set $za%dvq%d[%d] = 0x%x" % (i, j, k, MAGIC) + gdb.execute(cmd) + report(True, "%s" % cmd) + for j in range(0, 4): + for k in range(0, 4): + reg = "$za%dvq%d[%d]" % (i, j, k) + v = gdb.parse_and_eval(reg) + report(str(v.type) == "uint128_t", + "size of %s" % (reg)) + report(int(v) == MAGIC, "%s is 0x%x" % (reg, MAGIC)) + + for i in range(0, 4): + for j in range(0, 4): + for k in range(0, 4): + cmd = "set $za%dhd%d[%d] = 0x%x" % (i, j, k, MAGIC) + gdb.execute(cmd) + report(True, "%s" % cmd) + for j in range(0, 4): + for k in range(0, 4): + reg = "$za%dhd%d[%d]" % (i, j, k) + v = gdb.parse_and_eval(reg) + report(str(v.type) == "uint64_t", + "size of %s" % (reg)) + report(int(v) == MAGIC, "%s is 0x%x" % (reg, MAGIC)) + + for j in range(0, 4): + for k in range(0, 4): + cmd = "set $za%dvd%d[%d] = 0x%x" % (i, j, k, MAGIC) + gdb.execute(cmd) + report(True, "%s" % cmd) + for j in range(0, 4): + for k in range(0, 4): + reg = "$za%dvd%d[%d]" % (i, j, k) + v = gdb.parse_and_eval(reg) + report(str(v.type) == "uint64_t", + "size of %s" % (reg)) + report(int(v) == MAGIC, "%s is 0x%x" % (reg, MAGIC)) + + +parser = argparse.ArgumentParser(description="A gdbstub test for SME support") +parser.add_argument("--gdb_sme_tile_support", help="GDB support for SME tiles", \ + action="store_true") +args = parser.parse_args() + +if args.gdb_sme_tile_support: + main(run_test_slices, expected_arch="aarch64") +else: + main(run_test, expected_arch="aarch64") \ No newline at end of file -- 2.34.1