RFR: JDK-8266795: Remove dead code LowMemoryDetectorDisabler
David Holmes
dholmes at openjdk.java.net
Tue May 11 02:33:54 UTC 2021
On Tue, 11 May 2021 01:16:27 GMT, Hamlin Li <mli at openjdk.org> wrote:
> This is trivial change to remove dead code in low memory detector.
Hi Hamlin,
What you have seems fine, but unless I missed something it seems the whole notion of being enabled/disabled is unused:
// > 0 if temporary disabed
static volatile jint _disabled_count;
static bool temporary_disabled() { return _disabled_count > 0; }
static void disable() { Atomic::inc(&_disabled_count); }
static void enable() { Atomic::dec(&_disabled_count); }
Thanks,
David
-------------
Changes requested by dholmes (Reviewer).
PR: https://git.openjdk.java.net/jdk/pull/3964
More information about the hotspot-runtime-dev
mailing list