config: SystemC Gem5Control top level additions
Review Request #2503 - Created Nov. 16, 2014 and submitted
| Information | |
|---|---|
| Andreas Hansson | |
| gem5 | |
| default | |
| Reviewers | |
| Default | |
Changeset 10554:f6e35a3dcc8f
---------------------------
config: SystemC Gem5Control top level additions
This patch cleans up a few style issues and adds a few capabilities to the
SystemC top level 'Gem5Control/Gem5System' mechanism. These include:
1) A space to store/retrieve a version string for a model
2) A mechanism for registering functions to be called at the end of
elaboration to perform simulation setup tasks in SystemC
3) Adding setGDBRemotePort to the Gem5Control
4) Changing the sc_set_time_resolution behaviour to instead check that
the SystemC time resolution is already acceptable
I don't know what are we trying to achieve by interfacing gem5 and SystemC. But the patch seems fine to me.
-
util/systemc/sc_gem5_control.hh (Diff revision 1) -
What is this version for?
-
util/systemc/sc_gem5_control.hh (Diff revision 1) -
Can you elaborate on what an elaboration is? I am guessing it to be a SystemC term.
-
util/systemc/sc_gem5_control.cc (Diff revision 1) -
I am not against this indentation, but I think the norm is to align the arguments vertically.
