RFR(S): 8040807: G1: Enable G1CollectedHeap::stop()
Jon Masamitsu
jon.masamitsu at oracle.com
Tue May 20 14:45:30 UTC 2014
Looks good.
Reviewed.
Jon
On 05/20/2014 04:59 AM, Per Liden wrote:
> Looking for a couple of reviews in this patch.
>
> Summary: This patch re-enables the controlled stopping of G1's
> concurrent threads at VM shutdown. This could potentially cause hangs
> during VM shutdown because the G1 marking threads could get stuck in
> various places and fail to terminate. JDK-8040803 and JDK-8040804
> fixed these issues, so this is the final step to re-enable the actual
> stopping of those threads. This patch also moves the call to
> CollectedHeap::stop() a few lines down to group the GC related stuff
> together. It also adjusts/removes some comments that are no longer
> correct.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8040807
> Webrev: http://cr.openjdk.java.net/~pliden/8040807/webrev.0/
>
> Testing:
> - GC nightlies. 5 tests in this suite used to timeout because of the
> issue with hanging threads. They now pass.
> - JPRT
>
> Thanks!
> /Per
More information about the hotspot-gc-dev
mailing list