changeset b35ef789e6f6 in /z/repo/m5
details: http://repo.m5sim.org/m5?cmd=changeset;node=b35ef789e6f6
description:
        Statetrace: Make sure the current state is loaded to print the initial 
stack frame.
        The early call to child->step() was removed earlier because it confused 
the
        new differences-only protocol ARM sendState() was using. It's necessary 
that
        that gets called at least once before attempting to print the initial 
stack
        frame, though, because otherwise statetrace doesn't know what the stack
        pointer is. By putting the first call to child->step() in a common 
spot, both
        needs are met.

diffstat:

1 file changed, 1 insertion(+), 1 deletion(-)
util/statetrace/statetrace.cc |    2 +-

diffs (19 lines):

diff -r 1aa0b4673699 -r b35ef789e6f6 util/statetrace/statetrace.cc
--- a/util/statetrace/statetrace.cc     Wed Jul 29 00:18:26 2009 -0700
+++ b/util/statetrace/statetrace.cc     Wed Jul 29 00:35:49 2009 -0700
@@ -123,6 +123,7 @@
         cerr << "Couldn't start target program" << endl;
         return 1;
     }
+    child->step();
     if(printInitial)
     {
         child->outputStartState(cout);
@@ -157,7 +158,6 @@
             cerr << "Couldn't connect to server! " << strerror(errno) << endl;
             return 1;
         }
-        child->step();
         while(child->isTracing())
         {
                 if(!child->sendState(sock))
_______________________________________________
m5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/m5-dev

Reply via email to