misc: Fix order of object construction in the CxxConfigManager
authorChristian Menard <Christian.Menard@tu-dresden.de>
Fri, 10 Feb 2017 00:11:23 +0000 (19:11 -0500)
committerChristian Menard <Christian.Menard@tu-dresden.de>
Fri, 10 Feb 2017 00:11:23 +0000 (19:11 -0500)
commit5fd959260cbcc35f1f5187321f5c473f04647bf6
tree7333743f59b71f5611c6deb038ebb6cdce319c79
parentf4b14c73fc1c07b2c4658b4275ad121a6b6346d8
misc: Fix order of object construction in the CxxConfigManager

The CxxConfigManager schould create objects by traversing the object tree
starting from the root object. However, currently objects are created in
aplphabetical order, which only works if the root object alphabetically
comes before any system object (e.g. 'root' < 'system'. Otherwise (e.g.
'a_system' < 'root'), object construction may fail. The reason for this
behaviour is, that the call to findObject() in the sorting code also
constructs the object if it is not yet existent. Then findTraversalOrder()
calls findObject("root") and subseqeuently calls findObject() on all the
children, and so on. However, the call to findTraversalOrder() is
redundant, since all objects are already created in alphabetical order.
This patch simply removes the alphabetical ordering, leading to the objects
being created starting from 'root'.

Reviewed at http://reviews.gem5.org/r/3778/

Signed-off-by: Jason Lowe-Power <jason@lowepower.com>
src/sim/cxx_manager.cc