From 4f4a710457af4a9c8e72632995b55f106110ac9c Mon Sep 17 00:00:00 2001 From: Nilay Vaish Date: Thu, 5 Apr 2012 11:09:19 -0500 Subject: [PATCH] Config: corrects the way Ruby attaches to the DMA ports With recent changes to the memory system, a port cannot be assigned a peer port twice. While making use of the Ruby memory system in FS mode, DMA ports were assigned peer twice, once for the classic memory system and once for the Ruby memory system. This patch removes this double assignment of peer ports. --- configs/common/FSConfig.py | 13 +++++-------- configs/example/ruby_fs.py | 2 +- configs/example/ruby_mem_test.py | 8 ++++++-- configs/ruby/MESI_CMP_directory.py | 10 +++------- configs/ruby/MI_example.py | 11 +++-------- configs/ruby/MOESI_CMP_directory.py | 10 +++------- configs/ruby/MOESI_CMP_token.py | 10 +++------- configs/ruby/MOESI_hammer.py | 9 +++------ configs/ruby/Network_test.py | 4 ++-- configs/ruby/Ruby.py | 5 ++--- src/dev/x86/Pc.py | 4 ++-- src/dev/x86/SouthBridge.py | 5 +++-- 12 files changed, 36 insertions(+), 55 deletions(-) diff --git a/configs/common/FSConfig.py b/configs/common/FSConfig.py index 4c2d0a66c..de2d9181d 100644 --- a/configs/common/FSConfig.py +++ b/configs/common/FSConfig.py @@ -134,16 +134,14 @@ def makeLinuxAlphaRubySystem(mem_mode, mdesc = None): self.tsunami.attachIO(self.piobus) self.tsunami.ide.pio = self.piobus.master self.tsunami.ide.config = self.piobus.master - self.tsunami.ide.dma = self.piobus.slave self.tsunami.ethernet.pio = self.piobus.master self.tsunami.ethernet.config = self.piobus.master - self.tsunami.ethernet.dma = self.piobus.slave # # Store the dma devices for later connection to dma ruby ports. # Append an underscore to dma_devices to avoid the SimObjectVector check. # - self._dma_devices = [self.tsunami.ide, self.tsunami.ethernet] + self._dma_ports = [self.tsunami.ide.dma, self.tsunami.ethernet.dma] self.simple_disk = SimpleDisk(disk=RawDiskImage(image_file = mdesc.disk(), read_only = True)) @@ -408,8 +406,10 @@ def connectX86RubySystem(x86_sys): # the piobus a direct connection to physical memory # x86_sys.piobus.master = x86_sys.physmem.port - - x86_sys.pc.attachIO(x86_sys.piobus) + # add the ide to the list of dma devices that later need to attach to + # dma controllers + x86_sys._dma_ports = [x86_sys.pc.south_bridge.ide.dma] + x86_sys.pc.attachIO(x86_sys.piobus, x86_sys._dma_ports) def makeX86System(mem_mode, numCPUs = 1, mdesc = None, self = None, Ruby = False): @@ -432,9 +432,6 @@ def makeX86System(mem_mode, numCPUs = 1, mdesc = None, self = None, Ruby = False # Create and connect the busses required by each memory system if Ruby: connectX86RubySystem(self) - # add the ide to the list of dma devices that later need to attach to - # dma controllers - self._dma_devices = [self.pc.south_bridge.ide] else: connectX86ClassicSystem(self, numCPUs) diff --git a/configs/example/ruby_fs.py b/configs/example/ruby_fs.py index b5ce643b9..8f02b8cfd 100644 --- a/configs/example/ruby_fs.py +++ b/configs/example/ruby_fs.py @@ -96,7 +96,7 @@ if options.script is not None: system.readfile = options.script system.cpu = [CPUClass(cpu_id=i) for i in xrange(options.num_cpus)] -Ruby.create_system(options, system, system.piobus, system._dma_devices) +Ruby.create_system(options, system, system.piobus, system._dma_ports) for (i, cpu) in enumerate(system.cpu): # diff --git a/configs/example/ruby_mem_test.py b/configs/example/ruby_mem_test.py index 3fec09c47..4b0948365 100644 --- a/configs/example/ruby_mem_test.py +++ b/configs/example/ruby_mem_test.py @@ -115,13 +115,17 @@ if options.num_dmas > 0: percent_functional = 0, percent_uncacheable = 0, progress_interval = options.progress, - warn_on_failure = options.warn_on_failure) \ + suppress_func_warnings = + not options.suppress_func_warnings) \ for i in xrange(options.num_dmas) ] system.dma_devices = dmas else: dmas = [] -Ruby.create_system(options, system, dma_devices = dmas) +dma_ports = [] +for (i, dma) in enumerate(dmas): + dma_ports.append(dma.test) +Ruby.create_system(options, system, dma_ports = dma_ports) # # The tester is most effective when randomization is turned on and diff --git a/configs/ruby/MESI_CMP_directory.py b/configs/ruby/MESI_CMP_directory.py index 28aba4a1b..4d490705b 100644 --- a/configs/ruby/MESI_CMP_directory.py +++ b/configs/ruby/MESI_CMP_directory.py @@ -47,7 +47,7 @@ class L2Cache(RubyCache): def define_options(parser): return -def create_system(options, system, piobus, dma_devices, ruby_system): +def create_system(options, system, piobus, dma_ports, ruby_system): if buildEnv['PROTOCOL'] != 'MESI_CMP_directory': panic("This script requires the MESI_CMP_directory protocol to be built.") @@ -162,7 +162,7 @@ def create_system(options, system, piobus, dma_devices, ruby_system): cntrl_count += 1 - for i, dma_device in enumerate(dma_devices): + for i, dma_port in enumerate(dma_ports): # # Create the Ruby objects associated with the dma controller # @@ -175,12 +175,8 @@ def create_system(options, system, piobus, dma_devices, ruby_system): ruby_system = ruby_system) exec("system.dma_cntrl%d = dma_cntrl" % i) - if dma_device.type == 'MemTest': - exec("system.dma_cntrl%d.dma_sequencer.slave = dma_device.test" % i) - else: - exec("system.dma_cntrl%d.dma_sequencer.slave = dma_device.dma" % i) + exec("system.dma_cntrl%d.dma_sequencer.slave = dma_port" % i) dma_cntrl_nodes.append(dma_cntrl) - cntrl_count += 1 all_cntrls = l1_cntrl_nodes + \ diff --git a/configs/ruby/MI_example.py b/configs/ruby/MI_example.py index 29ed23b67..1285c4b1d 100644 --- a/configs/ruby/MI_example.py +++ b/configs/ruby/MI_example.py @@ -41,7 +41,7 @@ class Cache(RubyCache): def define_options(parser): return -def create_system(options, system, piobus, dma_devices, ruby_system): +def create_system(options, system, piobus, dma_ports, ruby_system): if buildEnv['PROTOCOL'] != 'MI_example': panic("This script requires the MI_example protocol to be built.") @@ -135,7 +135,7 @@ def create_system(options, system, piobus, dma_devices, ruby_system): cntrl_count += 1 - for i, dma_device in enumerate(dma_devices): + for i, dma_port in enumerate(dma_ports): # # Create the Ruby objects associated with the dma controller # @@ -148,13 +148,8 @@ def create_system(options, system, piobus, dma_devices, ruby_system): ruby_system = ruby_system) exec("system.dma_cntrl%d = dma_cntrl" % i) - if dma_device.type == 'MemTest': - exec("system.dma_cntrl%d.dma_sequencer.slave = dma_device.test" % i) - else: - exec("system.dma_cntrl%d.dma_sequencer.slave = dma_device.dma" % i) - dma_cntrl.dma_sequencer.slave = dma_device.dma + exec("system.dma_cntrl%d.dma_sequencer.slave = dma_port" % i) dma_cntrl_nodes.append(dma_cntrl) - cntrl_count += 1 all_cntrls = l1_cntrl_nodes + dir_cntrl_nodes + dma_cntrl_nodes diff --git a/configs/ruby/MOESI_CMP_directory.py b/configs/ruby/MOESI_CMP_directory.py index 753a3a37a..5bc23ea72 100644 --- a/configs/ruby/MOESI_CMP_directory.py +++ b/configs/ruby/MOESI_CMP_directory.py @@ -47,7 +47,7 @@ class L2Cache(RubyCache): def define_options(parser): return -def create_system(options, system, piobus, dma_devices, ruby_system): +def create_system(options, system, piobus, dma_ports, ruby_system): if buildEnv['PROTOCOL'] != 'MOESI_CMP_directory': panic("This script requires the MOESI_CMP_directory protocol to be built.") @@ -159,7 +159,7 @@ def create_system(options, system, piobus, dma_devices, ruby_system): cntrl_count += 1 - for i, dma_device in enumerate(dma_devices): + for i, dma_port in enumerate(dma_ports): # # Create the Ruby objects associated with the dma controller # @@ -172,12 +172,8 @@ def create_system(options, system, piobus, dma_devices, ruby_system): ruby_system = ruby_system) exec("system.dma_cntrl%d = dma_cntrl" % i) - if dma_device.type == 'MemTest': - exec("system.dma_cntrl%d.dma_sequencer.slave = dma_device.test" % i) - else: - exec("system.dma_cntrl%d.dma_sequencer.slave = dma_device.dma" % i) + exec("system.dma_cntrl%d.dma_sequencer.slave = dma_port" % i) dma_cntrl_nodes.append(dma_cntrl) - cntrl_count += 1 all_cntrls = l1_cntrl_nodes + \ diff --git a/configs/ruby/MOESI_CMP_token.py b/configs/ruby/MOESI_CMP_token.py index 064d6dd14..f621df767 100644 --- a/configs/ruby/MOESI_CMP_token.py +++ b/configs/ruby/MOESI_CMP_token.py @@ -54,7 +54,7 @@ def define_options(parser): parser.add_option("--allow-atomic-migration", action="store_true", help="allow migratory sharing for atomic only accessed blocks") -def create_system(options, system, piobus, dma_devices, ruby_system): +def create_system(options, system, piobus, dma_ports, ruby_system): if buildEnv['PROTOCOL'] != 'MOESI_CMP_token': panic("This script requires the MOESI_CMP_token protocol to be built.") @@ -183,7 +183,7 @@ def create_system(options, system, piobus, dma_devices, ruby_system): cntrl_count += 1 - for i, dma_device in enumerate(dma_devices): + for i, dma_port in enumerate(dma_ports): # # Create the Ruby objects associated with the dma controller # @@ -196,12 +196,8 @@ def create_system(options, system, piobus, dma_devices, ruby_system): ruby_system = ruby_system) exec("system.dma_cntrl%d = dma_cntrl" % i) - if dma_device.type == 'MemTest': - exec("system.dma_cntrl%d.dma_sequencer.slave = dma_device.test" % i) - else: - exec("system.dma_cntrl%d.dma_sequencer.slave = dma_device.dma" % i) + exec("system.dma_cntrl%d.dma_sequencer.slave = dma_port" % i) dma_cntrl_nodes.append(dma_cntrl) - cntrl_count += 1 all_cntrls = l1_cntrl_nodes + \ diff --git a/configs/ruby/MOESI_hammer.py b/configs/ruby/MOESI_hammer.py index dc9aa3392..3f9d31482 100644 --- a/configs/ruby/MOESI_hammer.py +++ b/configs/ruby/MOESI_hammer.py @@ -58,7 +58,7 @@ def define_options(parser): parser.add_option("--dir-on", action="store_true", help="Hammer: enable Full-bit Directory") -def create_system(options, system, piobus, dma_devices, ruby_system): +def create_system(options, system, piobus, dma_ports, ruby_system): if buildEnv['PROTOCOL'] != 'MOESI_hammer': panic("This script requires the MOESI_hammer protocol to be built.") @@ -195,7 +195,7 @@ def create_system(options, system, piobus, dma_devices, ruby_system): cntrl_count += 1 - for i, dma_device in enumerate(dma_devices): + for i, dma_port in enumerate(dma_ports): # # Create the Ruby objects associated with the dma controller # @@ -208,10 +208,7 @@ def create_system(options, system, piobus, dma_devices, ruby_system): ruby_system = ruby_system) exec("system.dma_cntrl%d = dma_cntrl" % i) - if dma_device.type == 'MemTest': - exec("system.dma_cntrl%d.dma_sequencer.slave = dma_device.test" % i) - else: - exec("system.dma_cntrl%d.dma_sequencer.slave = dma_device.dma" % i) + exec("system.dma_cntrl%d.dma_sequencer.slave = dma_port" % i) dma_cntrl_nodes.append(dma_cntrl) if options.recycle_latency: diff --git a/configs/ruby/Network_test.py b/configs/ruby/Network_test.py index 5c1936b51..ff07c6b4d 100644 --- a/configs/ruby/Network_test.py +++ b/configs/ruby/Network_test.py @@ -41,7 +41,7 @@ class Cache(RubyCache): def define_options(parser): return -def create_system(options, system, piobus, dma_devices, ruby_system): +def create_system(options, system, piobus, dma_ports, ruby_system): if buildEnv['PROTOCOL'] != 'Network_test': panic("This script requires the Network_test protocol to be built.") @@ -52,7 +52,7 @@ def create_system(options, system, piobus, dma_devices, ruby_system): # The Garnet tester protocol does not support fs nor dma # assert(piobus == None) - assert(dma_devices == []) + assert(dma_ports == []) # # The ruby network creation expects the list of nodes in the system to be diff --git a/configs/ruby/Ruby.py b/configs/ruby/Ruby.py index c9788b0a0..1911a1eb7 100644 --- a/configs/ruby/Ruby.py +++ b/configs/ruby/Ruby.py @@ -76,7 +76,7 @@ def define_options(parser): exec "import %s" % protocol eval("%s.define_options(parser)" % protocol) -def create_system(options, system, piobus = None, dma_devices = []): +def create_system(options, system, piobus = None, dma_ports = []): system.ruby = RubySystem(clock = options.clock, stats_filename = options.ruby_stats, @@ -87,8 +87,7 @@ def create_system(options, system, piobus = None, dma_devices = []): exec "import %s" % protocol try: (cpu_sequencers, dir_cntrls, all_cntrls) = \ - eval("%s.create_system(options, system, piobus, \ - dma_devices, ruby)" \ + eval("%s.create_system(options, system, piobus, dma_ports, ruby)" % protocol) except: print "Error: could not create sytem for ruby protocol %s" % protocol diff --git a/src/dev/x86/Pc.py b/src/dev/x86/Pc.py index 5b7d0864e..91292788c 100644 --- a/src/dev/x86/Pc.py +++ b/src/dev/x86/Pc.py @@ -69,8 +69,8 @@ class Pc(Platform): # A device to catch accesses to the non-existant floppy controller. fake_floppy = IsaFake(pio_addr=x86IOAddress(0x3f2), pio_size=2) - def attachIO(self, bus): - self.south_bridge.attachIO(bus) + def attachIO(self, bus, dma_ports = []): + self.south_bridge.attachIO(bus, dma_ports) self.i_dont_exist.pio = bus.master self.behind_pci.pio = bus.master self.com_1.pio = bus.master diff --git a/src/dev/x86/SouthBridge.py b/src/dev/x86/SouthBridge.py index 9f7070e96..87f4c3798 100644 --- a/src/dev/x86/SouthBridge.py +++ b/src/dev/x86/SouthBridge.py @@ -85,7 +85,7 @@ class SouthBridge(SimObject): ide.InterruptLine = 14 ide.InterruptPin = 1 - def attachIO(self, bus): + def attachIO(self, bus, dma_ports): # Route interupt signals self.int_lines = \ [X86IntLine(source=self.pic1.output, sink=self.io_apic.pin(0)), @@ -106,7 +106,8 @@ class SouthBridge(SimObject): self.dma1.pio = bus.master self.ide.pio = bus.master self.ide.config = bus.master - self.ide.dma = bus.slave + if dma_ports.count(self.ide.dma) == 0: + self.ide.dma = bus.slave self.keyboard.pio = bus.master self.pic1.pio = bus.master self.pic2.pio = bus.master