----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/1312/#review3220 -----------------------------------------------------------
Ship it! Ship It! - Steve Reinhardt On Aug. 7, 2012, 3:39 a.m., Andreas Hansson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/1312/ > ----------------------------------------------------------- > > (Updated Aug. 7, 2012, 3:39 a.m.) > > > Review request for Default. > > > Description > ------- > > Changeset 9155:eb11628d78be > --------------------------- > Port: Extend the QueuedPort interface and use where appropriate > > This patch extends the queued port interfaces with methods for > scheduling the transmission of a timing request/response. The methods > are named similar to the corresponding sendTiming(Snoop)Req/Resp, > replacing the "send" with "sched". As the queues are currently > unbounded, the methods always succeed and hence do not return a value. > > This functionality was previously provided in the subclasses by > calling PacketQueue::schedSendTiming with the appropriate > parameters. With this change, there is no need to introduce these > extra methods in the subclasses, and the use of the queued interface > is more uniform and explicit. > > > Diffs > ----- > > src/dev/x86/intdev.cc a61fdbbc1d45 > src/mem/cache/base.hh a61fdbbc1d45 > src/mem/cache/cache_impl.hh a61fdbbc1d45 > src/mem/packet_queue.cc a61fdbbc1d45 > src/mem/qport.hh a61fdbbc1d45 > src/mem/ruby/system/RubyPort.hh a61fdbbc1d45 > src/mem/ruby/system/RubyPort.cc a61fdbbc1d45 > src/mem/tport.cc a61fdbbc1d45 > > Diff: http://reviews.gem5.org/r/1312/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
