----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2490/ -----------------------------------------------------------
(Updated Nov. 24, 2014, 12:18 p.m.) Review request for Default. Repository: gem5 Description (updated) ------- Changeset 10563:3a541e8c1ff1 --------------------------- mem: Remove null-check bypassing in Packet::getPtr This patch removes the parameter that enables bypassing the null check in the Packet::getPtr method. A number of call sites assume the value to be non-null. The one odd case is the RubyTester, which issues zero-sized prefetches(!), and despite being reads they had no valid data pointer. This is now fixed, but the size oddity remains (unless anyone object or has any good suggestions). Finally, in the Ruby Sequencer, appropriate checks are made for flush packets as they have no valid data pointer. Diffs (updated) ----- src/cpu/testers/rubytest/Check.cc 426665ec11a9 src/mem/packet.hh 426665ec11a9 src/mem/ruby/slicc_interface/RubyRequest.cc 426665ec11a9 src/mem/ruby/slicc_interface/RubySlicc_Util.hh 426665ec11a9 src/mem/ruby/system/DMASequencer.cc 426665ec11a9 src/mem/ruby/system/Sequencer.cc 426665ec11a9 Diff: http://reviews.gem5.org/r/2490/diff/ Testing ------- Thanks, Andreas Hansson _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
