RFR: 8293972: runtime/NMT/NMTInitializationTest.java#default_long-off failed with "Suspiciously long bucket chains in lookup table." [v4]

Thu Jul 20 22:28:01 UTC 2023

On Thu, 20 Jul 2023 07:14:08 GMT, David Holmes <dholmes at openjdk.org> wrote:

>> src/hotspot/share/runtime/threads.cpp line 458:
>> 
>>> 456:   if (parse_result != JNI_OK) return parse_result;
>>> 457: 
>>> 458:   // Verify NMT flag(s) and log the initial state to the ouput
>> 
>> Nit: So far in this PR you have used the terms "NMT property", "NMT setting", "NMT arguments", and now "NMT flag(s)". Please pick one (not "property") and use consistently.
>
> Typo: ouput

Fixed.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/14607#discussion_r1270020894