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

[Carsten Varming]

Dear Jon,

I am pretty sure this has been that way for a long time. I have never seen
a problem, but I would like to see Hotspot shut down its threads as well.

Carsten

On Tue, Nov 3, 2015 at 5:40 PM, Jon Masamitsu <jon.masamitsu at oracle.com>
wrote:

> Jungwoo,
>
> Did you first see this bug in jdk8?  I'm wondering if it
> was always broken.
>
> 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/
>
> 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
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.org/pipermail/hotspot-gc-dev/attachments/20151103/f6e296ee/attachment.htm>