mem: Remove null-check bypassing in Packet::getPtr
authorAndreas Hansson <andreas.hansson@arm.com>
Tue, 2 Dec 2014 11:07:34 +0000 (06:07 -0500)
committerAndreas Hansson <andreas.hansson@arm.com>
Tue, 2 Dec 2014 11:07:34 +0000 (06:07 -0500)
commit25bfc249998b26403d50587eb66e6ee5e6de5b58
treec952ab3ac56ae3fe2f5f88fec797e6a1ca252650
parent0e63d2cd62bbab47a5b05b9b5bee8c1dc0da1683
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.
src/cpu/testers/rubytest/Check.cc
src/mem/packet.hh
src/mem/ruby/slicc_interface/RubyRequest.cc
src/mem/ruby/slicc_interface/RubySlicc_Util.hh
src/mem/ruby/system/DMASequencer.cc
src/mem/ruby/system/Sequencer.cc