----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/1596/#review3780 -----------------------------------------------------------
Ship it! LGTM I suppose the summary should be "config:" as opposed to "Config:", but that's all - Andreas Hansson On Jan. 4, 2013, 8:52 a.m., Malek Musleh wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/1596/ > ----------------------------------------------------------- > > (Updated Jan. 4, 2013, 8:52 a.m.) > > > Review request for Default. > > > Description > ------- > > Changeset 9376:fba1eda1cb0f > --------------------------- > Config: Move Ruby Objs to RubySystem in obj hierarchy > > This patch moves the contollers to be children of the ruby_system instead of > 'system' under the python object hierarchy. This is so that these objects can > inherit some of the ruby_system's parameter values without resorting to > calling a global system pointer during run-time. The Ruby Objects should be > direct children of the ruby_system instead of the main system object. > > Testing: > > > Diffs > ----- > > configs/ruby/MESI_CMP_directory.py 26ba525347fe980eda39e56fcb61e58c202402df > configs/ruby/MI_example.py 26ba525347fe980eda39e56fcb61e58c202402df > configs/ruby/MOESI_CMP_directory.py > 26ba525347fe980eda39e56fcb61e58c202402df > configs/ruby/MOESI_CMP_token.py 26ba525347fe980eda39e56fcb61e58c202402df > configs/ruby/MOESI_hammer.py 26ba525347fe980eda39e56fcb61e58c202402df > > Diff: http://reviews.gem5.org/r/1596/diff/ > > > Testing > ------- > > > Thanks, > > Malek Musleh > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
