These patches don't apply cleanly. Also, do these changes apply to the fixed pipeline as well? The whole fixed/flexible thing grates on me because there is so much code duplication in there. Can someone fix that?
Nate On Fri, Mar 11, 2011 at 9:33 AM, Tushar Krishna <tus...@csail.mit.edu>wrote: > This is an automatically generated e-mail. To reply, visit: > http://reviews.m5sim.org/r/562/ > Review request for Default, Ali Saidi, Gabe Black, Steve Reinhardt, > Nathan Binkert, and Brad Beckmann. > By Tushar Krishna. > Description > > minor fixes to garnet flexible pipeline code > > Diffs > > - src/mem/ruby/network/garnet/flexible-pipeline/GarnetNetwork.hh > (04ced731345f) > - src/mem/ruby/network/garnet/flexible-pipeline/GarnetNetwork.cc > (04ced731345f) > - src/mem/ruby/network/garnet/flexible-pipeline/NetworkInterface.cc > (04ced731345f) > - src/mem/ruby/network/garnet/flexible-pipeline/Router.cc > (04ced731345f) > - src/mem/ruby/network/garnet/flexible-pipeline/flit.hh (04ced731345f) > - src/mem/ruby/network/garnet/flexible-pipeline/flit.cc (04ced731345f) > > View Diff <http://reviews.m5sim.org/r/562/diff/> > _______________________________________________ m5-dev mailing list m5-dev@m5sim.org http://m5sim.org/mailman/listinfo/m5-dev