----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/660/#review1172 -----------------------------------------------------------
src/cpu/testers/networktest/networktest.cc <http://reviews.m5sim.org/r/660/#comment1608> Hey Brad, One concern that I have with my current implementation here is that if --fixed-pkts is enabled, then the tester is not scheduled after injecting maxPackets. Once all packets are delivered (=> no more events from the network side as well), there is no agent to call exitSimLoop to end the simulation, so it ends at m5's max tick (9223372036854775807) cycles. This does not slow down the simulation (since there are no events after all delivery), and is ok. But it does screw up the power stats etc which use Ruby_cycles, and sort of looks ugly since m5 prints out that the simulation ended after 9223372036854775807 cycles. One option is to end the simulation as soon as the tester injects maxPackets, but that wont result in all packets getting delivered, defeating the purpose of fixed-pkts which I added for network debugging. The other option is to keep scheduling the tester till simCycles, but simply stopping the generation of packets after maxPackets (this was what I was doing earlier). But this requires the simCycles input to be greater than the time by which all packets are expected to be injected (which depends upon maxPackets and injection rate). [Unlike the ruby random tester, the tester here does not track anything to determine when everything is delivered]. Any suggestions? Thanks, Tushar - Tushar On 2011-04-25 16:18:04, Tushar Krishna wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.m5sim.org/r/660/ > ----------------------------------------------------------- > > (Updated 2011-04-25 16:18:04) > > > Review request for Default, Ali Saidi, Gabe Black, Steve Reinhardt, Nathan > Binkert, and Brad Beckmann. > > > Summary > ------- > > NetworkTest: added sim_cycles parameter to the network tester. > > The network tester terminates after injecting for sim_cycles > (default=1000), instead of having to explicitly pass --maxtick from the > command line as before. If fixed_pkts is enabled, the tester stops > scheduling itself after injecting maxpackets number of packets. > The tester also works with zero command line arguments now. > > > Diffs > ----- > > configs/example/ruby_network_test.py de679a068dd8 > src/cpu/testers/networktest/NetworkTest.py de679a068dd8 > src/cpu/testers/networktest/networktest.hh de679a068dd8 > src/cpu/testers/networktest/networktest.cc de679a068dd8 > > Diff: http://reviews.m5sim.org/r/660/diff > > > Testing > ------- > > > Thanks, > > Tushar > > _______________________________________________ m5-dev mailing list m5-dev@m5sim.org http://m5sim.org/mailman/listinfo/m5-dev
