----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/1475/ -----------------------------------------------------------
Review request for Default. Description ------- Changeset 9308:0c8aa2de4649 --------------------------- config: Use SimpleDRAM in full-system, and with o3 and inorder This patch favours using SimpleDRAM with the default timing instead of SimpleMemory for all regressions that involve the o3 or inorder CPU, or are full system (in other words, where the actual performance of the memory is important for the overall performance). Moving forward, the solution for FSConfig and the users of fs.py and se.py is probably something similar to what we use to choose the CPU type. I envision a few pre-set configurations SimpleLPDDR2, SimpleDDR3, etc that can be choosen by a dram_type option. Feedback on this part is welcome. This patch changes plenty stats and adds all the DRAM controller related stats. A follow-on patch updates the relevant statistics. The total run-time for the entire regression goes up with ~5% with this patch due to the added complexity of the SimpleDRAM model. This is a concious trade-off to ensure that the model is properly tested. Diffs ----- configs/common/FSConfig.py ac608464be80 tests/configs/inorder-timing.py ac608464be80 tests/configs/o3-timing-checker.py ac608464be80 tests/configs/o3-timing-mp.py ac608464be80 tests/configs/o3-timing.py ac608464be80 Diff: http://reviews.gem5.org/r/1475/diff/ Testing ------- util/regress all passing (disregarding t1000 and eio) stats update will follow Thanks, Andreas Hansson _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
