Re: [m5-dev] changeset in m5: events: Make trace events happen at the right p...
Can you commit? Thanks, Nate On Wed, Feb 25, 2009 at 8:59 AM, Ali Saidi sa...@umich.edu wrote: That fixed it. Ali On Feb 25, 2009, at 1:37 AM, nathan binkert wrote: Sorry. It should be event.Event... Does that fix it? On Tue, Feb 24, 2009 at 10:27 PM, Ali Saidi sa...@umich.edu wrote: This changeset broke tracing that starts at a tick other than 0. command line: ./build/ALPHA_FS/m5.opt --trace-flags=Exec --trace- start=4 configs/example/fs.py Traceback (most recent call last): File string, line 1, in module File /tmp/m5/src/python/m5/main.py, line 315, in main e = event.create(trace.enable, Event.Trace_Enable_Pri) NameError: name 'Event' is not defined Ali On Feb 18, 2009, at 2:37 PM, Nathan Binkert wrote: changeset e9f9c0f7e5f0 in /z/repo/m5 details: http://repo.m5sim.org/m5?cmd=changeset;node=e9f9c0f7e5f0 description: events: Make trace events happen at the right priority. Also, while we're at it, remember that priorities are in the Event class and add a disable method to disable tracing. diffstat: 3 files changed, 9 insertions(+), 3 deletions(-) src/python/m5/event.py | 7 +-- src/python/m5/main.py | 2 +- src/python/m5/trace.py | 3 +++ diffs (47 lines): diff -r ce12f1cf978c -r e9f9c0f7e5f0 src/python/m5/event.py --- a/src/python/m5/event.py Tue Feb 17 19:24:46 2009 -0800 +++ b/src/python/m5/event.py Wed Feb 18 10:00:15 2009 -0800 @@ -35,13 +35,16 @@ def create(obj, priority=None): if priority is None: - priority = internal.event.Event.Default_Pri + priority = Event.Default_Pri return PythonEvent(obj, priority) + +# As a reminder, priorities found in sim/eventq.hh are stuck into the +# Event class by swig class Event(PythonEvent): def __init__(self, priority=None): if priority is None: - priority = internal.event.Event.Default_Pri + priority = Event.Default_Pri super(Event, self).__init__(self, priority) class ProgressEvent(Event): diff -r ce12f1cf978c -r e9f9c0f7e5f0 src/python/m5/main.py --- a/src/python/m5/main.py Tue Feb 17 19:24:46 2009 -0800 +++ b/src/python/m5/main.py Wed Feb 18 10:00:15 2009 -0800 @@ -312,7 +312,7 @@ if options.trace_start: check_tracing() - e = event.create(trace.enable) + e = event.create(trace.enable, Event.Trace_Enable_Pri) event.mainq.schedule(e, options.trace_start) else: trace.enable() diff -r ce12f1cf978c -r e9f9c0f7e5f0 src/python/m5/trace.py --- a/src/python/m5/trace.py Tue Feb 17 19:24:46 2009 -0800 +++ b/src/python/m5/trace.py Wed Feb 18 10:00:15 2009 -0800 @@ -32,6 +32,9 @@ from internal.trace import clear, output, set, ignore +def disable(): + internal.trace.cvar.enabled = False + def enable(): internal.trace.cvar.enabled = True ___ m5-dev mailing list m5-dev@m5sim.org http://m5sim.org/mailman/listinfo/m5-dev ___ m5-dev mailing list m5-dev@m5sim.org http://m5sim.org/mailman/listinfo/m5-dev ___ m5-dev mailing list m5-dev@m5sim.org http://m5sim.org/mailman/listinfo/m5-dev ___ m5-dev mailing list m5-dev@m5sim.org http://m5sim.org/mailman/listinfo/m5-dev ___ m5-dev mailing list m5-dev@m5sim.org http://m5sim.org/mailman/listinfo/m5-dev
Re: [m5-dev] changeset in m5: events: Make trace events happen at the right p...
Please run all the regressions before you do. I'm almost ready to push and I'd hate to start checking everything again. Gabe nathan binkert wrote: Can you commit? Thanks, Nate On Wed, Feb 25, 2009 at 8:59 AM, Ali Saidi sa...@umich.edu wrote: That fixed it. Ali On Feb 25, 2009, at 1:37 AM, nathan binkert wrote: Sorry. It should be event.Event... Does that fix it? On Tue, Feb 24, 2009 at 10:27 PM, Ali Saidi sa...@umich.edu wrote: This changeset broke tracing that starts at a tick other than 0. command line: ./build/ALPHA_FS/m5.opt --trace-flags=Exec --trace- start=4 configs/example/fs.py Traceback (most recent call last): File string, line 1, in module File /tmp/m5/src/python/m5/main.py, line 315, in main e = event.create(trace.enable, Event.Trace_Enable_Pri) NameError: name 'Event' is not defined Ali On Feb 18, 2009, at 2:37 PM, Nathan Binkert wrote: changeset e9f9c0f7e5f0 in /z/repo/m5 details: http://repo.m5sim.org/m5?cmd=changeset;node=e9f9c0f7e5f0 description: events: Make trace events happen at the right priority. Also, while we're at it, remember that priorities are in the Event class and add a disable method to disable tracing. diffstat: 3 files changed, 9 insertions(+), 3 deletions(-) src/python/m5/event.py |7 +-- src/python/m5/main.py |2 +- src/python/m5/trace.py |3 +++ diffs (47 lines): diff -r ce12f1cf978c -r e9f9c0f7e5f0 src/python/m5/event.py --- a/src/python/m5/event.py Tue Feb 17 19:24:46 2009 -0800 +++ b/src/python/m5/event.py Wed Feb 18 10:00:15 2009 -0800 @@ -35,13 +35,16 @@ def create(obj, priority=None): if priority is None: -priority = internal.event.Event.Default_Pri +priority = Event.Default_Pri return PythonEvent(obj, priority) + +# As a reminder, priorities found in sim/eventq.hh are stuck into the +# Event class by swig class Event(PythonEvent): def __init__(self, priority=None): if priority is None: -priority = internal.event.Event.Default_Pri +priority = Event.Default_Pri super(Event, self).__init__(self, priority) class ProgressEvent(Event): diff -r ce12f1cf978c -r e9f9c0f7e5f0 src/python/m5/main.py --- a/src/python/m5/main.py Tue Feb 17 19:24:46 2009 -0800 +++ b/src/python/m5/main.py Wed Feb 18 10:00:15 2009 -0800 @@ -312,7 +312,7 @@ if options.trace_start: check_tracing() -e = event.create(trace.enable) +e = event.create(trace.enable, Event.Trace_Enable_Pri) event.mainq.schedule(e, options.trace_start) else: trace.enable() diff -r ce12f1cf978c -r e9f9c0f7e5f0 src/python/m5/trace.py --- a/src/python/m5/trace.py Tue Feb 17 19:24:46 2009 -0800 +++ b/src/python/m5/trace.py Wed Feb 18 10:00:15 2009 -0800 @@ -32,6 +32,9 @@ from internal.trace import clear, output, set, ignore +def disable(): +internal.trace.cvar.enabled = False + def enable(): internal.trace.cvar.enabled = True ___ m5-dev mailing list m5-dev@m5sim.org http://m5sim.org/mailman/listinfo/m5-dev ___ m5-dev mailing list m5-dev@m5sim.org http://m5sim.org/mailman/listinfo/m5-dev ___ m5-dev mailing list m5-dev@m5sim.org http://m5sim.org/mailman/listinfo/m5-dev ___ m5-dev mailing list m5-dev@m5sim.org http://m5sim.org/mailman/listinfo/m5-dev ___ m5-dev mailing list m5-dev@m5sim.org http://m5sim.org/mailman/listinfo/m5-dev ___ m5-dev mailing list m5-dev@m5sim.org http://m5sim.org/mailman/listinfo/m5-dev
Re: [m5-dev] changeset in m5: events: Make trace events happen at the right p...
This change won't affect the regressions. It's only when you use the --trace-start command line option. On Wed, Feb 25, 2009 at 9:29 AM, Gabe Black gbl...@eecs.umich.edu wrote: Please run all the regressions before you do. I'm almost ready to push and I'd hate to start checking everything again. Gabe nathan binkert wrote: Can you commit? Thanks, Nate On Wed, Feb 25, 2009 at 8:59 AM, Ali Saidi sa...@umich.edu wrote: That fixed it. Ali On Feb 25, 2009, at 1:37 AM, nathan binkert wrote: Sorry. It should be event.Event... Does that fix it? On Tue, Feb 24, 2009 at 10:27 PM, Ali Saidi sa...@umich.edu wrote: This changeset broke tracing that starts at a tick other than 0. command line: ./build/ALPHA_FS/m5.opt --trace-flags=Exec --trace- start=4 configs/example/fs.py Traceback (most recent call last): File string, line 1, in module File /tmp/m5/src/python/m5/main.py, line 315, in main e = event.create(trace.enable, Event.Trace_Enable_Pri) NameError: name 'Event' is not defined Ali On Feb 18, 2009, at 2:37 PM, Nathan Binkert wrote: changeset e9f9c0f7e5f0 in /z/repo/m5 details: http://repo.m5sim.org/m5?cmd=changeset;node=e9f9c0f7e5f0 description: events: Make trace events happen at the right priority. Also, while we're at it, remember that priorities are in the Event class and add a disable method to disable tracing. diffstat: 3 files changed, 9 insertions(+), 3 deletions(-) src/python/m5/event.py | 7 +-- src/python/m5/main.py | 2 +- src/python/m5/trace.py | 3 +++ diffs (47 lines): diff -r ce12f1cf978c -r e9f9c0f7e5f0 src/python/m5/event.py --- a/src/python/m5/event.py Tue Feb 17 19:24:46 2009 -0800 +++ b/src/python/m5/event.py Wed Feb 18 10:00:15 2009 -0800 @@ -35,13 +35,16 @@ def create(obj, priority=None): if priority is None: - priority = internal.event.Event.Default_Pri + priority = Event.Default_Pri return PythonEvent(obj, priority) + +# As a reminder, priorities found in sim/eventq.hh are stuck into the +# Event class by swig class Event(PythonEvent): def __init__(self, priority=None): if priority is None: - priority = internal.event.Event.Default_Pri + priority = Event.Default_Pri super(Event, self).__init__(self, priority) class ProgressEvent(Event): diff -r ce12f1cf978c -r e9f9c0f7e5f0 src/python/m5/main.py --- a/src/python/m5/main.py Tue Feb 17 19:24:46 2009 -0800 +++ b/src/python/m5/main.py Wed Feb 18 10:00:15 2009 -0800 @@ -312,7 +312,7 @@ if options.trace_start: check_tracing() - e = event.create(trace.enable) + e = event.create(trace.enable, Event.Trace_Enable_Pri) event.mainq.schedule(e, options.trace_start) else: trace.enable() diff -r ce12f1cf978c -r e9f9c0f7e5f0 src/python/m5/trace.py --- a/src/python/m5/trace.py Tue Feb 17 19:24:46 2009 -0800 +++ b/src/python/m5/trace.py Wed Feb 18 10:00:15 2009 -0800 @@ -32,6 +32,9 @@ from internal.trace import clear, output, set, ignore +def disable(): + internal.trace.cvar.enabled = False + def enable(): internal.trace.cvar.enabled = True ___ m5-dev mailing list m5-dev@m5sim.org http://m5sim.org/mailman/listinfo/m5-dev ___ m5-dev mailing list m5-dev@m5sim.org http://m5sim.org/mailman/listinfo/m5-dev ___ m5-dev mailing list m5-dev@m5sim.org http://m5sim.org/mailman/listinfo/m5-dev ___ m5-dev mailing list m5-dev@m5sim.org http://m5sim.org/mailman/listinfo/m5-dev ___ m5-dev mailing list m5-dev@m5sim.org http://m5sim.org/mailman/listinfo/m5-dev ___ m5-dev mailing list m5-dev@m5sim.org http://m5sim.org/mailman/listinfo/m5-dev ___ m5-dev mailing list m5-dev@m5sim.org http://m5sim.org/mailman/listinfo/m5-dev
Re: [m5-dev] changeset in m5: events: Make trace events happen at the right p...
This changeset broke tracing that starts at a tick other than 0. command line: ./build/ALPHA_FS/m5.opt --trace-flags=Exec --trace- start=4 configs/example/fs.py Traceback (most recent call last): File string, line 1, in module File /tmp/m5/src/python/m5/main.py, line 315, in main e = event.create(trace.enable, Event.Trace_Enable_Pri) NameError: name 'Event' is not defined Ali On Feb 18, 2009, at 2:37 PM, Nathan Binkert wrote: changeset e9f9c0f7e5f0 in /z/repo/m5 details: http://repo.m5sim.org/m5?cmd=changeset;node=e9f9c0f7e5f0 description: events: Make trace events happen at the right priority. Also, while we're at it, remember that priorities are in the Event class and add a disable method to disable tracing. diffstat: 3 files changed, 9 insertions(+), 3 deletions(-) src/python/m5/event.py |7 +-- src/python/m5/main.py |2 +- src/python/m5/trace.py |3 +++ diffs (47 lines): diff -r ce12f1cf978c -r e9f9c0f7e5f0 src/python/m5/event.py --- a/src/python/m5/event.py Tue Feb 17 19:24:46 2009 -0800 +++ b/src/python/m5/event.py Wed Feb 18 10:00:15 2009 -0800 @@ -35,13 +35,16 @@ def create(obj, priority=None): if priority is None: -priority = internal.event.Event.Default_Pri +priority = Event.Default_Pri return PythonEvent(obj, priority) + +# As a reminder, priorities found in sim/eventq.hh are stuck into the +# Event class by swig class Event(PythonEvent): def __init__(self, priority=None): if priority is None: -priority = internal.event.Event.Default_Pri +priority = Event.Default_Pri super(Event, self).__init__(self, priority) class ProgressEvent(Event): diff -r ce12f1cf978c -r e9f9c0f7e5f0 src/python/m5/main.py --- a/src/python/m5/main.py Tue Feb 17 19:24:46 2009 -0800 +++ b/src/python/m5/main.py Wed Feb 18 10:00:15 2009 -0800 @@ -312,7 +312,7 @@ if options.trace_start: check_tracing() -e = event.create(trace.enable) +e = event.create(trace.enable, Event.Trace_Enable_Pri) event.mainq.schedule(e, options.trace_start) else: trace.enable() diff -r ce12f1cf978c -r e9f9c0f7e5f0 src/python/m5/trace.py --- a/src/python/m5/trace.py Tue Feb 17 19:24:46 2009 -0800 +++ b/src/python/m5/trace.py Wed Feb 18 10:00:15 2009 -0800 @@ -32,6 +32,9 @@ from internal.trace import clear, output, set, ignore +def disable(): +internal.trace.cvar.enabled = False + def enable(): internal.trace.cvar.enabled = True ___ m5-dev mailing list m5-dev@m5sim.org http://m5sim.org/mailman/listinfo/m5-dev ___ m5-dev mailing list m5-dev@m5sim.org http://m5sim.org/mailman/listinfo/m5-dev
Re: [m5-dev] changeset in m5: events: Make trace events happen at the right p...
Sorry. It should be event.Event... Does that fix it? On Tue, Feb 24, 2009 at 10:27 PM, Ali Saidi sa...@umich.edu wrote: This changeset broke tracing that starts at a tick other than 0. command line: ./build/ALPHA_FS/m5.opt --trace-flags=Exec --trace- start=4 configs/example/fs.py Traceback (most recent call last): File string, line 1, in module File /tmp/m5/src/python/m5/main.py, line 315, in main e = event.create(trace.enable, Event.Trace_Enable_Pri) NameError: name 'Event' is not defined Ali On Feb 18, 2009, at 2:37 PM, Nathan Binkert wrote: changeset e9f9c0f7e5f0 in /z/repo/m5 details: http://repo.m5sim.org/m5?cmd=changeset;node=e9f9c0f7e5f0 description: events: Make trace events happen at the right priority. Also, while we're at it, remember that priorities are in the Event class and add a disable method to disable tracing. diffstat: 3 files changed, 9 insertions(+), 3 deletions(-) src/python/m5/event.py | 7 +-- src/python/m5/main.py | 2 +- src/python/m5/trace.py | 3 +++ diffs (47 lines): diff -r ce12f1cf978c -r e9f9c0f7e5f0 src/python/m5/event.py --- a/src/python/m5/event.py Tue Feb 17 19:24:46 2009 -0800 +++ b/src/python/m5/event.py Wed Feb 18 10:00:15 2009 -0800 @@ -35,13 +35,16 @@ def create(obj, priority=None): if priority is None: - priority = internal.event.Event.Default_Pri + priority = Event.Default_Pri return PythonEvent(obj, priority) + +# As a reminder, priorities found in sim/eventq.hh are stuck into the +# Event class by swig class Event(PythonEvent): def __init__(self, priority=None): if priority is None: - priority = internal.event.Event.Default_Pri + priority = Event.Default_Pri super(Event, self).__init__(self, priority) class ProgressEvent(Event): diff -r ce12f1cf978c -r e9f9c0f7e5f0 src/python/m5/main.py --- a/src/python/m5/main.py Tue Feb 17 19:24:46 2009 -0800 +++ b/src/python/m5/main.py Wed Feb 18 10:00:15 2009 -0800 @@ -312,7 +312,7 @@ if options.trace_start: check_tracing() - e = event.create(trace.enable) + e = event.create(trace.enable, Event.Trace_Enable_Pri) event.mainq.schedule(e, options.trace_start) else: trace.enable() diff -r ce12f1cf978c -r e9f9c0f7e5f0 src/python/m5/trace.py --- a/src/python/m5/trace.py Tue Feb 17 19:24:46 2009 -0800 +++ b/src/python/m5/trace.py Wed Feb 18 10:00:15 2009 -0800 @@ -32,6 +32,9 @@ from internal.trace import clear, output, set, ignore +def disable(): + internal.trace.cvar.enabled = False + def enable(): internal.trace.cvar.enabled = True ___ m5-dev mailing list m5-dev@m5sim.org http://m5sim.org/mailman/listinfo/m5-dev ___ m5-dev mailing list m5-dev@m5sim.org http://m5sim.org/mailman/listinfo/m5-dev ___ m5-dev mailing list m5-dev@m5sim.org http://m5sim.org/mailman/listinfo/m5-dev
[m5-dev] changeset in m5: events: Make trace events happen at the right p...
changeset e9f9c0f7e5f0 in /z/repo/m5 details: http://repo.m5sim.org/m5?cmd=changeset;node=e9f9c0f7e5f0 description: events: Make trace events happen at the right priority. Also, while we're at it, remember that priorities are in the Event class and add a disable method to disable tracing. diffstat: 3 files changed, 9 insertions(+), 3 deletions(-) src/python/m5/event.py |7 +-- src/python/m5/main.py |2 +- src/python/m5/trace.py |3 +++ diffs (47 lines): diff -r ce12f1cf978c -r e9f9c0f7e5f0 src/python/m5/event.py --- a/src/python/m5/event.pyTue Feb 17 19:24:46 2009 -0800 +++ b/src/python/m5/event.pyWed Feb 18 10:00:15 2009 -0800 @@ -35,13 +35,16 @@ def create(obj, priority=None): if priority is None: -priority = internal.event.Event.Default_Pri +priority = Event.Default_Pri return PythonEvent(obj, priority) + +# As a reminder, priorities found in sim/eventq.hh are stuck into the +# Event class by swig class Event(PythonEvent): def __init__(self, priority=None): if priority is None: -priority = internal.event.Event.Default_Pri +priority = Event.Default_Pri super(Event, self).__init__(self, priority) class ProgressEvent(Event): diff -r ce12f1cf978c -r e9f9c0f7e5f0 src/python/m5/main.py --- a/src/python/m5/main.py Tue Feb 17 19:24:46 2009 -0800 +++ b/src/python/m5/main.py Wed Feb 18 10:00:15 2009 -0800 @@ -312,7 +312,7 @@ if options.trace_start: check_tracing() -e = event.create(trace.enable) +e = event.create(trace.enable, Event.Trace_Enable_Pri) event.mainq.schedule(e, options.trace_start) else: trace.enable() diff -r ce12f1cf978c -r e9f9c0f7e5f0 src/python/m5/trace.py --- a/src/python/m5/trace.pyTue Feb 17 19:24:46 2009 -0800 +++ b/src/python/m5/trace.pyWed Feb 18 10:00:15 2009 -0800 @@ -32,6 +32,9 @@ from internal.trace import clear, output, set, ignore +def disable(): +internal.trace.cvar.enabled = False + def enable(): internal.trace.cvar.enabled = True ___ m5-dev mailing list m5-dev@m5sim.org http://m5sim.org/mailman/listinfo/m5-dev
Re: [m5-dev] changeset in m5: events: Make trace events happen at the right p...
Thanks :-) On Wed, Feb 18, 2009 at 11:37 AM, Nathan Binkert n...@binkert.org wrote: changeset e9f9c0f7e5f0 in /z/repo/m5 details: http://repo.m5sim.org/m5?cmd=changeset;node=e9f9c0f7e5f0 description: events: Make trace events happen at the right priority. Also, while we're at it, remember that priorities are in the Event class and add a disable method to disable tracing. diffstat: 3 files changed, 9 insertions(+), 3 deletions(-) src/python/m5/event.py |7 +-- src/python/m5/main.py |2 +- src/python/m5/trace.py |3 +++ diffs (47 lines): diff -r ce12f1cf978c -r e9f9c0f7e5f0 src/python/m5/event.py --- a/src/python/m5/event.pyTue Feb 17 19:24:46 2009 -0800 +++ b/src/python/m5/event.pyWed Feb 18 10:00:15 2009 -0800 @@ -35,13 +35,16 @@ def create(obj, priority=None): if priority is None: -priority = internal.event.Event.Default_Pri +priority = Event.Default_Pri return PythonEvent(obj, priority) + +# As a reminder, priorities found in sim/eventq.hh are stuck into the +# Event class by swig class Event(PythonEvent): def __init__(self, priority=None): if priority is None: -priority = internal.event.Event.Default_Pri +priority = Event.Default_Pri super(Event, self).__init__(self, priority) class ProgressEvent(Event): diff -r ce12f1cf978c -r e9f9c0f7e5f0 src/python/m5/main.py --- a/src/python/m5/main.py Tue Feb 17 19:24:46 2009 -0800 +++ b/src/python/m5/main.py Wed Feb 18 10:00:15 2009 -0800 @@ -312,7 +312,7 @@ if options.trace_start: check_tracing() -e = event.create(trace.enable) +e = event.create(trace.enable, Event.Trace_Enable_Pri) event.mainq.schedule(e, options.trace_start) else: trace.enable() diff -r ce12f1cf978c -r e9f9c0f7e5f0 src/python/m5/trace.py --- a/src/python/m5/trace.pyTue Feb 17 19:24:46 2009 -0800 +++ b/src/python/m5/trace.pyWed Feb 18 10:00:15 2009 -0800 @@ -32,6 +32,9 @@ from internal.trace import clear, output, set, ignore +def disable(): +internal.trace.cvar.enabled = False + def enable(): internal.trace.cvar.enabled = True ___ m5-dev mailing list m5-dev@m5sim.org http://m5sim.org/mailman/listinfo/m5-dev ___ m5-dev mailing list m5-dev@m5sim.org http://m5sim.org/mailman/listinfo/m5-dev