RFR: 8266807: Windows os_windows-gtest broken for UseLargePages [v2]

Stefan Johansson sjohanss at openjdk.java.net
Fri May 28 12:28:22 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.

Stefan Johansson has updated the pull request incrementally with one additional commit since the last revision:

  Albert review, updated failure message.

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

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/4090/files
  - new: https://git.openjdk.java.net/jdk/pull/4090/files/bc86a6fa..e8803de1

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=4090&range=01
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=4090&range=00-01

  Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 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