----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/165/#review212 -----------------------------------------------------------
src/arch/arm/faults.cc <http://reviews.m5sim.org/r/165/#comment358> M5_USED_VAR was added instead. Having the PC here is nice for debugging src/arch/arm/faults.cc <http://reviews.m5sim.org/r/165/#comment359> Even though I didn't change it, it's ifxed src/arch/arm/linux/atag.hh <http://reviews.m5sim.org/r/165/#comment360> No because part of the spec says that every descriptor is in units of 4 byte chunks src/arch/arm/linux/atag.hh <http://reviews.m5sim.org/r/165/#comment362> The style guide is silent on the matter src/arch/arm/linux/atag.hh <http://reviews.m5sim.org/r/165/#comment361> Fine - Ali On 2010-08-13 09:44:46, Ali Saidi wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.m5sim.org/r/165/ > ----------------------------------------------------------- > > (Updated 2010-08-13 09:44:46) > > > Review request for Default. > > > Summary > ------- > > ARM: Add system for ARM/Linux and bootstrapping > > > Diffs > ----- > > src/arch/arm/ArmSystem.py 3c48b2b3cb83 > src/arch/arm/SConscript 3c48b2b3cb83 > src/arch/arm/faults.cc 3c48b2b3cb83 > src/arch/arm/linux/atag.hh PRE-CREATION > src/arch/arm/linux/system.hh PRE-CREATION > src/arch/arm/linux/system.cc PRE-CREATION > src/arch/arm/system.cc 3c48b2b3cb83 > > Diff: http://reviews.m5sim.org/r/165/diff > > > Testing > ------- > > > Thanks, > > Ali > > _______________________________________________ m5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/m5-dev
