x86: fix table walker assertion
authorBinh Pham <binhpham@cs.rutgers.edu>
Sat, 21 Jun 2014 17:39:44 +0000 (10:39 -0700)
committerBinh Pham <binhpham@cs.rutgers.edu>
Sat, 21 Jun 2014 17:39:44 +0000 (10:39 -0700)
commitb085db84afcbb4824d34b8755f4c09c1fcfefcee
tree624324e3628addbf4dbc1ea567021a16131ec324
parentb72c8798682d307adfe9c1c2a77ac5a6d6240c15
x86: fix table walker assertion

In a cycle, we could see a R and W requests corresponding to the same
page walk being sent to the memory. During the cycle that assertion
happens, we have 2 responses corresponding to the R and W above. We
also have a 'read' variable to keep track of the inflight Read
request, this gets reset to NULL right after we send out any R
request; and gets set to the next R in the page walk when a response
comes back.

The issue we are seeing here is when we get a response for W request,
assert(!read) fires because we got a response for R request right
before this, hence we set 'read' to NOT NULL value, pointing to the
next R request in the pagewalk!

This work was done while Binh was an intern at AMD Research.
src/arch/x86/pagetable_walker.cc