Integrated: 8266807: Windows os_windows-gtest broken for UseLargePages

Stefan Johansson sjohanss at openjdk.java.net
Mon May 31 12:54:26 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.

This pull request has now been integrated.

Changeset: 5df25dc0
Author:    Stefan Johansson <sjohanss at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/5df25dc088cfc3069e451b48c4f013d1d0491aa2
Stats:     23 lines in 1 file changed: 11 ins; 1 del; 11 mod

8266807: Windows os_windows-gtest broken for UseLargePages

Reviewed-by: iwalulya, ayang, tschatzl

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

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


More information about the hotspot-runtime-dev mailing list