----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/334/#review607 -----------------------------------------------------------
Ship it! Nevermind my previous statement. I needed to look closer. Everything looks good to me. - Brad On 2011-01-04 14:42:14, Nilay Vaish wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.m5sim.org/r/334/ > ----------------------------------------------------------- > > (Updated 2011-01-04 14:42:14) > > > Review request for Default. > > > Summary > ------- > > This patch changes the manner in which data is copied from L1 to L2 cache > in the implementation of the Hammer's cache coherence protocol. Earlier, > data was copied directly from one cache entry to another. This has been > broken in to two parts. First, the data is copied from the source cache > entry to a transaction buffer entry. Then, data is copied from the > transaction buffer entry to the destination cache entry. > > This has been done to maintain the invariant - at any given instant, a > multiple caches under a controller are exclusive with respect to each > other. > > > Diffs > ----- > > src/mem/protocol/MOESI_hammer-cache.sm UNKNOWN > > Diff: http://reviews.m5sim.org/r/334/diff > > > Testing > ------- > > The changes have been tested with ruby random tester using a single seed for > 100,000 loads and number of processors = 1, 2, 4 and 8. > > > Thanks, > > Nilay > >
_______________________________________________ m5-dev mailing list m5-dev@m5sim.org http://m5sim.org/mailman/listinfo/m5-dev