x86: Move miscreg initialization to the ISA class.

The initCPU function was setting a lot of values to zero or other
initial values, but that's something the ISA object can do as part of
its clear() method. This gets rid of a lot of code that was
individually zeroing registers, and also centralizes responsibility
for those registers in the ISA.

Change-Id: Iafcffd3f9329c39f77009b38b1696f91c36c117e
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/24185
Reviewed-by: Gabe Black <gabeblack@google.com>
Maintainer: Gabe Black <gabeblack@google.com>
Tested-by: kokoro <noreply+kokoro@google.com>

1 files changed, 2 insertions, 106 deletions

diff --git a/src/arch/x86/utility.cc b/src/arch/x86/utility.cc
index b430124ec..13019e90e 100644
--- a/src/arch/x86/utility.cc
+++ b/src/arch/x86/utility.cc
@@ -71,7 +71,8 @@ getArgument(ThreadContext *tc, int &number, uint16_t size, bool fp)
     }
 }
 
-void initCPU(ThreadContext *tc, int cpuId)
+void
+initCPU(ThreadContext *tc, int cpuId)
 {
     // This function is essentially performing a reset. The actual INIT
     // interrupt does a subset of this, so we'll piggyback on some of its
@@ -79,109 +80,11 @@ void initCPU(ThreadContext *tc, int cpuId)
     InitInterrupt init(0);
     init.invoke(tc);
 
-    PCState pc = tc->pcState();
-    pc.upc(0);
-    pc.nupc(1);
-    tc->pcState(pc);
-
-    // These next two loops zero internal microcode and implicit registers.
-    // They aren't specified by the ISA but are used internally by M5's
-    // implementation.
-    for (int index = 0; index < NumMicroIntRegs; index++) {
-        tc->setIntReg(INTREG_MICRO(index), 0);
-    }
-
-    for (int index = 0; index < NumImplicitIntRegs; index++) {
-        tc->setIntReg(INTREG_IMPLICIT(index), 0);
-    }
-
     // Set integer register EAX to 0 to indicate that the optional BIST
     // passed. No BIST actually runs, but software may still check this
     // register for errors.
     tc->setIntReg(INTREG_RAX, 0);
 
-    tc->setMiscReg(MISCREG_CR0, 0x0000000060000010ULL);
-    tc->setMiscReg(MISCREG_CR8, 0);
-
-    // TODO initialize x87, 64 bit, and 128 bit media state
-
-    tc->setMiscReg(MISCREG_MTRRCAP, 0x0508);
-    for (int i = 0; i < 8; i++) {
-        tc->setMiscReg(MISCREG_MTRR_PHYS_BASE(i), 0);
-        tc->setMiscReg(MISCREG_MTRR_PHYS_MASK(i), 0);
-    }
-    tc->setMiscReg(MISCREG_MTRR_FIX_64K_00000, 0);
-    tc->setMiscReg(MISCREG_MTRR_FIX_16K_80000, 0);
-    tc->setMiscReg(MISCREG_MTRR_FIX_16K_A0000, 0);
-    tc->setMiscReg(MISCREG_MTRR_FIX_4K_C0000, 0);
-    tc->setMiscReg(MISCREG_MTRR_FIX_4K_C8000, 0);
-    tc->setMiscReg(MISCREG_MTRR_FIX_4K_D0000, 0);
-    tc->setMiscReg(MISCREG_MTRR_FIX_4K_D8000, 0);
-    tc->setMiscReg(MISCREG_MTRR_FIX_4K_E0000, 0);
-    tc->setMiscReg(MISCREG_MTRR_FIX_4K_E8000, 0);
-    tc->setMiscReg(MISCREG_MTRR_FIX_4K_F0000, 0);
-    tc->setMiscReg(MISCREG_MTRR_FIX_4K_F8000, 0);
-
-    tc->setMiscReg(MISCREG_DEF_TYPE, 0);
-
-    tc->setMiscReg(MISCREG_MCG_CAP, 0x104);
-    tc->setMiscReg(MISCREG_MCG_STATUS, 0);
-    tc->setMiscReg(MISCREG_MCG_CTL, 0);
-
-    for (int i = 0; i < 5; i++) {
-        tc->setMiscReg(MISCREG_MC_CTL(i), 0);
-        tc->setMiscReg(MISCREG_MC_STATUS(i), 0);
-        tc->setMiscReg(MISCREG_MC_ADDR(i), 0);
-        tc->setMiscReg(MISCREG_MC_MISC(i), 0);
-    }
-
-    tc->setMiscReg(MISCREG_TSC, 0);
-    tc->setMiscReg(MISCREG_TSC_AUX, 0);
-
-    for (int i = 0; i < 4; i++) {
-        tc->setMiscReg(MISCREG_PERF_EVT_SEL(i), 0);
-        tc->setMiscReg(MISCREG_PERF_EVT_CTR(i), 0);
-    }
-
-    tc->setMiscReg(MISCREG_STAR, 0);
-    tc->setMiscReg(MISCREG_LSTAR, 0);
-    tc->setMiscReg(MISCREG_CSTAR, 0);
-
-    tc->setMiscReg(MISCREG_SF_MASK, 0);
-
-    tc->setMiscReg(MISCREG_KERNEL_GS_BASE, 0);
-
-    tc->setMiscReg(MISCREG_SYSENTER_CS, 0);
-    tc->setMiscReg(MISCREG_SYSENTER_ESP, 0);
-    tc->setMiscReg(MISCREG_SYSENTER_EIP, 0);
-
-    tc->setMiscReg(MISCREG_PAT, 0x0007040600070406ULL);
-
-    tc->setMiscReg(MISCREG_SYSCFG, 0x20601);
-
-    tc->setMiscReg(MISCREG_IORR_BASE0, 0);
-    tc->setMiscReg(MISCREG_IORR_BASE1, 0);
-
-    tc->setMiscReg(MISCREG_IORR_MASK0, 0);
-    tc->setMiscReg(MISCREG_IORR_MASK1, 0);
-
-    tc->setMiscReg(MISCREG_TOP_MEM, 0x4000000);
-    tc->setMiscReg(MISCREG_TOP_MEM2, 0x0);
-
-    tc->setMiscReg(MISCREG_DEBUG_CTL_MSR, 0);
-    tc->setMiscReg(MISCREG_LAST_BRANCH_FROM_IP, 0);
-    tc->setMiscReg(MISCREG_LAST_BRANCH_TO_IP, 0);
-    tc->setMiscReg(MISCREG_LAST_EXCEPTION_FROM_IP, 0);
-    tc->setMiscReg(MISCREG_LAST_EXCEPTION_TO_IP, 0);
-
-    // Invalidate the caches (this should already be done for us)
-
-    LocalApicBase lApicBase = 0;
-    lApicBase.base = 0xFEE00000 >> 12;
-    lApicBase.enable = 1;
-    lApicBase.bsp = (cpuId == 0);
-    tc->setMiscReg(MISCREG_APIC_BASE, lApicBase);
-
     Interrupts * interrupts = dynamic_cast<Interrupts *>(
             tc->getCpuPtr()->getInterruptController(0));
     assert(interrupts);
@@ -189,13 +92,6 @@ void initCPU(ThreadContext *tc, int cpuId)
     interrupts->setRegNoEffect(APIC_ID, cpuId << 24);
 
     interrupts->setRegNoEffect(APIC_VERSION, (5 << 16) | 0x14);
-
-    // TODO Set the SMRAM base address (SMBASE) to 0x00030000
-
-    tc->setMiscReg(MISCREG_VM_CR, 0);
-    tc->setMiscReg(MISCREG_IGNNE, 0);
-    tc->setMiscReg(MISCREG_SMM_CTL, 0);
-    tc->setMiscReg(MISCREG_VM_HSAVE_PA, 0);
 }
 
 void startupCPU(ThreadContext *tc, int cpuId)