----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/915/#review1749 -----------------------------------------------------------
Made a derived class FunctionalPacket to enable partial functional reads to not induce more overhead in the Packet class as requested by Steve, Nate and Ali. Modified blobHelper() in src/mem/port.cc to use FunctionalPacket instead of Packet. Over areas could use functional packets (src/arch/x86/pagetable_walker.cc and src/cpu/testers/memtest/memtest.cc) but have left them alone because they do not really need to be fixed. Tested by compiling and running using the CheckerCPU that exercises this code path heavily. No bugs found. - Geoffrey On 2011-12-12 12:37:51, Geoffrey Blake wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.m5sim.org/r/915/ > ----------------------------------------------------------- > > (Updated 2011-12-12 12:37:51) > > > 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/SConscript c1ab57ea8805 > src/mem/functional_packet.hh PRE-CREATION > src/mem/functional_packet.cc PRE-CREATION > src/mem/packet.hh c1ab57ea8805 > src/mem/packet.cc c1ab57ea8805 > src/mem/port.hh c1ab57ea8805 > src/mem/port.cc c1ab57ea8805 > > 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
