----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/1518/#review3657 -----------------------------------------------------------
src/arch/arm/linux/system.cc <http://reviews.gem5.org/r/1518/#comment3634> I'm personally fine with the static name. I doubt anyone has changed the same of the stats output file even though it's an option and largely we want analysis tools to be able to figure out if the file is there so they can operate on it. I src/arch/arm/linux/system.cc <http://reviews.gem5.org/r/1518/#comment3633> sure.. it can be read once instead of the current 4 times. - Ali Saidi On Oct. 25, 2012, 10:48 a.m., Ali Saidi wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/1518/ > ----------------------------------------------------------- > > (Updated Oct. 25, 2012, 10:48 a.m.) > > > Review request for Default. > > > Description > ------- > > Changeset 9319:49396f5b7637 > --------------------------- > ARM: dump stats and process info on context switches > > This patch enables dumping statistics and Linux process information on > context switch boundaries (__switch_to() calls) that are used for > Streamline integration (a graphical statistics viewer from ARM). > > > Diffs > ----- > > src/arch/arm/ArmSystem.py f634a34f2f0b > src/arch/arm/linux/system.hh f634a34f2f0b > src/arch/arm/linux/system.cc f634a34f2f0b > src/cpu/base.hh f634a34f2f0b > src/cpu/base.cc f634a34f2f0b > src/cpu/pc_event.cc f634a34f2f0b > src/mem/request.hh f634a34f2f0b > > Diff: http://reviews.gem5.org/r/1518/diff/ > > > Testing > ------- > > > Thanks, > > Ali Saidi > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
