ARM: dump stats and process info on context switches
Review Request #1518 - Created Oct. 25, 2012 and submitted
| Information | |
|---|---|
| Ali Saidi | |
| gem5 | |
| default | |
| Reviewers | |
| Default | |
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).
Posted (Oct. 26, 2012, 7:04 a.m.)
Do we need to update the checkpoint upgrade script since new variables are being added?
Posted (Oct. 31, 2012, 1:46 a.m.)
-
src/arch/arm/linux/system.cc (Diff revision 1) -
Should we have the filename statically defined like this? Or should we have a parameter that has this default value?
-
src/arch/arm/linux/system.cc (Diff revision 1) -
Can you not store the value of IntReg(2) in a local variable? What is this register used for?
Posted (Oct. 31, 2012, 1:51 a.m.)
-
src/arch/arm/linux/system.cc (Diff revision 1) -
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 (Diff revision 1) -
sure.. it can be read once instead of the current 4 times.