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



src/arch/x86/vtophys.cc
<http://reviews.m5sim.org/r/385/#comment1009>

    I really don't like this page size in request size hack. It's confusing and 
obfuscates what's going on, and it changes the meaning of a well defined field 
in the request object. There's really no good reason for it, and I'd love to 
see it go.


- Gabe


On 2011-01-10 11:26:26, Brad Beckmann wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviews.m5sim.org/r/385/
> -----------------------------------------------------------
> 
> (Updated 2011-01-10 11:26:26)
> 
> 
> Review request for Default, Ali Saidi, Gabe Black, Steve Reinhardt, and 
> Nathan Binkert.
> 
> 
> Summary
> -------
> 
> x86: implements vtophys
> 
> Calls walker to look up virt. to phys. page mapping
> 
> 
> Diffs
> -----
> 
>   src/arch/x86/vtophys.cc 9f9e10967912 
> 
> Diff: http://reviews.m5sim.org/r/385/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Brad
> 
>

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

Reply via email to