aboutsummaryrefslogtreecommitdiff
path: root/debug/gdbserver.py
diff options
context:
space:
mode:
authorTim Newsome <tim@sifive.com>2022-12-14 16:52:28 -0800
committerGitHub <noreply@github.com>2022-12-14 16:52:28 -0800
commit898c20bf0b74b2899d7491823cbbac84b5508751 (patch)
treecea2fb6d7176f4cfe14fc1d8f3bf2823a481fe55 /debug/gdbserver.py
parent45f4da6224ee254f235ff223f77e69dccf100c46 (diff)
downloadriscv-tests-898c20bf0b74b2899d7491823cbbac84b5508751.zip
riscv-tests-898c20bf0b74b2899d7491823cbbac84b5508751.tar.gz
riscv-tests-898c20bf0b74b2899d7491823cbbac84b5508751.tar.bz2
debug: Remove unnecessary exit() functions. (#437)
Also make the semi-hosting test program return 10. That's more fragile than returning 0, so makes for a better test.
Diffstat (limited to 'debug/gdbserver.py')
-rwxr-xr-xdebug/gdbserver.py12
1 files changed, 1 insertions, 11 deletions
diff --git a/debug/gdbserver.py b/debug/gdbserver.py
index a61722d..ac7b162 100755
--- a/debug/gdbserver.py
+++ b/debug/gdbserver.py
@@ -521,10 +521,7 @@ class DebugTest(GdbSingleHartTest):
self.gdb.b("_exit")
def exit(self, expected_result=0xc86455d4):
- output = self.gdb.c()
- assertIn("Breakpoint", output)
- assertIn("_exit", output)
- assertEqual(self.gdb.p("status"), expected_result)
+ super().exit(expected_result)
class DebugCompareSections(DebugTest):
def test(self):
@@ -944,13 +941,6 @@ class Semihosting(GdbSingleHartTest):
self.parkOtherHarts()
self.gdb.b("_exit")
- def exit(self, expected_result=0):
- output = self.gdb.c()
- assertIn("Breakpoint", output)
- assertIn("_exit", output)
- assertEqual(self.gdb.p("status"), expected_result)
- return output
-
def test(self):
with tempfile.NamedTemporaryFile(suffix=".data") as temp:
self.gdb.b("main:begin")