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


I guess this is fixed, given the check for isSecure currently found in 
cache_impl.hh?

- Andreas Hansson


On June 12, 2014, 10:47 p.m., Ali Saidi wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviews.gem5.org/r/2291/
> -----------------------------------------------------------
> 
> (Updated June 12, 2014, 10:47 p.m.)
> 
> 
> Review request for Default.
> 
> 
> Repository: gem5
> 
> 
> Description
> -------
> 
> Changeset 10239:af942fe9d30b
> ---------------------------
> mem: Properly set cache block status fields on writebacks
> 
> When a cacheline is written back to a lower-level cache, tags->insertBlock()
> sets various status parameters.  However these status bits were cleared
> immediately after calling.  This patch makes it so that these status fields 
> are
> not cleared.
> 
> 
> Diffs
> -----
> 
>   src/mem/cache/cache_impl.hh a2bb75a474fd 
> 
> Diff: http://reviews.gem5.org/r/2291/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Ali Saidi
> 
>

_______________________________________________
gem5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/gem5-dev

Reply via email to