> On Sept. 21, 2013, 8:39 a.m., Andreas Hansson wrote: > > src/sim/eventq_impl.hh, line 55 > > <http://reviews.gem5.org/r/1667/diff/9/?file=37731#file37731line55> > > > > is it worth checking inParallelMode first? > > > > As a side not, do we know what the performance impact is when not > > running in parallel mode? (perhaps this has been resolved already) > > Nilay Vaish wrote: > I'll run the regression tests and report the execution time differences > soon. > > Andreas Hansson wrote: > My _very_ early results suggest around -15% host_inst_rate for most of > the regressions, but I'll get back to you tomorrow with a more conclusive > result. > > Andreas Hansson wrote: > Ok, so a full regression run, including compilation of all ISAs is now > 18% slower. This is a fairly accurate measure of the slowdown, as the run > time is within +-10 min over the last months. > > This suggests actual simulation is more than 18% slower, which obviously > is a _big_ issue. I haven't done any profiling, but will keep you posted on > any findings.
That is painful. I hope that we can identify the source of the overhead and solve it with (at worst) some special-casing on (numEventQueues == 1) that isn't too intrusive or ugly. Thanks for thinking of this... it didn't even occur to me to check. - Steve ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/1667/#review4727 ----------------------------------------------------------- On Sept. 21, 2013, 11:36 a.m., Nilay Vaish wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/1667/ > ----------------------------------------------------------- > > (Updated Sept. 21, 2013, 11:36 a.m.) > > > Review request for Default. > > > Repository: gem5 > > > Description > ------- > > Changeset 9885:ab020183761b > --------------------------- > sim: simulate with multiple event queues > This patch extends the patch Steve posted on the reviewboard (846). The patch > updated with all the changes that have taken place over last 15 months. Code > has been added so as actually carry out a quantum-based parallel simulation. > > The patch was tested in two different configurations: > 1. ruby_network_test.py: in this simulation L1 cache controllers receive > requests from the cpu. The requests are replied to immediately without > any communication taking place with any other level. > 2. twosys-tsunami-simple-atomic: this configuration simulates a client-server > system which are connected by an ethernet link. > > We still lack the ability to communicate using message buffers or ports. But > other things like simulation start and end, synchronizing after every quantum > seem to be working. > > > Diffs > ----- > > src/SConscript 372d3611c693 > src/base/barrier.hh PRE-CREATION > src/cpu/base.cc 372d3611c693 > src/dev/etherlink.cc 372d3611c693 > src/python/m5/SimObject.py 372d3611c693 > src/python/m5/event.py 372d3611c693 > src/python/m5/main.py 372d3611c693 > src/python/m5/simulate.py 372d3611c693 > src/python/swig/event.i 372d3611c693 > src/sim/Root.py 372d3611c693 > src/sim/SConscript 372d3611c693 > src/sim/core.hh 372d3611c693 > src/sim/core.cc 372d3611c693 > src/sim/debug.cc 372d3611c693 > src/sim/eventq.hh 372d3611c693 > src/sim/eventq.cc 372d3611c693 > src/sim/eventq_impl.hh 372d3611c693 > src/sim/global_event.hh PRE-CREATION > src/sim/global_event.cc PRE-CREATION > src/sim/root.cc 372d3611c693 > src/sim/serialize.hh 372d3611c693 > src/sim/serialize.cc 372d3611c693 > src/sim/sim_events.hh 372d3611c693 > src/sim/sim_events.cc 372d3611c693 > src/sim/sim_exit.hh 372d3611c693 > src/sim/sim_object.cc 372d3611c693 > src/sim/simulate.hh 372d3611c693 > src/sim/simulate.cc 372d3611c693 > src/sim/stat_control.cc 372d3611c693 > > Diff: http://reviews.gem5.org/r/1667/diff/ > > > Testing > ------- > > > Thanks, > > Nilay Vaish > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
