----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3013/#review7161 -----------------------------------------------------------
Hi Nilay, I plan on posting a patch which enables multi-instance Ruby; it will come with a configuration file that can be used to test the functionality. So far, the patch works well enough with multi-instance that I have not noticed problems. However, this specific issue with the two static variables in this patch still exists; we must have unintentionally missed it. I do however think that it can be obviated by using the main method that we used in our patch. (Essentially, everything that was static before has been moved into the per-instance RubySystem object or obviated in some other way.) I'm am currently rebasing my patches from an older revision to the tip of the tree and encountering some problems with more recent Ruby patches. As soon as those are worked out, I will post my patch and we can discuss the merits/demerits. As a forewarning, the solution is not elegant but it does work. I'm sure that we can refactor it into something cleaner if necessary. - Brandon Potter On Aug. 9, 2015, 6:26 p.m., Nilay Vaish wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/3013/ > ----------------------------------------------------------- > > (Updated Aug. 9, 2015, 6:26 p.m.) > > > Review request for Default. > > > Repository: gem5 > > > Description > ------- > > Changeset 11050:8f8fc7ddc278 > --------------------------- > ruby: convert global statistical variables to local ones. > Several statistical quantities collected by ruby where recorded / published > using global variables. Since we want to have multiple ruby systems working > together, this is a hindrance. Hence, we are convert such global variables to > local ones. > > > Diffs > ----- > > src/mem/ruby/slicc_interface/AbstractController.hh 863d314f6356 > src/mem/ruby/slicc_interface/AbstractController.cc 863d314f6356 > src/mem/slicc/symbols/StateMachine.py 863d314f6356 > > Diff: http://reviews.gem5.org/r/3013/diff/ > > > Testing > ------- > > > Thanks, > > Nilay Vaish > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
