----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/1265/#review2941 -----------------------------------------------------------
Ship it! Ship It! - Ali Saidi On June 11, 2012, 3:36 a.m., Andreas Hansson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/1265/ > ----------------------------------------------------------- > > (Updated June 11, 2012, 3:36 a.m.) > > > Review request for Default. > > > Description > ------- > > Changeset 9073:6bb29a29c452 > --------------------------- > Bus: Add a notion of layers to the buses > > This patch moves all flow control, arbitration and state information > into a bus layer. The layer is thus responsible for all the state > transitions, and for keeping hold of the retry list. Consequently the > layer is also responsible for the draining. > > With this change, the non-coherent and coherent bus are given a single > layer to avoid changing any temporal behaviour, but the patch opens up > for adding more layers. > > > Diffs > ----- > > src/mem/bus.hh 35ac3a6f8ee0 > src/mem/bus.cc 35ac3a6f8ee0 > src/mem/coherent_bus.hh 35ac3a6f8ee0 > src/mem/coherent_bus.cc 35ac3a6f8ee0 > src/mem/noncoherent_bus.hh 35ac3a6f8ee0 > src/mem/noncoherent_bus.cc 35ac3a6f8ee0 > > Diff: http://reviews.gem5.org/r/1265/diff/ > > > Testing > ------- > > util/regress all passing (disregarding t1000 and eio) > > > Thanks, > > Andreas Hansson > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
