Hey Andreas, Your timeline looks good to me! My only question would be about that MIPS support in the old model and if that is worth keeping around in some form? For example, if the old inorder is the only model that actually runs MIPS regressions, maybe just keep the MIPS SE-mode regressions for the old inorder. At the very least, we ensure that the ISA definition/code from MIPS doesn't get broken as changes keep coming.
Alternatively, if MIPS ISA code is something that is O3 regressions are handling or if MIPS ISA is something that we are not worried about keeping alive, I agree on the name switch and removal of the old inorder code from the repository on your timeline. A brief announcement in September letting people know they have "X days" to grab the old inorder code before it's removed and minor will be renamed to inorder sounds good to me as well. -Korey On Tue, Aug 12, 2014 at 3:39 AM, Andreas Hansson via gem5-dev < [email protected]> wrote: > Hi all, > > Hopefully everyone is enjoying the new-and-improved in-order ‘minor’ CPU > model. As you may have noticed, at the moment both of the in-order CPU > models are used in the regression flow. The old ‘inorder’ is primarily used > for alpha (and the odd mips and sparc regression), with the new ‘minor’ > model used far more extensively for both arm and alpha. As we run both > ‘inorder' and ‘minor’, a full regression has increased from a total of 13 - > 14 hours to 17 - 18 hours. In practical terms that means that a regression > run started in the evening (when leaving work) is no longer finished the > morning after. To remedy the regression run-time issue, and also focus our > development efforts on the ‘minor’ model, I propose the following time > table: > > August (now) > Remove ‘inorder' regressions and add deprecation warning when > instantiating ‘in order’ CPUs instructing the user to shift to using > ‘minor'. Send out e-mail to user list. > > September > Remove the ‘inorder' code from the repository. Another e-mail announcement > to both user and dev list. > > October > Rename ‘minor' to ‘inorder'. > > What are your thoughts on this timeline? Does it all sounds reasonable? > > Thanks, > > Andreas > > -- IMPORTANT NOTICE: The contents of this email and any attachments are > confidential and may also be privileged. If you are not the intended > recipient, please notify the sender immediately and do not disclose the > contents to any other person, use it for any purpose, or store or copy the > information in any medium. Thank you. > > ARM Limited, Registered office 110 Fulbourn Road, Cambridge CB1 9NJ, > Registered in England & Wales, Company No: 2557590 > ARM Holdings plc, Registered office 110 Fulbourn Road, Cambridge CB1 9NJ, > Registered in England & Wales, Company No: 2548782 > _______________________________________________ > gem5-dev mailing list > [email protected] > http://m5sim.org/mailman/listinfo/gem5-dev > -- - Korey _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
