----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/1273/ -----------------------------------------------------------
Review request for Default. Description ------- Changeset 9081:944fe8294fa3 --------------------------- Port: Add protocol-agnostic ports in the port hierarchy This patch adds an additional level of ports in the inheritance hierarchy, separating out the protocol-specific and protocl-agnostic parts. All the functionality related to the binding of ports is now confined to use BaseMaster/BaseSlavePorts, and all the protocol-specific parts stay in the Master/SlavePort. In the future it will be possible to add other protocol-specific implementations. The functions used in the binding of ports, i.e. getMaster/SlavePort now use the base classes, and the index parameter is updated to use the PortID typedef with the symbolic InvalidPortID as the default. Diffs ----- src/arch/arm/table_walker.hh fa77985a87c6 src/arch/arm/table_walker.cc fa77985a87c6 src/arch/arm/tlb.hh fa77985a87c6 src/arch/arm/tlb.cc fa77985a87c6 src/arch/x86/interrupts.hh fa77985a87c6 src/arch/x86/pagetable_walker.hh fa77985a87c6 src/arch/x86/pagetable_walker.cc fa77985a87c6 src/arch/x86/tlb.hh fa77985a87c6 src/arch/x86/tlb.cc fa77985a87c6 src/cpu/base.hh fa77985a87c6 src/cpu/base.cc fa77985a87c6 src/cpu/testers/directedtest/RubyDirectedTester.hh fa77985a87c6 src/cpu/testers/directedtest/RubyDirectedTester.cc fa77985a87c6 src/cpu/testers/memtest/memtest.hh fa77985a87c6 src/cpu/testers/memtest/memtest.cc fa77985a87c6 src/cpu/testers/networktest/networktest.hh fa77985a87c6 src/cpu/testers/networktest/networktest.cc fa77985a87c6 src/cpu/testers/rubytest/RubyTester.hh fa77985a87c6 src/cpu/testers/rubytest/RubyTester.cc fa77985a87c6 src/dev/copy_engine.hh fa77985a87c6 src/dev/copy_engine.cc fa77985a87c6 src/dev/dma_device.hh fa77985a87c6 src/dev/dma_device.cc fa77985a87c6 src/dev/io_device.hh fa77985a87c6 src/dev/io_device.cc fa77985a87c6 src/dev/pcidev.hh fa77985a87c6 src/dev/x86/i82094aa.hh fa77985a87c6 src/mem/bridge.hh fa77985a87c6 src/mem/bridge.cc fa77985a87c6 src/mem/bus.hh fa77985a87c6 src/mem/bus.cc fa77985a87c6 src/mem/cache/base.hh fa77985a87c6 src/mem/cache/base.cc fa77985a87c6 src/mem/comm_monitor.hh fa77985a87c6 src/mem/comm_monitor.cc fa77985a87c6 src/mem/mem_object.hh fa77985a87c6 src/mem/mem_object.cc fa77985a87c6 src/mem/port.hh fa77985a87c6 src/mem/port.cc fa77985a87c6 src/mem/ruby/system/RubyPort.hh fa77985a87c6 src/mem/ruby/system/RubyPort.cc fa77985a87c6 src/mem/simple_mem.hh fa77985a87c6 src/mem/simple_mem.cc fa77985a87c6 src/python/swig/pyobject.cc fa77985a87c6 src/sim/system.hh fa77985a87c6 src/sim/system.cc fa77985a87c6 src/sim/tlb.hh fa77985a87c6 Diff: http://reviews.gem5.org/r/1273/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
