----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3115/#review7225 -----------------------------------------------------------
src/mem/ruby/system/RubyPort.cc (line 512) <http://reviews.gem5.org/r/3115/#comment6146> Why call new here in the first place? - Andreas Hansson On Sept. 16, 2015, 6:06 p.m., Joel Hestness wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/3115/ > ----------------------------------------------------------- > > (Updated Sept. 16, 2015, 6:06 p.m.) > > > Review request for Default. > > > Repository: gem5 > > > Description > ------- > > Changeset 11092:03402924ac1e > --------------------------- > ruby: RubyPort delete snoop requests > > In RubyPort::ruby_eviction_callback, prior changes fixed a memory leak caused > by instantiating separate packets for each port that the eviction was > forwarded > to. That change, however, left the instantiated request to also leak. Delete > the request. > > > Diffs > ----- > > src/mem/ruby/system/RubyPort.cc 62e1504b9c64 > > Diff: http://reviews.gem5.org/r/3115/diff/ > > > Testing > ------- > > Compiled gem5.debug with --without-tcmalloc. Ran large tests with Valgrind. > > > Thanks, > > Joel Hestness > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
