----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3778/#review9266 -----------------------------------------------------------
Ship it! Tested this and saving/loading checkpoints still seems to work just fine. - Joe Gross On Jan. 9, 2017, 7:53 a.m., Christian Menard wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/3778/ > ----------------------------------------------------------- > > (Updated Jan. 9, 2017, 7:53 a.m.) > > > Review request for Default. > > > Repository: gem5 > > > Description > ------- > > There is a bug in the CxxConfig manager that may lead to Objects beeing > created in the wrong order. I noted the issue, when a System Object used a > wrong FullSystem Flag because it was constructed before the Root object. This > happens when the system has a name that comes before 'root' in aplhabetical > order. Since findObject(name) creates the Object if not yet present, all > objects are constructed in alphabetical order. This may break object > dependencies. > > > This patch fixes the bug by removing the alphabetical ordering. > > > Diffs > ----- > > src/sim/cxx_manager.cc c10c50cb8ac9 > > Diff: http://reviews.gem5.org/r/3778/diff/ > > > Testing > ------- > > > Thanks, > > Christian Menard > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
