----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/653/#review1150 -----------------------------------------------------------
src/mem/ruby/network/BasicLink.py <http://reviews.m5sim.org/r/653/#comment1592> could you add a comment somewhere that the bw_multiplier field is only used by the simple network? src/mem/ruby/network/BasicRouter.hh <http://reviews.m5sim.org/r/653/#comment1588> typo in "relation" src/mem/ruby/network/Network.py <http://reviews.m5sim.org/r/653/#comment1593> Some parameters such as topology, number_of_virtual_networks etc are used by all networks. bandwidth and adaptive routing parameters are only used by simple network. Either we could move them to a SimpleNetwork.py kind of file, OR perhaps we should move the BaseGarnetNetwork.py parameters here into Network.py with appropriate comments? src/mem/ruby/network/garnet/fixed-pipeline/GarnetLink_d.py <http://reviews.m5sim.org/r/653/#comment1590> I have been thinking of renaming vcs_per_class to vcs_per_vnet for sometime now, to make it consistent with "virtual networks" (instead of message class). But that would need a change in BaseGarnetNetwork.py/hh/cc too? If it is difficult for you to do, I can do that in a subsequent patch. src/mem/ruby/network/garnet/fixed-pipeline/GarnetNetwork_d.cc <http://reviews.m5sim.org/r/653/#comment1591> Why is "direction" needed in all the links? Don't the src/dest appropriately set up the producers and consumers for the link? Is "direction" being used for stats? To be consistent, the direction of a credit link will be opposite to the direction of the corresponding network link. - Tushar On 2011-04-22 15:16:58, Brad Beckmann wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.m5sim.org/r/653/ > ----------------------------------------------------------- > > (Updated 2011-04-22 15:16:58) > > > 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 914389024c33 > configs/ruby/MI_example.py 914389024c33 > configs/ruby/MOESI_CMP_directory.py 914389024c33 > configs/ruby/MOESI_CMP_token.py 914389024c33 > configs/ruby/MOESI_hammer.py 914389024c33 > configs/ruby/Network_test.py 914389024c33 > configs/ruby/Ruby.py 914389024c33 > src/mem/protocol/RubySlicc_Exports.sm 914389024c33 > 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 914389024c33 > src/mem/ruby/network/Network.py 914389024c33 > src/mem/ruby/network/SConscript 914389024c33 > src/mem/ruby/network/Topology.hh PRE-CREATION > src/mem/ruby/network/Topology.cc PRE-CREATION > src/mem/ruby/network/garnet/fixed-pipeline/CreditLink_d.hh 914389024c33 > 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 914389024c33 > src/mem/ruby/network/garnet/fixed-pipeline/GarnetNetwork_d.cc 914389024c33 > src/mem/ruby/network/garnet/fixed-pipeline/GarnetRouter_d.py PRE-CREATION > src/mem/ruby/network/garnet/fixed-pipeline/NetworkLink_d.hh 914389024c33 > src/mem/ruby/network/garnet/fixed-pipeline/NetworkLink_d.cc 914389024c33 > src/mem/ruby/network/garnet/fixed-pipeline/Router_d.hh 914389024c33 > src/mem/ruby/network/garnet/fixed-pipeline/Router_d.cc 914389024c33 > src/mem/ruby/network/garnet/fixed-pipeline/SConscript 914389024c33 > 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 914389024c33 > src/mem/ruby/network/garnet/flexible-pipeline/GarnetNetwork.cc 914389024c33 > src/mem/ruby/network/garnet/flexible-pipeline/GarnetRouter.py PRE-CREATION > src/mem/ruby/network/garnet/flexible-pipeline/NetworkLink.hh 914389024c33 > src/mem/ruby/network/garnet/flexible-pipeline/NetworkLink.cc 914389024c33 > src/mem/ruby/network/garnet/flexible-pipeline/Router.hh 914389024c33 > src/mem/ruby/network/garnet/flexible-pipeline/Router.cc 914389024c33 > src/mem/ruby/network/garnet/flexible-pipeline/SConscript 914389024c33 > src/mem/ruby/network/orion/NetworkPower.cc 914389024c33 > src/mem/ruby/network/simple/SimpleNetwork.hh 914389024c33 > src/mem/ruby/network/simple/SimpleNetwork.cc 914389024c33 > src/mem/ruby/network/topologies/Crossbar.py 914389024c33 > src/mem/ruby/network/topologies/Mesh.py 914389024c33 > src/mem/ruby/network/topologies/MeshDirCorners.py 914389024c33 > src/mem/ruby/slicc_interface/AbstractController.hh 914389024c33 > src/mem/ruby/slicc_interface/Controller.py 914389024c33 > > 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
