diff options
author | Andreas Hansson <andreas.hansson@arm.com> | 2013-11-01 11:56:13 -0400 |
---|---|---|
committer | Andreas Hansson <andreas.hansson@arm.com> | 2013-11-01 11:56:13 -0400 |
commit | c9a8b7b14744a4a3f0745483d3536f48fd29ca61 (patch) | |
tree | 8e0b66282c4cddad067e8213f5af3ba1c9f03b93 | |
parent | 3e6da89419e5d53dc8a0df446473a4f28808bc46 (diff) | |
download | gem5-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.py | 2 | ||||
-rw-r--r-- | src/python/m5/main.py | 33 | ||||
-rw-r--r-- | src/sim/debug.cc | 2 | ||||
-rw-r--r-- | src/sim/debug.hh | 4 |
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 |