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



src/cpu/base.cc
<http://reviews.m5sim.org/r/318/#comment716>

    I like how this seems pretty general purpose and things shouldn't get lost 
in the shuffle this way (I think) but it would be nice if it didn't assume 
there was only one port that needed to be moved. Off hand I don't have any 
suggestion to fix it that wouldn't be overly cumbersome.


- Gabe


On 2010-11-19 16:11:58, Ali Saidi wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviews.m5sim.org/r/318/
> -----------------------------------------------------------
> 
> (Updated 2010-11-19 16:11:58)
> 
> 
> Review request for Default.
> 
> 
> Summary
> -------
> 
> ARM: Support switchover with hardware table walkers
> 
> 
> Diffs
> -----
> 
>   src/arch/arm/table_walker.cc 6286bb50127e 
>   src/arch/arm/tlb.hh 6286bb50127e 
>   src/arch/arm/tlb.cc 6286bb50127e 
>   src/cpu/base.cc 6286bb50127e 
>   src/sim/tlb.hh 6286bb50127e 
> 
> Diff: http://reviews.m5sim.org/r/318/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Ali
> 
>

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

Reply via email to