summaryrefslogtreecommitdiff
path: root/src/dev/net/ns_gige.cc
diff options
context:
space:
mode:
authorSteve Reinhardt <steve.reinhardt@amd.com>2016-01-17 18:27:46 -0800
committerSteve Reinhardt <steve.reinhardt@amd.com>2016-01-17 18:27:46 -0800
commit28a0e5a165842bb3e5bed144cc5b64437810e4fb (patch)
tree75a66c54be627107db729e188485ff2ea9c2e3c3 /src/dev/net/ns_gige.cc
parentfcfe6e798d0bc8032e803c0a95dad026e87c3260 (diff)
downloadgem5-28a0e5a165842bb3e5bed144cc5b64437810e4fb.tar.xz
sim: don't ignore SIG_TRAP
By ignoring SIG_TRAP, using --debug-break <N> when not connected to a debugger becomes a no-op. Apparently this was intended to be a feature, though the rationale is not clear. If we don't ignore SIG_TRAP, then using --debug-break <N> when not connected to a debugger causes the simulation process to terminate at tick N. This is occasionally useful, e.g., if you just want to collect a trace for a specific window of execution then you can combine this with --debug-start to do exactly that. In addition to not ignoring the signal, this patch also updates the --debug-break help message and deletes a handful of unprotected calls to Debug::breakpoint() that relied on the prior behavior.
Diffstat (limited to 'src/dev/net/ns_gige.cc')
-rw-r--r--src/dev/net/ns_gige.cc3
1 files changed, 0 insertions, 3 deletions
diff --git a/src/dev/net/ns_gige.cc b/src/dev/net/ns_gige.cc
index a1dc23b50..3bf048972 100644
--- a/src/dev/net/ns_gige.cc
+++ b/src/dev/net/ns_gige.cc
@@ -951,7 +951,6 @@ NSGigE::cpuIntrPost(Tick when)
intrTick = when;
if (intrTick < curTick()) {
- Debug::breakpoint();
intrTick = curTick();
}
@@ -1725,7 +1724,6 @@ NSGigE::txKick()
tcp->sum(cksum(tcp));
txTcpChecksums++;
} else {
- Debug::breakpoint();
warn_once("TCPPKT set, but not UDP!\n");
}
}
@@ -1735,7 +1733,6 @@ NSGigE::txKick()
ip->sum(cksum(ip));
txIpChecksums++;
} else {
- Debug::breakpoint();
warn_once("IPPKT set, but not UDP!\n");
}
}