On Tue, 14 Apr 2015, Joel Hestness wrote:

This patch didn't address my requested change (
http://reviews.gem5.org/r/2702/), and shouldn't have been checked in. The
AbstractController should not have or even need another pointer to the
RubySystem, because it can access it with g_system_ptr. This is a dead
simple fix. Adding the pointer is going in the wrong direction for
abstraction.


Joel, in my opinion we should be eliminating global / static variables. I have personally found them to be a problem while trying to parallelize Ruby.

--
Nilay
_______________________________________________
gem5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/gem5-dev

Reply via email to