----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/951/#review1924 -----------------------------------------------------------
Ship it! I confess to not reading all the new bridge code closely, but overall this looks fine. src/mem/bridge.hh <http://reviews.m5sim.org/r/951/#comment2427> typo - Steve On 2012-01-11 02:31:03, Andreas Hansson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.m5sim.org/r/951/ > ----------------------------------------------------------- > > (Updated 2012-01-11 02:31:03) > > > Review request for Default. > > > Summary > ------- > > MEM: Make the bus bridge unidirectional and fixed address range > > This patch makes the bus bridge uni-directional and specialises the > bus ports to be a master port and a slave port. This greatly > simplifies the assumptions on both sides as either port only has to > deal with requests or responses. The following patches introduce the > notion of master and slave ports, and would not be possible without > this split of responsibilities. > > In making the bridge unidirectional, the address range mechanism of > the bridge is also changed. For the cases where communication is > taking place both ways, an additional bridge is needed. This causes > issues with the existing mechanism, as the busses cannot determine > when to stop iterating the address updates from the two bridges. To > avoid this issue, and also greatly simplify the specification, the > bridge now has a fixed set of address ranges, specified at creation > time. > > > Diffs > ----- > > configs/common/FSConfig.py 508bbec99e58 > configs/example/fs.py 508bbec99e58 > src/dev/arm/RealView.py 508bbec99e58 > src/mem/Bridge.py 508bbec99e58 > src/mem/bridge.hh 508bbec99e58 > src/mem/bridge.cc 508bbec99e58 > tests/configs/pc-o3-timing.py 508bbec99e58 > tests/configs/pc-simple-atomic.py 508bbec99e58 > tests/configs/pc-simple-timing.py 508bbec99e58 > tests/configs/realview-o3-dual.py 508bbec99e58 > tests/configs/realview-o3.py 508bbec99e58 > tests/configs/realview-simple-atomic-dual.py 508bbec99e58 > tests/configs/realview-simple-atomic.py 508bbec99e58 > tests/configs/realview-simple-timing-dual.py 508bbec99e58 > tests/configs/realview-simple-timing.py 508bbec99e58 > tests/configs/tsunami-inorder.py 508bbec99e58 > tests/configs/tsunami-o3-dual.py 508bbec99e58 > tests/configs/tsunami-o3.py 508bbec99e58 > tests/configs/tsunami-simple-atomic-dual.py 508bbec99e58 > tests/configs/tsunami-simple-atomic.py 508bbec99e58 > tests/configs/tsunami-simple-timing-dual.py 508bbec99e58 > tests/configs/tsunami-simple-timing.py 508bbec99e58 > tests/configs/twosys-tsunami-simple-atomic.py 508bbec99e58 > > Diff: http://reviews.m5sim.org/r/951/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
