----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2201/#review4951 -----------------------------------------------------------
Hi Amin, This fix is already part of: http://reviews.gem5.org/r/2196/ Thanks though! If you've got a few minutes to spare, please have a look at the DRAM patches I posted last week. It would be good to get it all done before ISPASS. - Andreas Hansson On March 11, 2014, 1:04 a.m., Amin Farmahini wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/2201/ > ----------------------------------------------------------- > > (Updated March 11, 2014, 1:04 a.m.) > > > Review request for Default. > > > Repository: gem5 > > > Description > ------- > > Consider this scenario: Read queue becomes full, so retryRdReq becomes true. > But write queue is not full. Once a write req is accepted retryRdReq is > changed to false, so no read retry is ever sent to the requester. > > > Diffs > ----- > > src/mem/simple_dram.cc 580b47334a97 > > Diff: http://reviews.gem5.org/r/2201/diff/ > > > Testing > ------- > > > Thanks, > > Amin Farmahini > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
