RFR: 8266807: Windows os_windows-gtest broken for UseLargePages

Stefan Johansson sjohanss at openjdk.java.net
Tue May 18 12:14:58 UTC 2021


Please review this change to fix the large page testing on Windows.

**Summary**
The comparisons in the test check for reservations `!= NULL` and consider that a failed reservation, that is incorrect. This leads to the test never really doing anything. When `UseLargePages` is false, the test is skipped and when true and the user is configured with lock permissions (needed to use large pages), the test is skipped after the first allocation (since it is wrongly seen as a failure).

When a user doesn't have permissions to lock memory, the JVM will turn off large pages so this will be the same as running with `-UseLargePages`. 

The test has been changed to skip testing if the first large page reservation fail, similar to what was intended before. After that the rest of the reservations are expected to pass, apart from the reservation trying to reserve in an already existing reservation.

**Testing**
Manual testing on a Windows host with and without large pages enabled. Also tested through mach5.

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

Commit messages:
 - Remove log-line.
 - 8266807: Windows os_windows-gtest broken for UseLargePages

Changes: https://git.openjdk.java.net/jdk/pull/4090/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=4090&range=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8266807
  Stats: 22 lines in 1 file changed: 10 ins; 1 del; 11 mod
  Patch: https://git.openjdk.java.net/jdk/pull/4090.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/4090/head:pull/4090

PR: https://git.openjdk.java.net/jdk/pull/4090


More information about the hotspot-runtime-dev mailing list