RFR: 8266807: Windows os_windows-gtest broken for UseLargePages

Ivan Walulya iwalulya at openjdk.java.net
Fri May 28 09:18:06 UTC 2021


On Tue, 18 May 2021 11:08:50 GMT, Stefan Johansson <sjohanss at openjdk.org> wrote:

> 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.

lgtm!

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

Marked as reviewed by iwalulya (Committer).

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


More information about the hotspot-runtime-dev mailing list