Hello All,
I think the if condition that's checking to find the right condition for
auto-precharge in doDRAMAccess() can be simplified
Original :
while (!(got_more_hits &&
(got_bank_conflict || pageMgmt == Enums::close_adaptive))
&&
p != queue.end()) {
Simplified:
while (!got_more_hits && p != queue.end()) {
The above simplification comes as both close-adaptive and open-adaptive
policies keep row open if a hit is found. Otherwise the search for a hit
continues until the end of the queue and during the search
got_bank_conflict gets updated anyways.
I am planning to put this simplification on reviewboard (along with another
bug fix that I have). I would appreciate it if anyone using DRAM controller
has any comments regarding this simplification.
Thank you,
-Rizwana
_______________________________________________
gem5-users mailing list
[email protected]
http://m5sim.org/cgi-bin/mailman/listinfo/gem5-users
