-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviews.gem5.org/r/2276/#review5104
-----------------------------------------------------------



src/mem/ruby/system/Sequencer.cc
<http://reviews.gem5.org/r/2276/#comment4622>

    Not sure what your thoughts are, but using "auto" would clean it up quite a 
bit (here and other places in the file).


- Andreas Hansson


On May 14, 2014, 5:47 a.m., Steve Reinhardt wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviews.gem5.org/r/2276/
> -----------------------------------------------------------
> 
> (Updated May 14, 2014, 5:47 a.m.)
> 
> 
> Review request for Default.
> 
> 
> Repository: gem5
> 
> 
> Description
> -------
> 
> Changeset 10228:f377969aa1b2
> ---------------------------
> 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/ruby/system/Sequencer.hh 34f48d0dac97d04f3d1e129a1f4aa00d869b1d8f 
>   src/mem/ruby/system/Sequencer.cc 34f48d0dac97d04f3d1e129a1f4aa00d869b1d8f 
> 
> 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

Reply via email to