changeset 50ce4deacda9 in /z/repo/gem5
details: http://repo.gem5.org/gem5?cmd=changeset;node=50ce4deacda9
description:
        x86: Fix switching of CPUs
        This patch prevents creation of interrupt controller for
        cpus that will be switched in later

diffstat:

 configs/common/CacheConfig.py |   1 +
 configs/example/fs.py         |   1 +
 src/cpu/BaseCPU.py            |  32 +++++++++++++++++++++++++-------
 src/cpu/o3/cpu.cc             |   2 +-
 4 files changed, 28 insertions(+), 8 deletions(-)

diffs (103 lines):

diff -r dedd8be81731 -r 50ce4deacda9 configs/common/CacheConfig.py
--- a/configs/common/CacheConfig.py     Thu Mar 01 11:36:59 2012 -0600
+++ b/configs/common/CacheConfig.py     Thu Mar 01 11:37:02 2012 -0600
@@ -70,6 +70,7 @@
                                                       PageTableWalkerCache())
             else:
                 system.cpu[i].addPrivateSplitL1Caches(icache, dcache)
+        system.cpu[i].createInterruptController()
         if options.l2cache:
             system.cpu[i].connectAllPorts(system.tol2bus, system.membus)
         else:
diff -r dedd8be81731 -r 50ce4deacda9 configs/example/fs.py
--- a/configs/example/fs.py     Thu Mar 01 11:36:59 2012 -0600
+++ b/configs/example/fs.py     Thu Mar 01 11:37:02 2012 -0600
@@ -188,6 +188,7 @@
         drive_sys = makeArmSystem(drive_mem_mode, options.machine_type, bm[1])
 
     drive_sys.cpu = DriveCPUClass(cpu_id=0)
+    drive_sys.cpu.createInterruptController()
     drive_sys.cpu.connectAllPorts(drive_sys.membus)
     if options.fastmem:
         drive_sys.cpu.physmem_port = drive_sys.physmem.port
diff -r dedd8be81731 -r 50ce4deacda9 src/cpu/BaseCPU.py
--- a/src/cpu/BaseCPU.py        Thu Mar 01 11:36:59 2012 -0600
+++ b/src/cpu/BaseCPU.py        Thu Mar 01 11:37:02 2012 -0600
@@ -100,33 +100,32 @@
         dtb = Param.SparcTLB(SparcTLB(), "Data TLB")
         itb = Param.SparcTLB(SparcTLB(), "Instruction TLB")
         interrupts = Param.SparcInterrupts(
-                SparcInterrupts(), "Interrupt Controller")
+                NULL, "Interrupt Controller")
     elif buildEnv['TARGET_ISA'] == 'alpha':
         dtb = Param.AlphaTLB(AlphaDTB(), "Data TLB")
         itb = Param.AlphaTLB(AlphaITB(), "Instruction TLB")
         interrupts = Param.AlphaInterrupts(
-                AlphaInterrupts(), "Interrupt Controller")
+                NULL, "Interrupt Controller")
     elif buildEnv['TARGET_ISA'] == 'x86':
         dtb = Param.X86TLB(X86TLB(), "Data TLB")
         itb = Param.X86TLB(X86TLB(), "Instruction TLB")
-        _localApic = X86LocalApic(pio_addr=0x2000000000000000)
-        interrupts = Param.X86LocalApic(_localApic, "Interrupt Controller")
+        interrupts = Param.X86LocalApic(NULL, "Interrupt Controller")
     elif buildEnv['TARGET_ISA'] == 'mips':
         dtb = Param.MipsTLB(MipsTLB(), "Data TLB")
         itb = Param.MipsTLB(MipsTLB(), "Instruction TLB")
         interrupts = Param.MipsInterrupts(
-                MipsInterrupts(), "Interrupt Controller")
+                NULL, "Interrupt Controller")
     elif buildEnv['TARGET_ISA'] == 'arm':
         dtb = Param.ArmTLB(ArmTLB(), "Data TLB")
         itb = Param.ArmTLB(ArmTLB(), "Instruction TLB")
         interrupts = Param.ArmInterrupts(
-                ArmInterrupts(), "Interrupt Controller")
+                NULL, "Interrupt Controller")
     elif buildEnv['TARGET_ISA'] == 'power':
         UnifiedTLB = Param.Bool(True, "Is this a Unified TLB?")
         dtb = Param.PowerTLB(PowerTLB(), "Data TLB")
         itb = Param.PowerTLB(PowerTLB(), "Instruction TLB")
         interrupts = Param.PowerInterrupts(
-                PowerInterrupts(), "Interrupt Controller")
+                NULL, "Interrupt Controller")
     else:
         print "Don't know what TLB to use for ISA %s" % \
             buildEnv['TARGET_ISA']
@@ -164,6 +163,25 @@
         _uncached_slave_ports += ["interrupts.pio", "interrupts.int_slave"]
         _uncached_master_ports += ["interrupts.int_master"]
 
+    def createInterruptController(self):
+        if buildEnv['TARGET_ISA'] == 'sparc':
+            self.interrupts = SparcInterrupts()
+        elif buildEnv['TARGET_ISA'] == 'alpha':
+            self.interrupts = AlphaInterrupts()
+        elif buildEnv['TARGET_ISA'] == 'x86':
+            _localApic = X86LocalApic(pio_addr=0x2000000000000000)
+            self.interrupts = _localApic
+        elif buildEnv['TARGET_ISA'] == 'mips':
+            self.interrupts = MipsInterrupts()
+        elif buildEnv['TARGET_ISA'] == 'arm':
+            self.interrupts = ArmInterrupts()
+        elif buildEnv['TARGET_ISA'] == 'power':
+            self.interrupts = PowerInterrupts()
+        else:
+            print "Don't know what Interrupt Controller to use for ISA %s" % \
+                buildEnv['TARGET_ISA']
+            sys.exit(1)
+
     def connectCachedPorts(self, bus):
         for p in self._cached_ports:
             exec('self.%s = bus.slave' % p)
diff -r dedd8be81731 -r 50ce4deacda9 src/cpu/o3/cpu.cc
--- a/src/cpu/o3/cpu.cc Thu Mar 01 11:36:59 2012 -0600
+++ b/src/cpu/o3/cpu.cc Thu Mar 01 11:37:02 2012 -0600
@@ -653,7 +653,7 @@
     if (icachePort.isConnected())
         fetch.setIcache();
 
-    if (FullSystem) {
+    if (FullSystem && !params()->defer_registration) {
         for (ThreadID tid = 0; tid < numThreads; tid++) {
             ThreadContext *src_tc = threadContexts[tid];
             TheISA::initCPU(src_tc, src_tc->contextId());
_______________________________________________
gem5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/gem5-dev

Reply via email to