> On Sept. 21, 2012, 9:17 a.m., Nilay Vaish wrote: > >
Happy with the changes Nilay? - Andreas ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/1429/#review3507 ----------------------------------------------------------- On Sept. 21, 2012, 9:50 a.m., Andreas Hansson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/1429/ > ----------------------------------------------------------- > > (Updated Sept. 21, 2012, 9:50 a.m.) > > > Review request for Default. > > > Description > ------- > > This patch transitions the bus to use the AddrRange operations instead > of directly accessing the start and end. The change facilitates the > move to a more elaborate AddrRange class that also supports address > striping in the bus by specifying interleaving bits in the ranges. > > Two new functions are added to the AddrRange to determine if two > ranges intersect, and if one is a subset of another. The bus > propagation of address ranges is also tweaked such that an update is > only propagated if the bus received information from all the > downstream slave modules. This avoids the iteration and need for the > cycle-breaking scheme that was previously used. > > > Diffs > ----- > > src/base/addr_range.hh 4672c12307a7 > src/mem/bus.hh 4672c12307a7 > src/mem/bus.cc 4672c12307a7 > > Diff: http://reviews.gem5.org/r/1429/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
