-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviews.m5sim.org/r/338/#review720
-----------------------------------------------------------



src/cpu/o3/commit_impl.hh
<http://reviews.m5sim.org/r/338/#comment984>

    I split this into two functions, but boolean is if interrupt == NoFault or 
not. The flow control is much better and after it passes regressions I'll post 
it. 



src/cpu/o3/fetch_impl.hh
<http://reviews.m5sim.org/r/338/#comment985>

    Moved the check into fetchCacheLine. No ugly macros, but there still is an 
alpha specific bit



src/cpu/o3/fetch_impl.hh
<http://reviews.m5sim.org/r/338/#comment986>

    merged



src/cpu/o3/iew_impl.hh
<http://reviews.m5sim.org/r/338/#comment987>

    yup


- Ali


On 2010-12-06 16:10:48, Ali Saidi wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviews.m5sim.org/r/338/
> -----------------------------------------------------------
> 
> (Updated 2010-12-06 16:10:48)
> 
> 
> Review request for Default, Ali Saidi, Gabe Black, Steve Reinhardt, and 
> Nathan Binkert.
> 
> 
> Summary
> -------
> 
> O3: Fixes fetch deadlock when the interrupt master clears single before CPU 
> handles it.
> Then the cpu should restart fetch stage to fetch from the original execution
> path.
> 
> 
> Diffs
> -----
> 
>   src/arch/arm/interrupts.hh 2b5fbdcbfb5d 
>   src/cpu/o3/commit_impl.hh 2b5fbdcbfb5d 
>   src/cpu/o3/fetch_impl.hh 2b5fbdcbfb5d 
>   src/cpu/o3/iew_impl.hh 2b5fbdcbfb5d 
> 
> Diff: http://reviews.m5sim.org/r/338/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Ali
> 
>

_______________________________________________
m5-dev mailing list
m5-dev@m5sim.org
http://m5sim.org/mailman/listinfo/m5-dev

Reply via email to