> On 2010-08-19 08:55:19, Nathan Binkert wrote: > > Seems reasonable. How does the mask work and why is it different for two > > different OSes? Can you add a better comment somewhere? > > Ali Saidi wrote: > ARM linux is compiled at some virtual address that is platform dependent > (although they're almost all the same). However, however it needs to be > loaded in physical memory which normally starts at address 0. We did the same > thing for Alpha years ago. Some bare-metal tool sets generate code where > physical address == virtual address so this doesn't need to be done. > > > > Nathan Binkert wrote: > Can you explain this in the code somewhere? > > Ali Saidi wrote: > adde to system.hh > > > Gabe Black wrote: > Could we handle this instead by respecting the virtual vs. physical > addresses in the ELFes (ELVes?)? I think I tried that before and the answer > was no, but I don't remember why.
I'm pretty sure we tried that before and it didn't work, although the reason escapes me. Either way this change gives us more flexibility in the future. - Ali ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/160/#review230 ----------------------------------------------------------- On 2010-08-13 09:42:08, Ali Saidi wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.m5sim.org/r/160/ > ----------------------------------------------------------- > > (Updated 2010-08-13 09:42:08) > > > Review request for Default. > > > Summary > ------- > > Loader: Make the load address mask be a parameter of the system rather than a > constant. This allows one two different OS requirements for the same ISA to > be handled. > > > Diffs > ----- > > src/arch/alpha/AlphaSystem.py 3c48b2b3cb83 > src/arch/alpha/isa_traits.hh 3c48b2b3cb83 > src/arch/alpha/system.cc 3c48b2b3cb83 > src/arch/arm/ArmSystem.py 3c48b2b3cb83 > src/arch/arm/isa_traits.hh 3c48b2b3cb83 > src/arch/mips/MipsSystem.py 3c48b2b3cb83 > src/arch/mips/isa_traits.hh 3c48b2b3cb83 > src/arch/mips/system.cc 3c48b2b3cb83 > src/arch/sparc/SparcSystem.py 3c48b2b3cb83 > src/arch/sparc/isa_traits.hh 3c48b2b3cb83 > src/arch/x86/X86System.py 3c48b2b3cb83 > src/arch/x86/isa_traits.hh 3c48b2b3cb83 > src/sim/System.py 3c48b2b3cb83 > src/sim/system.hh 3c48b2b3cb83 > src/sim/system.cc 3c48b2b3cb83 > > Diff: http://reviews.m5sim.org/r/160/diff > > > Testing > ------- > > > Thanks, > > Ali > > _______________________________________________ m5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/m5-dev
