RFR: 8190925: Deprecate FastTLABRefill and set default off (original) (raw)

Marcus Larsson marcus.larsson at oracle.com
Tue Nov 14 08:57:21 UTC 2017

Looks good!

Thanks, Marcus

On 2017-11-13 14:33, Robbin Ehn wrote:

Hi all, please review.

The option FastTLABRefill work only in C1 (interpreter and c2 don't use this option) when using either SerialGC or ParrallelGC (and deprecated CMS). Both the newly integrated Memory-Leak Profiler and upcoming Low Overhead Profiler don't work when FastTLABRefill is enabled. For more info please see CSR: https://bugs.openjdk.java.net/browse/JDK-8190926 Issue: https://bugs.openjdk.java.net/browse/JDK-8190925 Code below, thanks Robbin! diff -r cf127be65014 src/hotspot/share/runtime/arguments.cpp --- a/src/hotspot/share/runtime/arguments.cpp    Sat Nov 11 01:21:09 2017 +0100 +++ b/src/hotspot/share/runtime/arguments.cpp    Mon Nov 13 14:26:27 2017 +0100 @@ -385,2 +385,3 @@ { "UseMembar",                    JDKVersion::jdk(10), JDKVersion::jdk(11), JDKVersion::jdk(12) }, +  { "FastTLABRefill",               JDKVersion::jdk(10), JDKVersion::jdk(11), JDKVersion::jdk(12) }, { "IgnoreUnverifiableClassesDuringDump", JDKVersion::jdk(10), JDKVersion::undefined(), JDKVersion::undefined() }, diff -r cf127be65014 src/hotspot/share/runtime/globals.hpp --- a/src/hotspot/share/runtime/globals.hpp    Sat Nov 11 01:21:09 2017 +0100 +++ b/src/hotspot/share/runtime/globals.hpp    Mon Nov 13 14:26:27 2017 +0100 @@ -2015,4 +2015,4 @@ __ -  product(bool, FastTLABRefill, _true,                                       _ -          "Use fast TLAB refill _code")                                      _ +  product(bool, FastTLABRefill, _false,                                      _ +          "(Deprecated) Use fast TLAB refill _code")                         _ __

More information about the hotspot-dev mailing list