-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviews.gem5.org/r/2504/#review5482
-----------------------------------------------------------


Please ignore my last review. I made a mistake with my patches.

In FS, X86 mode I was able to boot with python variant, checkpoint, run a short 
program. Then I was able to restore from checkpoint and run the same program 
again. And in systemc variant then I was able to restore from checkpoint and 
run the same program.

The exit was not clean though. It failed with
fatal: Missed an event at time 123366500 gem5: 5699875688000, SystemC: 
5699875688000  @ tick 5699875688000
[eventLoop:sc_module.cc, line 192]

But that may be due to m5_exit in the test program.


- Cagdas Dirik


On Nov. 17, 2014, 6:19 a.m., Andreas Hansson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviews.gem5.org/r/2504/
> -----------------------------------------------------------
> 
> (Updated Nov. 17, 2014, 6:19 a.m.)
> 
> 
> Review request for Default.
> 
> 
> Repository: gem5
> 
> 
> Description
> -------
> 
> Changeset 10555:d1e51dc6cf86
> ---------------------------
> config: Fix to SystemC example's event handling
> 
> This patch fixes checkpoint restore in the SystemC hosting example by handling
> early PollEvent events correctly before any EventQueue events are posted.
> 
> The SystemC event queue handler (SCEventQueue) reports an error if the event
> loop is entered with no Events posted.  It is possible for this to happen
> after instantiate due to PollEvent events.  This patch separates out
> `external' events into a different handler in sc_module.cc to prevent the
> error from occurring.
> 
> 
> Diffs
> -----
> 
>   util/systemc/Makefile 1a9e235cab09 
>   util/systemc/main.cc 1a9e235cab09 
>   util/systemc/sc_module.hh 1a9e235cab09 
>   util/systemc/sc_module.cc 1a9e235cab09 
> 
> Diff: http://reviews.gem5.org/r/2504/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Andreas Hansson
> 
>

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

Reply via email to