We can do this all in the run-test.py script so remove the extraneous
bits from the individual tests which got copied from the original
non-CI gdb tests.
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
tests/guest-debug/run-test.py | 2 ++
tests/tcg/aarch64/gdbstub/test-sve-ioctl.py | 3 ---
tests/tcg/aarch64/gdbstub/test-sve.py | 3 ---
tests/tcg/multiarch/gdbstub/memory.py | 3 ---
tests/tcg/multiarch/gdbstub/sha1.py | 4 ----
tests/tcg/multiarch/gdbstub/test-proc-mappings.py | 4 ----
tests/tcg/multiarch/gdbstub/test-qxfer-auxv-read.py | 4 ----
tests/tcg/multiarch/gdbstub/test-thread-breakpoint.py | 4 ----
tests/tcg/s390x/gdbstub/test-signals-s390x.py | 4 ----
tests/tcg/s390x/gdbstub/test-svc.py | 4 ----
10 files changed, 2 insertions(+), 33 deletions(-)
diff --git a/tests/guest-debug/run-test.py b/tests/guest-debug/run-test.py
index de6106a5e5..c0d0075e2e 100755
--- a/tests/guest-debug/run-test.py
+++ b/tests/guest-debug/run-test.py
@@ -86,6 +86,8 @@ def log(output, msg):
gdb_cmd += " %s" % (args.gdb_args)
# run quietly and ignore .gdbinit
gdb_cmd += " -q -n -batch"
+ # disable pagination
+ gdb_cmd += " -ex 'set pagination off'"
# disable prompts in case of crash
gdb_cmd += " -ex 'set confirm off'"
# connect to remote
diff --git a/tests/tcg/aarch64/gdbstub/test-sve-ioctl.py b/tests/tcg/aarch64/gdbstub/test-sve-ioctl.py
index b9ef169c1a..ee8d467e59 100644
--- a/tests/tcg/aarch64/gdbstub/test-sve-ioctl.py
+++ b/tests/tcg/aarch64/gdbstub/test-sve-ioctl.py
@@ -76,9 +76,6 @@ def run_test():
exit(0)
try:
- # These are not very useful in scripts
- gdb.execute("set pagination off")
-
# Run the actual tests
run_test()
except:
diff --git a/tests/tcg/aarch64/gdbstub/test-sve.py b/tests/tcg/aarch64/gdbstub/test-sve.py
index ef57c7412c..afd8ece98d 100644
--- a/tests/tcg/aarch64/gdbstub/test-sve.py
+++ b/tests/tcg/aarch64/gdbstub/test-sve.py
@@ -66,9 +66,6 @@ def run_test():
exit(0)
try:
- # These are not very useful in scripts
- gdb.execute("set pagination off")
-
# Run the actual tests
run_test()
except:
diff --git a/tests/tcg/multiarch/gdbstub/memory.py b/tests/tcg/multiarch/gdbstub/memory.py
index 67864ad902..dd25e72281 100644
--- a/tests/tcg/multiarch/gdbstub/memory.py
+++ b/tests/tcg/multiarch/gdbstub/memory.py
@@ -115,9 +115,6 @@ def run_test():
exit(0)
try:
- # These are not very useful in scripts
- gdb.execute("set pagination off")
-
# Run the actual tests
run_test()
except (gdb.error):
diff --git a/tests/tcg/multiarch/gdbstub/sha1.py b/tests/tcg/multiarch/gdbstub/sha1.py
index 423b720e6d..416728415f 100644
--- a/tests/tcg/multiarch/gdbstub/sha1.py
+++ b/tests/tcg/multiarch/gdbstub/sha1.py
@@ -73,10 +73,6 @@ def run_test():
exit(0)
try:
- # These are not very useful in scripts
- gdb.execute("set pagination off")
- gdb.execute("set confirm off")
-
# Run the actual tests
run_test()
except (gdb.error):
diff --git a/tests/tcg/multiarch/gdbstub/test-proc-mappings.py b/tests/tcg/multiarch/gdbstub/test-proc-mappings.py
index 5e3e5a2fb7..04ec61d219 100644
--- a/tests/tcg/multiarch/gdbstub/test-proc-mappings.py
+++ b/tests/tcg/multiarch/gdbstub/test-proc-mappings.py
@@ -51,10 +51,6 @@ def main():
exit(0)
try:
- # These are not very useful in scripts
- gdb.execute("set pagination off")
- gdb.execute("set confirm off")
-
# Run the actual tests
run_test()
except gdb.error:
diff --git a/tests/tcg/multiarch/gdbstub/test-qxfer-auxv-read.py b/tests/tcg/multiarch/gdbstub/test-qxfer-auxv-read.py
index d91e8fdf19..926fa962b7 100644
--- a/tests/tcg/multiarch/gdbstub/test-qxfer-auxv-read.py
+++ b/tests/tcg/multiarch/gdbstub/test-qxfer-auxv-read.py
@@ -42,10 +42,6 @@ def run_test():
exit(0)
try:
- # These are not very useful in scripts
- gdb.execute("set pagination off")
- gdb.execute("set confirm off")
-
# Run the actual tests
run_test()
except (gdb.error):
diff --git a/tests/tcg/multiarch/gdbstub/test-thread-breakpoint.py b/tests/tcg/multiarch/gdbstub/test-thread-breakpoint.py
index 798d508bc7..e57d2a8db8 100644
--- a/tests/tcg/multiarch/gdbstub/test-thread-breakpoint.py
+++ b/tests/tcg/multiarch/gdbstub/test-thread-breakpoint.py
@@ -45,10 +45,6 @@ def run_test():
exit(0)
try:
- # These are not very useful in scripts
- gdb.execute("set pagination off")
- gdb.execute("set confirm off")
-
# Run the actual tests
run_test()
except (gdb.error):
diff --git a/tests/tcg/s390x/gdbstub/test-signals-s390x.py b/tests/tcg/s390x/gdbstub/test-signals-s390x.py
index 80a284b475..ca2bbc0b03 100644
--- a/tests/tcg/s390x/gdbstub/test-signals-s390x.py
+++ b/tests/tcg/s390x/gdbstub/test-signals-s390x.py
@@ -61,10 +61,6 @@ def run_test():
exit(0)
try:
- # These are not very useful in scripts
- gdb.execute("set pagination off")
- gdb.execute("set confirm off")
-
# Run the actual tests
run_test()
except (gdb.error):
diff --git a/tests/tcg/s390x/gdbstub/test-svc.py b/tests/tcg/s390x/gdbstub/test-svc.py
index 18fad3f163..804705fede 100644
--- a/tests/tcg/s390x/gdbstub/test-svc.py
+++ b/tests/tcg/s390x/gdbstub/test-svc.py
@@ -49,10 +49,6 @@ def main():
exit(0)
try:
- # These are not very useful in scripts
- gdb.execute("set pagination off")
- gdb.execute("set confirm off")
-
# Run the actual tests
run_test()
except gdb.error:
--
2.39.2
On 8/10/23 08:36, Alex Bennée wrote: > We can do this all in the run-test.py script so remove the extraneous > bits from the individual tests which got copied from the original > non-CI gdb tests. > > Signed-off-by: Alex Bennée<alex.bennee@linaro.org> > --- > tests/guest-debug/run-test.py | 2 ++ > tests/tcg/aarch64/gdbstub/test-sve-ioctl.py | 3 --- > tests/tcg/aarch64/gdbstub/test-sve.py | 3 --- > tests/tcg/multiarch/gdbstub/memory.py | 3 --- > tests/tcg/multiarch/gdbstub/sha1.py | 4 ---- > tests/tcg/multiarch/gdbstub/test-proc-mappings.py | 4 ---- > tests/tcg/multiarch/gdbstub/test-qxfer-auxv-read.py | 4 ---- > tests/tcg/multiarch/gdbstub/test-thread-breakpoint.py | 4 ---- > tests/tcg/s390x/gdbstub/test-signals-s390x.py | 4 ---- > tests/tcg/s390x/gdbstub/test-svc.py | 4 ---- > 10 files changed, 2 insertions(+), 33 deletions(-) Reviewed-by: Richard Henderson <richard.henderson@linaro.org> r~
On 10/8/23 17:36, Alex Bennée wrote: > We can do this all in the run-test.py script so remove the extraneous > bits from the individual tests which got copied from the original > non-CI gdb tests. > > Signed-off-by: Alex Bennée <alex.bennee@linaro.org> > --- > tests/guest-debug/run-test.py | 2 ++ > tests/tcg/aarch64/gdbstub/test-sve-ioctl.py | 3 --- > tests/tcg/aarch64/gdbstub/test-sve.py | 3 --- > tests/tcg/multiarch/gdbstub/memory.py | 3 --- > tests/tcg/multiarch/gdbstub/sha1.py | 4 ---- > tests/tcg/multiarch/gdbstub/test-proc-mappings.py | 4 ---- > tests/tcg/multiarch/gdbstub/test-qxfer-auxv-read.py | 4 ---- > tests/tcg/multiarch/gdbstub/test-thread-breakpoint.py | 4 ---- > tests/tcg/s390x/gdbstub/test-signals-s390x.py | 4 ---- > tests/tcg/s390x/gdbstub/test-svc.py | 4 ---- > 10 files changed, 2 insertions(+), 33 deletions(-) Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
© 2016 - 2024 Red Hat, Inc.