----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3809/#review9426 -----------------------------------------------------------
Ship it! Ship It! - Jason Lowe-Power On Feb. 12, 2017, 4:36 a.m., Tushar Krishna wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/3809/ > ----------------------------------------------------------- > > (Updated Feb. 12, 2017, 4:36 a.m.) > > > Review request for Default, Brad Beckmann, Jieming Yin, Matthew Poremba, and > Onur Kayiran. > > > Repository: gem5 > > > Description > ------- > > ruby: fix round robin arbiter in garnet2.0 > The rr arbiter pointer in garnet was getting updated on every request, > even if there is no grant. This was leading to a huge variance in wait > time at a router at high injection rates. > This patch corrects it to update upon a grant. > > > Diffs > ----- > > src/mem/ruby/network/garnet2.0/SwitchAllocator.cc 79c3f6a60392 > > Diff: http://reviews.gem5.org/r/3809/diff/ > > > Testing > ------- > > > Thanks, > > Tushar Krishna > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
