> On 2011-04-27 16:03:25, Tushar Krishna wrote: > > src/mem/ruby/network/garnet/fixed-pipeline/GarnetLink_d.py, line 44 > > <http://reviews.m5sim.org/r/661/diff/1/?file=12097#file12097line44> > > > > In garnet, the link width is actually equal to flit size (i.e. the > > entire flit is injected into the link). [bigger packets such as data are > > thereby broken into multiple flits]. > > So not sure if having both a "bandwidth_factor" and a "flit_size" > > parameter inside BaseGarnetNetwork.py will work. > > Perhaps we should set flit_size equal to bandwidth_factor there? > > "link_bandwidth" is a clearer term but I understand that is not true > > for simple network where the endpoint bandwidth comes into play... > > Brad Beckmann wrote: > Yeah, I don't want to call it link_bandwidth or simply bandwidth because > the multipliers do impact the final bandwidth for the simple network. > > The confusion I have, and the reason why I changed it, is before Garnet > links could be of only one width, even though one would specify links of > different widths in the topology. Can we remove the flit _size parameter in > BaseGarnetNetwork and just rely on the per link bandwidths (i.e. > channel_width)? Right now, only the NetworkInterfaces use the flit_size > parameter and it seems more natural to have that be a configuration parameter > for just the NIs and not the entire network. Maybe that is a good reason to > make the NIs sim_objects. > > What do you think? That seems like the right solution to me, but I would > like to know your thoughts before I spend time implementing it. >
Garnet actually does not use the bw parameters from the topology files at all, and just uses the latency ones. You are right in that only the NI needs the flit size parameter when creating flits. However, the routers inject the complete flit into the link. The number of cycles it takes in the link depends upon link_latency. If we use the channel bandwidth parameter, then a flit size of 16, and a channel width of 8 (=> phit), requires that the flit should take 2 cycles to cross the link. But if the link latency is say 3 cycles, then the flit should take 2x3 = 6 cycles. Currently Garnet does not implement this. The flit would take 3-cycles in this case. Do you want me to modify garnet to implement this? - Tushar ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/661/#review1161 ----------------------------------------------------------- On 2011-04-27 10:38:47, Brad Beckmann wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.m5sim.org/r/661/ > ----------------------------------------------------------- > > (Updated 2011-04-27 10:38:47) > > > Review request for Default, Ali Saidi, Gabe Black, Steve Reinhardt, and > Nathan Binkert. > > > Summary > ------- > > network: generalized bw for basic links > > Generalized the specification of bw for basic links. This value is then > translated to the specific value used by the simple and Garnet networks. > > > Diffs > ----- > > configs/ruby/Ruby.py b02bca5aed04 > src/mem/ruby/network/BasicLink.hh PRE-CREATION > src/mem/ruby/network/BasicLink.cc PRE-CREATION > src/mem/ruby/network/BasicLink.py PRE-CREATION > src/mem/ruby/network/garnet/fixed-pipeline/GarnetLink_d.py PRE-CREATION > src/mem/ruby/network/simple/SConscript b02bca5aed04 > src/mem/ruby/network/simple/SimpleLink.hh PRE-CREATION > src/mem/ruby/network/simple/SimpleLink.cc PRE-CREATION > src/mem/ruby/network/simple/SimpleLink.py PRE-CREATION > src/mem/ruby/network/simple/SimpleNetwork.cc b02bca5aed04 > > Diff: http://reviews.m5sim.org/r/661/diff > > > Testing > ------- > > > Thanks, > > Brad > > _______________________________________________ m5-dev mailing list m5-dev@m5sim.org http://m5sim.org/mailman/listinfo/m5-dev
