> On Feb. 6, 2015, 8:16 a.m., Andreas Hansson wrote: > > Ship It!
Here it would be good to test with the new memtest: http://reviews.gem5.org/r/2612/ Note that the caches suffer from message deadlock at the moment, and the test dies very quickly. I shall post a patch that fixes the message deadlock in the next few days. - Andreas ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2637/#review5848 ----------------------------------------------------------- On Feb. 6, 2015, 12:38 a.m., Steve Reinhardt wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/2637/ > ----------------------------------------------------------- > > (Updated Feb. 6, 2015, 12:38 a.m.) > > > Review request for Default. > > > Repository: gem5 > > > Description > ------- > > Changeset 10684:ded42ff6f410 > --------------------------- > mem: clean up write buffer check in Cache::handleSnoop() > > The 'if (writebacks.size)' check was redundant, because > writeBuffer.findMatches() would return false if the > writebacks list was empty. > > Also renamed 'mshr' to 'wb_entry' in this context since > we are pointing at a writebuffer entry and not an MSHR > (even though it's the same C++ class). > > > Diffs > ----- > > src/mem/cache/cache_impl.hh 3d17366c0423a59478ae63d40c8feeea34df218a > > Diff: http://reviews.gem5.org/r/2637/diff/ > > > Testing > ------- > > > Thanks, > > Steve Reinhardt > > _______________________________________________ gem5-dev mailing list gem5-dev@gem5.org http://m5sim.org/mailman/listinfo/gem5-dev