diff options
author | Tim Newsome <tim@sifive.com> | 2016-03-15 16:15:53 -0700 |
---|---|---|
committer | Tim Newsome <tim@sifive.com> | 2016-05-23 12:12:10 -0700 |
commit | 9f0d9a40322b31c8630f3591d2bd2247929e0629 (patch) | |
tree | f0d75fe9d828afa08f69a717325d511b1144b88b | |
parent | 0cc52e8e5dd9dadf7835869e0de4c9daa1e0d5db (diff) | |
download | spike-9f0d9a40322b31c8630f3591d2bd2247929e0629.zip spike-9f0d9a40322b31c8630f3591d2bd2247929e0629.tar.gz spike-9f0d9a40322b31c8630f3591d2bd2247929e0629.tar.bz2 |
Minor cleanup.
-rw-r--r-- | riscv/gdbserver.cc | 6 | ||||
-rwxr-xr-x | tests/gdbserver-smoke.py | 2 |
2 files changed, 4 insertions, 4 deletions
diff --git a/riscv/gdbserver.cc b/riscv/gdbserver.cc index c83c83c..0b36421 100644 --- a/riscv/gdbserver.cc +++ b/riscv/gdbserver.cc @@ -91,7 +91,6 @@ gdbserver_t::gdbserver_t(uint16_t port, sim_t *sim) : client_fd(0), recv_buf(64 * 1024), send_buf(64 * 1024) { - // TODO: listen on socket socket_fd = socket(AF_INET, SOCK_STREAM, 0); if (socket_fd == -1) { fprintf(stderr, "failed to make socket: %s (%d)\n", strerror(errno), errno); @@ -403,8 +402,9 @@ void gdbserver_t::handle_register_read(const std::vector<uint8_t> &packet) try { send(p->get_csr(n - REG_CSR0)); } catch(trap_t& t) { - // TODO, can we return error here, and update 'info reg all' to display - // it sensibly? + // It would be nicer to return an error here, but if you do that then gdb + // exits out of 'info registers all' as soon as it encounters a register + // that can't be read. send((reg_t) 0); } } else { diff --git a/tests/gdbserver-smoke.py b/tests/gdbserver-smoke.py index 306f6f2..51e8cc9 100755 --- a/tests/gdbserver-smoke.py +++ b/tests/gdbserver-smoke.py @@ -20,7 +20,7 @@ class DebugTest(unittest.TestCase): self.spike.wait() def test_turbostep(self): - """Single step until the program exits. TODO""" + """Single step a bunch of times.""" last_pc = None for _ in range(100): self.gdb.command("stepi") |