SPARC: Tighten up the clone system call and SPARCs copyRegs.
authorGabe Black <gblack@eecs.umich.edu>
Sat, 25 Apr 2009 06:11:21 +0000 (23:11 -0700)
committerGabe Black <gblack@eecs.umich.edu>
Sat, 25 Apr 2009 06:11:21 +0000 (23:11 -0700)
src/arch/sparc/regfile.cc
src/sim/syscall_emul.cc

index 4efac3d4e7c6f5bce90f3fde2893d35dda5f16ab..a88c6c931f587f6ee5e5acabcab8f43e2d376db4 100644 (file)
@@ -359,19 +359,20 @@ void SparcISA::copyRegs(ThreadContext *src, ThreadContext *dest)
     for (int x = 0; x < MaxGL; ++x) {
         src->setMiscRegNoEffect(MISCREG_GL, x);
         dest->setMiscRegNoEffect(MISCREG_GL, x);
-        for (int y = 0; y < 8; y++)
+        // Skip %g0 which is always zero.
+        for (int y = 1; y < 8; y++)
             dest->setIntReg(y, src->readIntReg(y));
     }
-    //Locals/Ins/Outs
+    //Locals and ins. Outs are all also ins.
     for (int x = 0; x < NWindows; ++x) {
          src->setMiscRegNoEffect(MISCREG_CWP, x);
          dest->setMiscRegNoEffect(MISCREG_CWP, x);
-         for (int y = 8; y < 32; y++)
+         for (int y = 16; y < 32; y++)
              dest->setIntReg(y, src->readIntReg(y));
     }
-    //MicroIntRegs
-    for (int y = 0; y < NumMicroIntRegs; ++y)
-        dest->setIntReg(y+32, src->readIntReg(y+32));
+    //Microcode reg and pseudo int regs (misc regs in the integer regfile).
+    for (int y = NumIntArchRegs; y < NumIntArchRegs + NumMicroIntRegs; ++y)
+        dest->setIntReg(y, src->readIntReg(y));
 
     //Restore src's GL, CWP
     src->setMiscRegNoEffect(MISCREG_GL, old_gl);
@@ -379,7 +380,7 @@ void SparcISA::copyRegs(ThreadContext *src, ThreadContext *dest)
 
 
     // Then loop through the floating point registers.
-    for (int i = 0; i < SparcISA::NumFloatRegs; ++i) {
+    for (int i = 0; i < SparcISA::NumFloatArchRegs; ++i) {
         dest->setFloatRegBits(i, src->readFloatRegBits(i));
     }
 
index 108d750636cda7803d7a030a65e2023584adb636..941c56530706494afed242132b70b7bebbf2e30f 100644 (file)
@@ -723,11 +723,7 @@ cloneFunc(SyscallDesc *desc, int callnum, LiveProcess *process,
 
         ctc->setPC(tc->readNextPC());
         ctc->setNextPC(tc->readNextPC() + sizeof(TheISA::MachInst));
-
-        // In SPARC, need NNPC too...
-        #if THE_ISA == SPARC_ISA
-            ctc->setNextNPC(tc->readNextNPC() + sizeof(TheISA::MachInst));
-        #endif
+        ctc->setNextNPC(tc->readNextNPC() + sizeof(TheISA::MachInst));
 
         ctc->activate();