----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/1001/#review2040 -----------------------------------------------------------
Ship it! seems fine to me. Steve? - Ali Saidi On Jan. 18, 2012, 2:32 a.m., Andreas Hansson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/1001/ > ----------------------------------------------------------- > > (Updated Jan. 18, 2012, 2:32 a.m.) > > > Review request for Default. > > > Description > ------- > > MEM: Introduce the master/slave port roles in the Python classes > > This patch classifies all ports in Python as either Master or Slave > and enfores a binding of master to slave. Conceptually, a master (such > as a CPU or DMA port) issues requests, and receives responses, and > conversely, a slave (such as a memory or a PIO device) receives > requests and sends back responses. Currently there is no > differentiation between coherent and non-coherent masters and slaves. > > The classification as master/slave also involves splitting the dual > role port of the bus into a master and slave port and updating all the > system assembly scripts to use the appropriate port. Similarly, the > interrupt devices have to have their int_port split into a master and > slave port. The intdev and its children have minimal changes to > facilitate the extra port. > > Note that this patch does not enfore any port typing in the C++ world, > it merely ensures that the Python objects have a notion of the port > roles and are connected in an appropriate manner. This check is > carried when two ports are connected, e.g. bus.master = > memory.port. The following patches will make use of the > classifications and specialise the C++ ports into masters and slaves. > > > Diffs > ----- > > configs/common/CacheConfig.py 03e09db82c80 > configs/common/FSConfig.py 03e09db82c80 > configs/example/fs.py 03e09db82c80 > configs/example/ruby_fs.py 03e09db82c80 > configs/example/se.py 03e09db82c80 > src/arch/arm/ArmTLB.py 03e09db82c80 > src/arch/x86/X86LocalApic.py 03e09db82c80 > src/arch/x86/X86TLB.py 03e09db82c80 > src/arch/x86/interrupts.hh 03e09db82c80 > src/cpu/BaseCPU.py 03e09db82c80 > src/cpu/simple/AtomicSimpleCPU.py 03e09db82c80 > src/cpu/testers/directedtest/RubyDirectedTester.py 03e09db82c80 > src/cpu/testers/memtest/MemTest.py 03e09db82c80 > src/cpu/testers/networktest/NetworkTest.py 03e09db82c80 > src/cpu/testers/rubytest/RubyTester.py 03e09db82c80 > src/dev/Device.py 03e09db82c80 > src/dev/Ethernet.py 03e09db82c80 > src/dev/Pci.py 03e09db82c80 > src/dev/alpha/Tsunami.py 03e09db82c80 > src/dev/arm/RealView.py 03e09db82c80 > src/dev/x86/I82094AA.py 03e09db82c80 > src/dev/x86/Pc.py 03e09db82c80 > src/dev/x86/SouthBridge.py 03e09db82c80 > src/dev/x86/i82094aa.hh 03e09db82c80 > src/dev/x86/intdev.hh 03e09db82c80 > src/mem/Bridge.py 03e09db82c80 > src/mem/Bus.py 03e09db82c80 > src/mem/PhysicalMemory.py 03e09db82c80 > src/mem/cache/BaseCache.py 03e09db82c80 > src/mem/ruby/system/Sequencer.py 03e09db82c80 > src/python/m5/params.py 03e09db82c80 > src/sim/System.py 03e09db82c80 > tests/configs/inorder-timing.py 03e09db82c80 > tests/configs/memtest.py 03e09db82c80 > tests/configs/o3-timing-mp-ruby.py 03e09db82c80 > tests/configs/o3-timing-mp.py 03e09db82c80 > tests/configs/o3-timing-ruby.py 03e09db82c80 > tests/configs/o3-timing.py 03e09db82c80 > tests/configs/pc-o3-timing.py 03e09db82c80 > tests/configs/pc-simple-atomic.py 03e09db82c80 > tests/configs/pc-simple-timing.py 03e09db82c80 > tests/configs/realview-o3-dual.py 03e09db82c80 > tests/configs/realview-o3.py 03e09db82c80 > tests/configs/realview-simple-atomic-dual.py 03e09db82c80 > tests/configs/realview-simple-atomic.py 03e09db82c80 > tests/configs/realview-simple-timing-dual.py 03e09db82c80 > tests/configs/realview-simple-timing.py 03e09db82c80 > tests/configs/simple-atomic-mp-ruby.py 03e09db82c80 > tests/configs/simple-atomic-mp.py 03e09db82c80 > tests/configs/simple-atomic.py 03e09db82c80 > tests/configs/simple-timing-mp.py 03e09db82c80 > tests/configs/simple-timing.py 03e09db82c80 > tests/configs/tsunami-inorder.py 03e09db82c80 > tests/configs/tsunami-o3-dual.py 03e09db82c80 > tests/configs/tsunami-o3.py 03e09db82c80 > tests/configs/tsunami-simple-atomic-dual.py 03e09db82c80 > tests/configs/tsunami-simple-atomic.py 03e09db82c80 > tests/configs/tsunami-simple-timing-dual.py 03e09db82c80 > tests/configs/tsunami-simple-timing.py 03e09db82c80 > tests/configs/twosys-tsunami-simple-atomic.py 03e09db82c80 > > Diff: http://reviews.gem5.org/r/1001/diff/diff > > > Testing > ------- > > util/regress all passing (disregarding t1000 and eio) > > > Thanks, > > Andreas Hansson > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
