> On 2011-04-29 15:27:14, Tushar Krishna wrote: > > src/cpu/testers/networktest/networktest.cc, line 213 > > <http://reviews.m5sim.org/r/660/diff/1/?file=12042#file12042line213> > > > > 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
I don't think there is an easy way to support the fixed-pkts option based on the current NetworkTest implementation. Ending the simulation as soon as the tester injects maxPackets seems likek the only reasonable thing to do. Unless we can remove the fixed-pkts option all together. The other options seem like brittle hacks. If you want to spend a bit more time on it, I suspect we could provide the directory a pointer to the NetworkTest object and have the directory call say a NetworkTest.finish() function. - Brad ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/660/#review1172 ----------------------------------------------------------- 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
