summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAndreas Hansson <andreas.hansson@arm.com>2013-11-01 11:56:13 -0400
committerAndreas Hansson <andreas.hansson@arm.com>2013-11-01 11:56:13 -0400
commitc9a8b7b14744a4a3f0745483d3536f48fd29ca61 (patch)
tree8e0b66282c4cddad067e8213f5af3ba1c9f03b93
parent3e6da89419e5d53dc8a0df446473a4f28808bc46 (diff)
downloadgem5-c9a8b7b14744a4a3f0745483d3536f48fd29ca61.tar.xz
sim: Clarify the difference between tracing and debugging
This patch changes the name the command-line options related to debug output to all start with "debug" rather than being a mix of that and "trace". It also makes it clear that the breakpoint time is specified in ticks and not in cycles.
-rw-r--r--src/python/m5/debug.py2
-rw-r--r--src/python/m5/main.py33
-rw-r--r--src/sim/debug.cc2
-rw-r--r--src/sim/debug.hh4
4 files changed, 19 insertions, 22 deletions
diff --git a/src/python/m5/debug.py b/src/python/m5/debug.py
index 508bf2842..22b16aa5b 100644
--- a/src/python/m5/debug.py
+++ b/src/python/m5/debug.py
@@ -31,7 +31,7 @@ from UserDict import DictMixin
import internal
from internal.debug import SimpleFlag, CompoundFlag
-from internal.debug import schedBreakCycle, setRemoteGDBPort
+from internal.debug import schedBreak, setRemoteGDBPort
from m5.util import printList
def help():
diff --git a/src/python/m5/main.py b/src/python/m5/main.py
index a02d43335..611bc99bb 100644
--- a/src/python/m5/main.py
+++ b/src/python/m5/main.py
@@ -95,23 +95,20 @@ def parse_options():
# Debugging options
group("Debugging Options")
option("--debug-break", metavar="TIME[,TIME]", action='append', split=',',
- help="Cycle to create a breakpoint")
+ help="Tick to create a breakpoint")
option("--debug-help", action='store_true',
- help="Print help on trace flags")
+ help="Print help on debug flags")
option("--debug-flags", metavar="FLAG[,FLAG]", action='append', split=',',
- help="Sets the flags for tracing (-FLAG disables a flag)")
+ help="Sets the flags for debug output (-FLAG disables a flag)")
+ option("--debug-start", metavar="TIME", type='int',
+ help="Start debug output at TIME (must be in ticks)")
+ option("--debug-file", metavar="FILE", default="cout",
+ help="Sets the output file for debug [Default: %default]")
+ option("--debug-ignore", metavar="EXPR", action='append', split=':',
+ help="Ignore EXPR sim objects")
option("--remote-gdb-port", type='int', default=7000,
help="Remote gdb base port (set to 0 to disable listening)")
- # Tracing options
- group("Trace Options")
- option("--trace-start", metavar="TIME", type='int',
- help="Start tracing at TIME (must be in ticks)")
- option("--trace-file", metavar="FILE", default="cout",
- help="Sets the output file for tracing [Default: %default]")
- option("--trace-ignore", metavar="EXPR", action='append', split=':',
- help="Ignore EXPR sim objects")
-
# Help options
group("Help Options")
option("--list-sim-objects", action='store_true', default=False,
@@ -316,7 +313,7 @@ def main(*args):
# set debugging options
debug.setRemoteGDBPort(options.remote_gdb_port)
for when in options.debug_break:
- debug.schedBreakCycle(int(when))
+ debug.schedBreak(int(when))
if options.debug_flags:
check_tracing()
@@ -338,16 +335,16 @@ def main(*args):
else:
debug.flags[flag].enable()
- if options.trace_start:
+ if options.debug_start:
check_tracing()
- e = event.create(trace.enable, event.Event.Trace_Enable_Pri)
- event.mainq.schedule(e, options.trace_start)
+ e = event.create(trace.enable, event.Event.Debug_Enable_Pri)
+ event.mainq.schedule(e, options.debug_start)
else:
trace.enable()
- trace.output(options.trace_file)
+ trace.output(options.debug_file)
- for ignore in options.trace_ignore:
+ for ignore in options.debug_ignore:
check_tracing()
trace.ignore(ignore)
diff --git a/src/sim/debug.cc b/src/sim/debug.cc
index 51b92740e..bfb28198d 100644
--- a/src/sim/debug.cc
+++ b/src/sim/debug.cc
@@ -82,7 +82,7 @@ DebugBreakEvent::description() const
// (callable from debugger)
//
void
-schedBreakCycle(Tick when)
+schedBreak(Tick when)
{
mainEventQueue.schedule(new DebugBreakEvent, when);
warn("need to stop all queues");
diff --git a/src/sim/debug.hh b/src/sim/debug.hh
index bc1bb1e62..c29251a1e 100644
--- a/src/sim/debug.hh
+++ b/src/sim/debug.hh
@@ -40,9 +40,9 @@
/** Cause the simulator to execute a breakpoint
- * @param when the cycle to break
+ * @param when the tick to break
*/
-void schedBreakCycle(Tick when);
+void schedBreak(Tick when);
/** Cause the simulator to return to python to create a checkpoint
* @param when the cycle to break