> On Sept. 14, 2015, 6:14 p.m., Andreas Hansson wrote: > > Hi Erfan, > > > > I was thinking of something much simpler for the bandwidth throttling, much > > like what the SimpleMemory is doing, but here we would do it for the > > "popping" from the send queue. Thus, the incoming packets would still just > > look at the queues being full or not. We would simply throttle the send > > events based on the lane bandwidth (you can use Param.MemoryBandwdith for > > that), and the size of each flit/packet. There should be no need for any > > rounding or epochs. Simply do not schedule the next send event straight > > away, but wait enough ticks to make up for the message we just sent. Does > > that make sense?
Hello Andreas, Thanks for your comment. I have removed throttling and fixed the problem in trySendTiming(). Now the measured bandwidth on the request path is exactly equal to its theoretical limit. You can verify this by adding the debug flag: SerialLink Param.MemoryBandwidth was not required, instead bandwidth is defined by these parameters: link_frequency, num_serial_links, num_lanes_per_link. Next, I will look at the response path and see if I can do something similar for it. - Erfan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2993/#review7173 ----------------------------------------------------------- On Sept. 15, 2015, 8:46 a.m., Erfan Azarkhish wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/2993/ > ----------------------------------------------------------- > > (Updated Sept. 15, 2015, 8:46 a.m.) > > > Review request for Default. > > > Repository: gem5 > > > Description > ------- > > Changeset 11098:6ce558162154 > --------------------------- > Serial Link model for the HMC > > > Diffs > ----- > > src/mem/SConscript efaacec43726 > src/mem/SerialLink.py PRE-CREATION > src/mem/serial_link.hh PRE-CREATION > src/mem/serial_link.cc PRE-CREATION > > Diff: http://reviews.gem5.org/r/2993/diff/ > > > Testing > ------- > > > Thanks, > > Erfan Azarkhish > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
