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

Kim Barrett kim.barrett at oracle.com
Thu Nov 12 14:56:48 UTC 2015


On Nov 11, 2015, at 12:27 PM, Jon Masamitsu <jon.masamitsu at oracle.com> wrote:
> 
> Jungwoo,
> 
> I reworked the patch just a bit. 
> 
> http://cr.openjdk.java.net/~jmasa/8141356/webrev.00/
> 
> I moved the definition of stop() to the .cpp file.  The .cpp
> file already had an include of concurrentMarkSweepThread.hpp
> and I think we're trying to minimize the include of .hpp files
> into other .hpp files.
> 
> I also added the guard
> #if INCLUDE_ALL_GCS
> 
> There is a build that does not include all the GC
> and the guard is needed in that build.

Reworked version looks good to me.




More information about the hotspot-gc-dev mailing list