----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/915/#review1700 -----------------------------------------------------------
src/mem/packet.hh <http://reviews.m5sim.org/r/915/#comment2181> Ok maybe I got things confused here. Is the idea that you can have an address that is not aligned to any word (e.g. only byte aligned) and that any bytes could be valid within the burst (hence the mask)? Do the bytes have to come in whole words or can they be partial? Do the valid bytes all reside at the beginning of the burst, i.e. from the start address or could they be anywhere? I am merely trying to understand if this is indeed a byte enable (and if so how organised) or if it is more a way of having smaller than "getBlockSize()" bursts. Does that make sense? - Andreas On 2011-11-28 09:05:53, Geoffrey Blake wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.m5sim.org/r/915/ > ----------------------------------------------------------- > > (Updated 2011-11-28 09:05:53) > > > Review request for Default, Ali Saidi, Gabe Black, Steve Reinhardt, and > Nathan Binkert. > > > Summary > ------- > > Packet: Enable functional reads of partial data to packet class > > This patch fixes a long standing defficiency in the packet class where > it was unable to handle finding data that partially satisfied a request. > > This splits out changes made to the packet class in the checkercpu patch as > requested by Ali. > > > Diffs > ----- > > src/mem/packet.hh 62dee0c98d53 > src/mem/packet.cc 62dee0c98d53 > > Diff: http://reviews.m5sim.org/r/915/diff > > > Testing > ------- > > Compiles. No functional changes made from CheckerCPU patch to this patch for > packet class, and CheckerCPU fully exercised this code path during testing. > > > Thanks, > > Geoffrey > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
