Correct! On Nov 6, 2014, at 12:57 PM, Ali Saidi <sa...@umich.edu<mailto:sa...@umich.edu>> wrote:
This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2430/ On November 6th, 2014, 7:48 p.m. UTC, Cagdas Dirik wrote: This patch seems to be broken for X86 when restoring from checkpoints. A sample test crashes with segmentation fault. Here are the steps: 0. Sample test program does int array manipulation and creates a checkpoint before computation. 1. Run in se mode with --cpu-type=atomic --mem-type=simple_mem to keep things simple. This will create the checkpoint. 2. Run in se mode with --cpu-type=atomic --mem-type=simple_mem -r 1 and it will run fine. 3. Now run gem5.opt.cxx m5out/config.ini -r m5out/cpt.XYZ and it will seg fault right away at: Simulate:: Entering event queue @ 7088000. Starting simulation... EventQueue (MainEventQueue-0)::schedule event (Event_21) for 18446744073709551615 EventQueue (MainEventQueue-0)::serviceOne event named Event_8 at 7088500 Program received signal SIGSEGV, Segmentation fault. getPage (addr=4195355, this=0x0) at build/X86/cpu/decode_cache.hh:86 86 if (recent[0] != pageMap.end()) { (gdb) Trying it with other cpu-type and other mem-type results in similar seg faults. It looks like this patch is missing some state initialization or some other checkpoint restore functionality for X86. I assume it's working for you without restoring from checkpoints? The code looks like state is being restored, but we've only ever tested it with ARM although I'm not sure why there would be a difference. - Ali On October 9th, 2014, 2:32 p.m. UTC, Andreas Hansson wrote: Review request for Default. By Andreas Hansson. Updated Oct. 9, 2014, 2:32 p.m. Repository: gem5 Description Changeset 10440:faea9dd0b3b1 --------------------------- config: Add the ability to read a config file using C++ and Python This patch adds the ability to load in config.ini files generated from gem5 into another instance of gem5 built without Python configuration support. The intended use case is for configuring gem5 when it is a library embedded in another simulation system. A parallel config file reader is also provided purely in Python to demonstrate the approach taken and to provided similar functionality for as-yet-unknown use models. The Python configuration file reader can read both .ini and .json files. C++ configuration file reading: A command line option has been added for scons to enable C++ configuration file reading: --with-cxx-config There is an example in util/cxx_config that shows C++ configuration in action. util/cxx_config/README explains how to build the example. Configuration is achieved by the object CxxConfigManager. It handles reading object descriptions from a CxxConfigFileBase object which wraps a config file reader. The wrapper class CxxIniFile is provided which wraps an IniFile for reading .ini files. Reading .json files from C++ would be possible with a similar wrapper and a JSON parser. After reading object descriptions, CxxConfigManager creates SimObjectParam-derived objects from the classes in the (generated with this patch) directory build/ARCH/cxx_config CxxConfigManager can then build SimObjects from those SimObjectParams (in an order dictated by the SimObject-value parameters on other objects) and bind ports of the produced SimObjects. A minimal set of instantiate-replacing member functions are provided by CxxConfigManager and few of the member functions of SimObject (such as drain) are extended onto CxxConfigManager. Python configuration file reading (configs/example/read_config.py): A Python version of the reader is also supplied with a similar interface to CxxConfigFileBase (In Python: ConfigFile) to config file readers. The Python config file reading will handle both .ini and .json files. The object construction strategy is slightly different in Python from the C++ reader as you need to avoid objects prematurely becoming the children of other objects when setting parameters. Port binding also needs to be strictly in the same port-index order as the original instantiation. Diffs * SConstruct (148b96b7bc77) * configs/example/read_config.py (PRE-CREATION) * src/SConscript (148b96b7bc77) * src/python/m5/SimObject.py (148b96b7bc77) * src/python/m5/params.py (148b96b7bc77) * src/sim/SConscript (148b96b7bc77) * src/sim/cxx_config.hh (PRE-CREATION) * src/sim/cxx_config.cc (PRE-CREATION) * src/sim/cxx_config_ini.hh (PRE-CREATION) * src/sim/cxx_config_ini.cc (PRE-CREATION) * src/sim/cxx_manager.hh (PRE-CREATION) * src/sim/cxx_manager.cc (PRE-CREATION) * util/cxx_config/Makefile (PRE-CREATION) * util/cxx_config/README (PRE-CREATION) * util/cxx_config/main.cc<http://main.cc> (PRE-CREATION) * util/cxx_config/stats.hh (PRE-CREATION) * util/cxx_config/stats.cc<http://stats.cc> (PRE-CREATION) View Diff<http://reviews.gem5.org/r/2430/diff/> _______________________________________________ gem5-dev mailing list gem5-dev@gem5.org http://m5sim.org/mailman/listinfo/gem5-dev