----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2494/ -----------------------------------------------------------
(Updated Nov. 30, 2014, 9:24 a.m.) Review request for Default. Repository: gem5 Description (updated) ------- Changeset 10567:4b6d25498f4b --------------------------- mem: Assume all dynamic packet data is array allocated This patch simplifies how we deal with dynamically allocated data in the packet, always assuming that it is array allocated, and hence should be array deallocated (delete[] as opposed to delete). The only uses of dataDynamic was in the Ruby testers, and these are now changed to use Packet::allocate or dataDynamicArray as appropriate. The ARRAY_DATA flag in the packet is removed accordingly. No defragmentation of the flags is done at this point, leaving a gap in the bit masks. Going forward I would suggest a name change to better reflect the semantics, perhaps: dataStatic -> dataNoFree dataDynamic -> dataToFree Diffs (updated) ----- src/cpu/minor/lsq.cc dd04eb06ad42 src/cpu/o3/fetch_impl.hh dd04eb06ad42 src/cpu/simple/timing.cc dd04eb06ad42 src/cpu/testers/directedtest/InvalidateGenerator.cc dd04eb06ad42 src/cpu/testers/directedtest/SeriesRequestGenerator.cc dd04eb06ad42 src/cpu/testers/memtest/memtest.cc dd04eb06ad42 src/cpu/testers/networktest/networktest.cc dd04eb06ad42 src/cpu/testers/rubytest/Check.cc dd04eb06ad42 src/cpu/testers/traffic_gen/generators.cc dd04eb06ad42 src/mem/packet.hh dd04eb06ad42 Diff: http://reviews.gem5.org/r/2494/diff/ Testing ------- Thanks, Andreas Hansson _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
