Review Board 2.0.15


ARM: Use a stl queue for the table walker state

Review Request #158 - Created Aug. 13, 2010 and submitted

Information
Ali Saidi
gem5
Reviewers
Default
ARM: Use a stl queue for the table walker state

   
Posted (Aug. 19, 2010, 1:51 a.m.)
Why exactly did you make this change.  The FifoBuffer class limits (I think) the number of entries allowed and you removed that restriction.  Are you sure you want to do that?
  1. we don't really want to limit the number of requests the cpu can queue here. instead we'll limit the number of simultaneous requests that the table walker can have outstanding (currently 1). If we don't do this the cpu load/store/fetch gets even uglier since it can now have to replay a load because of a temporary table walker full case. 
src/arch/arm/table_walker.hh (Diff revision 1)
 
 
We put the standard includes above the rest in sorted order.
  1. fixed.