----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2276/#review5314 -----------------------------------------------------------
This patch seems to be interfering with the ruby drain functionality, I have tried a switch cpu from the timing to the detailed cpu model and the draining of the Sequencer will not end. I believe that the issue is related to the number of outstanding requests not being updated properly. - Emilio Castillo On June 21, 2014, 5 p.m., Steve Reinhardt wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/2276/ > ----------------------------------------------------------- > > (Updated June 21, 2014, 5 p.m.) > > > Review request for Default. > > > Repository: gem5 > > > Description > ------- > > Changeset 10240:b01d667ec431 > --------------------------- > ruby: don't make O3 CPU squash on loads that hit outstanding requests > > Mismatch between O3 and Ruby in handling aliased requests: Ruby > returns false when it sees aliased request or memory blocked. O3 > squash and refetch when it sees false signal from Ruby. > > Fix: Merging readRequestTable and writeRequestTable in a single table > that maps requesting address and all requests that are aliased with > the address. > > This work was done while Binh was an intern at AMD Research. > > > Diffs > ----- > > src/mem/packet.cc b21b3aad6bd1d01ac8a4d8030479bbca417af8d1 > src/mem/protocol/RubySlicc_Exports.sm > b21b3aad6bd1d01ac8a4d8030479bbca417af8d1 > src/mem/ruby/system/DMASequencer.hh > b21b3aad6bd1d01ac8a4d8030479bbca417af8d1 > src/mem/ruby/system/DMASequencer.cc > b21b3aad6bd1d01ac8a4d8030479bbca417af8d1 > src/mem/ruby/system/RubyPort.hh b21b3aad6bd1d01ac8a4d8030479bbca417af8d1 > src/mem/ruby/system/RubyPort.cc b21b3aad6bd1d01ac8a4d8030479bbca417af8d1 > src/mem/ruby/system/RubyPortProxy.hh > b21b3aad6bd1d01ac8a4d8030479bbca417af8d1 > src/mem/ruby/system/RubyPortProxy.cc > b21b3aad6bd1d01ac8a4d8030479bbca417af8d1 > src/mem/ruby/system/Sequencer.hh b21b3aad6bd1d01ac8a4d8030479bbca417af8d1 > src/mem/ruby/system/Sequencer.cc b21b3aad6bd1d01ac8a4d8030479bbca417af8d1 > > Diff: http://reviews.gem5.org/r/2276/diff/ > > > Testing > ------- > > > Thanks, > > Steve Reinhardt > > _______________________________________________ gem5-dev mailing list gem5-dev@gem5.org http://m5sim.org/mailman/listinfo/gem5-dev
