Request for Review (XS): 7143760 Memory leak in GarbageCollectionNotifications

Mikael Gerdin mikael.gerdin at
Fri Feb 10 03:29:12 PST 2012

Hi Fred,

I ran the repro on a build with your fix applied and it appears that the leak 
is fixed. Thanks for finding it so quick :)


On Friday 10 February 2012 10.27.30 Frederic Parain wrote:
> Here's a small fix (one line) for CR 7143760  Memory leak in
> GarbageCollectionNotifications
> There's a missing HandleMark at the beginning of the
> GCNotifier::sendNotificatin() method. Without this HandleMark, all
> handles used when creating GC notifications are kept alive causing a
> double leak: in the Java heap and in the thread local handle area of the
> service thread.
> Here's the CR:
> (Warning, the changeset referenced in the CR is not the
> one containing the original bug).
> Here's the webrev:
> Thanks,
> Fred

More information about the hotspot-runtime-dev mailing list