----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/653/#review1157 -----------------------------------------------------------
Hi Brad, the updated patch hasn't applied cleanly and so the diff is not visible. - Tushar - Tushar On 2011-04-25 11:54:17, Brad Beckmann wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.m5sim.org/r/653/ > ----------------------------------------------------------- > > (Updated 2011-04-25 11:54:17) > > > Review request for Default, Ali Saidi, Gabe Black, Steve Reinhardt, and > Nathan Binkert. > > > Summary > ------- > > network: convert links & switches to first class C++ SimObjects > > This patch converts links and switches from second class simobjects that were > virtually ignored by the networks (both simple and Garnet) to first class > simobjects that directly correspond to c++ ojbects manipulated by the > topology and network classes. This is especially true for Garnet, where the > links and switches directly correspond to specific C++ objects. > > By making this change, many aspects of the Topology class were simplified. > > > Diffs > ----- > > configs/ruby/MESI_CMP_directory.py 12afb5f7f678 > configs/ruby/MI_example.py 12afb5f7f678 > configs/ruby/MOESI_CMP_directory.py 12afb5f7f678 > configs/ruby/MOESI_CMP_token.py 12afb5f7f678 > configs/ruby/MOESI_hammer.py 12afb5f7f678 > configs/ruby/Network_test.py 12afb5f7f678 > configs/ruby/Ruby.py 12afb5f7f678 > src/mem/protocol/RubySlicc_Exports.sm 12afb5f7f678 > src/mem/ruby/network/BasicLink.hh PRE-CREATION > src/mem/ruby/network/BasicLink.cc PRE-CREATION > src/mem/ruby/network/BasicLink.py PRE-CREATION > src/mem/ruby/network/BasicRouter.hh PRE-CREATION > src/mem/ruby/network/BasicRouter.cc PRE-CREATION > src/mem/ruby/network/BasicRouter.py PRE-CREATION > src/mem/ruby/network/Network.hh 12afb5f7f678 > src/mem/ruby/network/Network.py 12afb5f7f678 > src/mem/ruby/network/SConscript 12afb5f7f678 > src/mem/ruby/network/Topology.hh 12afb5f7f678 > src/mem/ruby/network/Topology.cc 12afb5f7f678 > src/mem/ruby/network/garnet/fixed-pipeline/CreditLink_d.hh 12afb5f7f678 > src/mem/ruby/network/garnet/fixed-pipeline/GarnetLink_d.hh PRE-CREATION > src/mem/ruby/network/garnet/fixed-pipeline/GarnetLink_d.cc PRE-CREATION > src/mem/ruby/network/garnet/fixed-pipeline/GarnetLink_d.py PRE-CREATION > src/mem/ruby/network/garnet/fixed-pipeline/GarnetNetwork_d.hh 12afb5f7f678 > src/mem/ruby/network/garnet/fixed-pipeline/GarnetNetwork_d.cc 12afb5f7f678 > src/mem/ruby/network/garnet/fixed-pipeline/GarnetRouter_d.py PRE-CREATION > src/mem/ruby/network/garnet/fixed-pipeline/NetworkLink_d.hh 12afb5f7f678 > src/mem/ruby/network/garnet/fixed-pipeline/NetworkLink_d.cc 12afb5f7f678 > src/mem/ruby/network/garnet/fixed-pipeline/Router_d.hh 12afb5f7f678 > src/mem/ruby/network/garnet/fixed-pipeline/Router_d.cc 12afb5f7f678 > src/mem/ruby/network/garnet/fixed-pipeline/SConscript 12afb5f7f678 > src/mem/ruby/network/garnet/flexible-pipeline/GarnetLink.hh PRE-CREATION > src/mem/ruby/network/garnet/flexible-pipeline/GarnetLink.cc PRE-CREATION > src/mem/ruby/network/garnet/flexible-pipeline/GarnetLink.py PRE-CREATION > src/mem/ruby/network/garnet/flexible-pipeline/GarnetNetwork.hh 12afb5f7f678 > src/mem/ruby/network/garnet/flexible-pipeline/GarnetNetwork.cc 12afb5f7f678 > src/mem/ruby/network/garnet/flexible-pipeline/GarnetRouter.py PRE-CREATION > src/mem/ruby/network/garnet/flexible-pipeline/NetworkLink.hh 12afb5f7f678 > src/mem/ruby/network/garnet/flexible-pipeline/NetworkLink.cc 12afb5f7f678 > src/mem/ruby/network/garnet/flexible-pipeline/Router.hh 12afb5f7f678 > src/mem/ruby/network/garnet/flexible-pipeline/Router.cc 12afb5f7f678 > src/mem/ruby/network/garnet/flexible-pipeline/SConscript 12afb5f7f678 > src/mem/ruby/network/orion/NetworkPower.cc 12afb5f7f678 > src/mem/ruby/network/simple/SimpleNetwork.hh 12afb5f7f678 > src/mem/ruby/network/simple/SimpleNetwork.cc 12afb5f7f678 > src/mem/ruby/network/topologies/Crossbar.py 12afb5f7f678 > src/mem/ruby/network/topologies/Mesh.py 12afb5f7f678 > src/mem/ruby/network/topologies/MeshDirCorners.py 12afb5f7f678 > src/mem/ruby/slicc_interface/AbstractController.hh 12afb5f7f678 > src/mem/ruby/slicc_interface/Controller.py 12afb5f7f678 > > Diff: http://reviews.m5sim.org/r/653/diff > > > Testing > ------- > > > Thanks, > > Brad > > _______________________________________________ m5-dev mailing list m5-dev@m5sim.org http://m5sim.org/mailman/listinfo/m5-dev