----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/1001/ -----------------------------------------------------------
Review request for Default. Summary ------- 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.m5sim.org/r/1001/diff Testing ------- util/regress all passing (disregarding t1000 and eio) Thanks, Andreas _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
