RFR(S): 8141356: Explicitly stop CMS threads during VM termination.

Jon Masamitsu jon.masamitsu at oracle.com
Thu Nov 5 16:00:51 UTC 2015


Jungwoo,

Change looks good.  I'll sponsor.

I'll need to get 1 more review.

Jon

On 11/03/2015 01:14 PM, Jungwoo Ha wrote:
> BUG: https://bugs.openjdk.java.net/browse/JDK-8141356
> Webrev: http://cr.openjdk.java.net/~jwha/8141356/webrev.00/ 
> <http://cr.openjdk.java.net/%7Ejwha/8141356/webrev.00/>
>
> Inside before_exit(), it says
>
>   // Stop concurrent GC threads
> Universe::heap()->stop();
>
> but GenCollectedHeap never implemented stop() method, and thus calling 
> empty SharedHeap::stop() method.
> This causes CMS threads to run during the VM termination and ends up 
> crashing.
> ConcurrentMarkSweepThread::stop() is currently a dead code and never 
> gets called.
> The patch just implemented GenCollectedHeap::stop() to call 
> ConcurrentMarkSweepThread::stop().
>
> Can someone sponsor this patch?
>
> -- 
> Jungwoo Ha | Google | jwha at google.com <mailto:jwha at google.com>
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.org/pipermail/hotspot-gc-dev/attachments/20151105/398e2c94/attachment.htm>


More information about the hotspot-gc-dev mailing list