sanchayanmaity/gem5
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

ruby: Removed if else statement in create_system

Browse source 
Based on Steve's suggestion, the ugly if-elif statement and multiple protocol
module import calls are removed and replaced with exec statements using the
protocol string.
This commit is contained in:
Brad Beckmann 2010-01-29 20:29:33 -08:00
parent 1feae85017
commit c441590b15
1 changed files with 10 additions and 40 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

46
configs/ruby/Ruby.py
View file

@ -32,48 +32,18 @@ from m5.objects import *
from m5.defines import buildEnv
from m5.util import addToPath
import MOESI_hammer
import MESI_CMP_directory
import MOESI_CMP_directory
import MI_example
import MOESI_CMP_token
protocol = buildEnv['PROTOCOL']
exec "import %s" % protocol
def create_system(options, physmem, piobus = None, dma_devices = []):
protocol = buildEnv['PROTOCOL']
if protocol == "MOESI_hammer":
try:
(cpu_sequencers, dir_cntrls, all_cntrls) = \
MOESI_hammer.create_system(options, \
physmem, \
piobus, \
dma_devices)
elif protocol == "MESI_CMP_directory":
(cpu_sequencers, dir_cntrls, all_cntrls) = \
MESI_CMP_directory.create_system(options, \
physmem, \
piobus, \
dma_devices)
elif protocol == "MOESI_CMP_directory":
(cpu_sequencers, dir_cntrls, all_cntrls) = \
MOESI_CMP_directory.create_system(options, \
physmem, \
piobus, \
dma_devices)
elif protocol == "MI_example":
(cpu_sequencers, dir_cntrls, all_cntrls) = \
MI_example.create_system(options, \
physmem, \
piobus, \
dma_devices)
elif protocol == "MOESI_CMP_token":
(cpu_sequencers, dir_cntrls, all_cntrls) = \
MOESI_CMP_token.create_system(options, \
physmem, \
piobus, \
dma_devices)
else:
print "Error: unsupported ruby protocol"
eval("%s.create_system(options, physmem, piobus, dma_devices)" \
% protocol)
except:
print "Error: could not create sytem for ruby protocol %s" % protocol
sys.exit(1)
#