----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3121/#review7207 -----------------------------------------------------------
Do *not* check in this patch. I know I've already saidy this, but I will repeat, do not make further changes to the sm files until we check in our GPU patches. Even after we check in our GPU patches, this patch must be split up before it should be considered for review. This is far too extensive and the changes in this patch go well beyond the patch description. First off, there are many, many unrelated style changes in this patch. Second, you cannot remove the Subblock object. The Subblock object is an important data structure for all testers. Beyond the RubyTester, we have internal testers that rely on the Subblock object. - Brad Beckmann On Sept. 17, 2015, 1:18 a.m., Nilay Vaish wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/3121/ > ----------------------------------------------------------- > > (Updated Sept. 17, 2015, 1:18 a.m.) > > > Review request for Default. > > > Repository: gem5 > > > Description > ------- > > Changeset 11124:3030ff525efb > --------------------------- > ruby: block size in RubySystem to be object specific > > This patch started with changing the variable for block size bits and bytes in > RubySystem from static to object specific. Most of the other changes made > were > required to support the original changes. > > > Diffs > ----- > > src/mem/ruby/network/garnet/fixed-pipeline/VirtualChannel_d.cc 5a2e1b1b5c43 > src/mem/ruby/network/garnet/fixed-pipeline/flitBuffer_d.hh 5a2e1b1b5c43 > src/mem/ruby/network/garnet/fixed-pipeline/flitBuffer_d.cc 5a2e1b1b5c43 > src/mem/ruby/network/garnet/fixed-pipeline/flit_d.hh 5a2e1b1b5c43 > src/mem/ruby/network/garnet/fixed-pipeline/flit_d.cc 5a2e1b1b5c43 > src/mem/ruby/network/garnet/flexible-pipeline/GarnetNetwork.hh 5a2e1b1b5c43 > src/mem/ruby/network/garnet/flexible-pipeline/GarnetNetwork.cc 5a2e1b1b5c43 > src/mem/ruby/network/garnet/flexible-pipeline/NetworkInterface.hh > 5a2e1b1b5c43 > src/mem/ruby/network/garnet/flexible-pipeline/NetworkInterface.cc > 5a2e1b1b5c43 > src/mem/ruby/network/garnet/flexible-pipeline/NetworkLink.hh 5a2e1b1b5c43 > src/mem/ruby/network/garnet/flexible-pipeline/NetworkLink.cc 5a2e1b1b5c43 > src/mem/ruby/network/garnet/flexible-pipeline/Router.hh 5a2e1b1b5c43 > src/mem/ruby/network/garnet/flexible-pipeline/Router.cc 5a2e1b1b5c43 > src/mem/ruby/network/garnet/flexible-pipeline/flit.hh 5a2e1b1b5c43 > src/mem/ruby/network/garnet/flexible-pipeline/flit.cc 5a2e1b1b5c43 > src/mem/ruby/network/garnet/flexible-pipeline/flitBuffer.hh 5a2e1b1b5c43 > src/mem/ruby/network/garnet/flexible-pipeline/flitBuffer.cc 5a2e1b1b5c43 > src/mem/ruby/network/simple/SimpleNetwork.hh 5a2e1b1b5c43 > src/mem/ruby/network/simple/SimpleNetwork.cc 5a2e1b1b5c43 > src/mem/ruby/network/simple/Switch.hh 5a2e1b1b5c43 > src/mem/ruby/network/simple/Switch.cc 5a2e1b1b5c43 > src/mem/ruby/profiler/AddressProfiler.cc 5a2e1b1b5c43 > src/mem/ruby/slicc_interface/AbstractController.hh 5a2e1b1b5c43 > src/mem/ruby/slicc_interface/AbstractController.cc 5a2e1b1b5c43 > src/mem/ruby/slicc_interface/Controller.py 5a2e1b1b5c43 > src/mem/ruby/slicc_interface/Message.hh 5a2e1b1b5c43 > src/mem/ruby/slicc_interface/RubyRequest.hh 5a2e1b1b5c43 > src/mem/ruby/slicc_interface/RubyRequest.cc 5a2e1b1b5c43 > src/mem/ruby/slicc_interface/RubySlicc_Util.hh 5a2e1b1b5c43 > src/mem/ruby/structures/CacheMemory.hh 5a2e1b1b5c43 > src/mem/ruby/structures/CacheMemory.cc 5a2e1b1b5c43 > src/mem/ruby/structures/DirectoryMemory.hh 5a2e1b1b5c43 > src/mem/ruby/structures/DirectoryMemory.cc 5a2e1b1b5c43 > src/mem/ruby/structures/DirectoryMemory.py 5a2e1b1b5c43 > src/mem/ruby/structures/PerfectCacheMemory.hh 5a2e1b1b5c43 > src/mem/ruby/structures/PersistentTable.cc 5a2e1b1b5c43 > src/mem/ruby/structures/Prefetcher.hh 5a2e1b1b5c43 > src/mem/ruby/structures/Prefetcher.cc 5a2e1b1b5c43 > src/mem/ruby/structures/RubyCache.py 5a2e1b1b5c43 > src/mem/ruby/structures/RubyPrefetcher.py 5a2e1b1b5c43 > src/mem/ruby/structures/TBETable.hh 5a2e1b1b5c43 > src/mem/ruby/structures/TimerTable.cc 5a2e1b1b5c43 > src/mem/ruby/system/CacheRecorder.hh 5a2e1b1b5c43 > src/mem/ruby/system/CacheRecorder.cc 5a2e1b1b5c43 > src/mem/ruby/system/DMASequencer.hh 5a2e1b1b5c43 > src/mem/ruby/system/DMASequencer.cc 5a2e1b1b5c43 > src/mem/ruby/system/RubyPort.cc 5a2e1b1b5c43 > src/mem/ruby/system/RubySystem.hh 5a2e1b1b5c43 > src/mem/ruby/system/RubySystem.cc 5a2e1b1b5c43 > src/mem/ruby/system/Sequencer.hh 5a2e1b1b5c43 > src/mem/ruby/system/Sequencer.cc 5a2e1b1b5c43 > src/mem/ruby/system/Sequencer.py 5a2e1b1b5c43 > src/mem/slicc/ast/ObjDeclAST.py 5a2e1b1b5c43 > src/mem/slicc/symbols/StateMachine.py 5a2e1b1b5c43 > src/mem/ruby/network/garnet/fixed-pipeline/Router_d.cc 5a2e1b1b5c43 > src/mem/ruby/network/garnet/fixed-pipeline/Switch_d.hh 5a2e1b1b5c43 > src/mem/ruby/network/garnet/fixed-pipeline/Switch_d.cc 5a2e1b1b5c43 > src/mem/ruby/network/garnet/fixed-pipeline/VirtualChannel_d.hh 5a2e1b1b5c43 > src/mem/ruby/network/garnet/fixed-pipeline/NetworkInterface_d.hh > 5a2e1b1b5c43 > src/mem/ruby/network/garnet/fixed-pipeline/NetworkInterface_d.cc > 5a2e1b1b5c43 > src/mem/ruby/network/garnet/fixed-pipeline/NetworkLink_d.hh 5a2e1b1b5c43 > src/mem/ruby/network/garnet/fixed-pipeline/NetworkLink_d.cc 5a2e1b1b5c43 > src/mem/ruby/network/garnet/fixed-pipeline/OutputUnit_d.hh 5a2e1b1b5c43 > src/mem/ruby/network/garnet/fixed-pipeline/OutputUnit_d.cc 5a2e1b1b5c43 > src/mem/ruby/network/garnet/fixed-pipeline/Router_d.hh 5a2e1b1b5c43 > src/mem/ruby/network/garnet/fixed-pipeline/InputUnit_d.cc 5a2e1b1b5c43 > src/cpu/testers/directedtest/RubyDirectedTester.hh 5a2e1b1b5c43 > src/cpu/testers/rubytest/Check.hh 5a2e1b1b5c43 > src/cpu/testers/rubytest/Check.cc 5a2e1b1b5c43 > src/cpu/testers/rubytest/CheckTable.hh 5a2e1b1b5c43 > src/cpu/testers/rubytest/CheckTable.cc 5a2e1b1b5c43 > src/cpu/testers/rubytest/RubyTester.hh 5a2e1b1b5c43 > src/cpu/testers/rubytest/RubyTester.cc 5a2e1b1b5c43 > src/cpu/testers/rubytest/RubyTester.py 5a2e1b1b5c43 > src/mem/protocol/MESI_Three_Level-L0cache.sm 5a2e1b1b5c43 > src/mem/protocol/MESI_Three_Level-L1cache.sm 5a2e1b1b5c43 > src/mem/protocol/MESI_Three_Level-msg.sm 5a2e1b1b5c43 > src/mem/protocol/MESI_Two_Level-L1cache.sm 5a2e1b1b5c43 > src/mem/protocol/MESI_Two_Level-L2cache.sm 5a2e1b1b5c43 > src/mem/protocol/MESI_Two_Level-dir.sm 5a2e1b1b5c43 > src/mem/protocol/MESI_Two_Level-dma.sm 5a2e1b1b5c43 > src/mem/protocol/MESI_Two_Level-msg.sm 5a2e1b1b5c43 > src/mem/protocol/MI_example-cache.sm 5a2e1b1b5c43 > src/mem/protocol/MI_example-dir.sm 5a2e1b1b5c43 > src/mem/protocol/MI_example-msg.sm 5a2e1b1b5c43 > src/mem/protocol/MOESI_CMP_directory-L1cache.sm 5a2e1b1b5c43 > src/mem/protocol/MOESI_CMP_directory-L2cache.sm 5a2e1b1b5c43 > src/mem/protocol/MOESI_CMP_directory-dir.sm 5a2e1b1b5c43 > src/mem/protocol/MOESI_CMP_directory-dma.sm 5a2e1b1b5c43 > src/mem/protocol/MOESI_CMP_directory-msg.sm 5a2e1b1b5c43 > src/mem/protocol/MOESI_CMP_token-L1cache.sm 5a2e1b1b5c43 > src/mem/protocol/MOESI_CMP_token-L2cache.sm 5a2e1b1b5c43 > src/mem/protocol/MOESI_CMP_token-dir.sm 5a2e1b1b5c43 > src/mem/protocol/MOESI_CMP_token-msg.sm 5a2e1b1b5c43 > src/mem/protocol/MOESI_hammer-cache.sm 5a2e1b1b5c43 > src/mem/protocol/MOESI_hammer-dir.sm 5a2e1b1b5c43 > src/mem/protocol/MOESI_hammer-msg.sm 5a2e1b1b5c43 > src/mem/protocol/Network_test-cache.sm 5a2e1b1b5c43 > src/mem/protocol/Network_test-msg.sm 5a2e1b1b5c43 > src/mem/protocol/RubySlicc_Defines.sm 5a2e1b1b5c43 > src/mem/protocol/RubySlicc_Exports.sm 5a2e1b1b5c43 > src/mem/protocol/RubySlicc_MemControl.sm 5a2e1b1b5c43 > src/mem/protocol/RubySlicc_Util.sm 5a2e1b1b5c43 > src/mem/ruby/common/Address.hh 5a2e1b1b5c43 > src/mem/ruby/common/Address.cc 5a2e1b1b5c43 > src/mem/ruby/common/DataBlock.hh 5a2e1b1b5c43 > src/mem/ruby/common/DataBlock.cc 5a2e1b1b5c43 > src/mem/ruby/common/SConscript 5a2e1b1b5c43 > src/mem/ruby/common/SubBlock.hh 5a2e1b1b5c43 > src/mem/ruby/common/SubBlock.cc 5a2e1b1b5c43 > src/mem/ruby/filters/AbstractBloomFilter.hh 5a2e1b1b5c43 > src/mem/ruby/filters/AbstractBloomFilter.cc PRE-CREATION > src/mem/ruby/filters/BlockBloomFilter.hh 5a2e1b1b5c43 > src/mem/ruby/filters/BlockBloomFilter.cc 5a2e1b1b5c43 > src/mem/ruby/filters/BulkBloomFilter.hh 5a2e1b1b5c43 > src/mem/ruby/filters/BulkBloomFilter.cc 5a2e1b1b5c43 > src/mem/ruby/filters/H3BloomFilter.hh 5a2e1b1b5c43 > src/mem/ruby/filters/H3BloomFilter.cc 5a2e1b1b5c43 > src/mem/ruby/filters/LSB_CountingBloomFilter.hh 5a2e1b1b5c43 > src/mem/ruby/filters/LSB_CountingBloomFilter.cc 5a2e1b1b5c43 > src/mem/ruby/filters/MultiBitSelBloomFilter.hh 5a2e1b1b5c43 > src/mem/ruby/filters/MultiBitSelBloomFilter.cc 5a2e1b1b5c43 > src/mem/ruby/filters/MultiGrainBloomFilter.hh 5a2e1b1b5c43 > src/mem/ruby/filters/MultiGrainBloomFilter.cc 5a2e1b1b5c43 > src/mem/ruby/filters/NonCountingBloomFilter.hh 5a2e1b1b5c43 > src/mem/ruby/filters/NonCountingBloomFilter.cc 5a2e1b1b5c43 > src/mem/ruby/filters/SConscript 5a2e1b1b5c43 > src/mem/ruby/network/MessageBuffer.hh 5a2e1b1b5c43 > src/mem/ruby/network/MessageBuffer.cc 5a2e1b1b5c43 > src/mem/ruby/network/Network.hh 5a2e1b1b5c43 > src/mem/ruby/network/Network.cc 5a2e1b1b5c43 > src/mem/ruby/network/Network.py 5a2e1b1b5c43 > src/mem/ruby/network/garnet/fixed-pipeline/GarnetNetwork_d.hh 5a2e1b1b5c43 > src/mem/ruby/network/garnet/fixed-pipeline/GarnetNetwork_d.cc 5a2e1b1b5c43 > src/mem/ruby/network/garnet/fixed-pipeline/InputUnit_d.hh 5a2e1b1b5c43 > > Diff: http://reviews.gem5.org/r/3121/diff/ > > > Testing > ------- > > All the protocols compile and pass hello world. > > > Thanks, > > Nilay Vaish > > _______________________________________________ gem5-dev mailing list gem5-dev@gem5.org http://m5sim.org/mailman/listinfo/gem5-dev