----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/1214/#review2795 -----------------------------------------------------------
Ship it! Seems fine, but I wonder about the new/free vs memcpy for the stl structs. - Ali Saidi On May 23, 2012, 6:31 a.m., Andreas Hansson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/1214/ > ----------------------------------------------------------- > > (Updated May 23, 2012, 6:31 a.m.) > > > Review request for Default. > > > Description > ------- > > Changeset 9020:a6bf1fdde3bb > --------------------------- > Bridge: Split deferred request, response and sender state > > This patch splits the PacketBuffer class into a RequestState and a > DeferredRequest and DeferredResponse. Only the requests need a > SenderState, and the deferred requests and responses only need an > associated point in time for the request and the response queue. > > Besides the cleaning up, the goal is to simplify the transition to a > new port handshake, and with these changes, the two packet queues are > starting to look very similar to the generic packet queue, but > currently they do a few unique things relating to the NACK and > counting of requests/responses that the packet queue cannot be > conveniently used. This will be addressed in a later patch. > > > Diffs > ----- > > src/mem/bridge.hh 4fbbd05809d2 > src/mem/bridge.cc 4fbbd05809d2 > > Diff: http://reviews.gem5.org/r/1214/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
