RFR: 8267845: Add @requires to avoid running G1 large pages test with wrong page size
Stefan Johansson
sjohanss at openjdk.java.net
Thu May 27 14:22:19 UTC 2021
Please review this small test-fix.
**Summary**
These two tests try to verify that large pages are used correctly for the heap and the G1 internal data structures. The tests start separate processes that are run with specific arguments to know the expected outcome. The sub-processes don't inherit any parameters passed to the test. This is good, but if the parameter `LargePageSizeInBytes` is passed to the test, the driver process will run with this flag and pick up a different large page size than test-processes. This won't work since the driver will use its found large page size to verify the output from the test.
The simple fix is to simply not run this test if this flag is specified. For the test to work with this flag, more work has to be done to analyze how different page sizes will affect the test and for now I think this is the best approach.
**Testing**
Manual verification the the tests won't run if `LargePageSizeInBytes` is set.
-------------
Commit messages:
- 8267845: Add @requires to avoid running G1 large pages test with wrong page size
Changes: https://git.openjdk.java.net/jdk/pull/4226/files
Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=4226&range=00
Issue: https://bugs.openjdk.java.net/browse/JDK-8267845
Stats: 2 lines in 2 files changed: 2 ins; 0 del; 0 mod
Patch: https://git.openjdk.java.net/jdk/pull/4226.diff
Fetch: git fetch https://git.openjdk.java.net/jdk pull/4226/head:pull/4226
PR: https://git.openjdk.java.net/jdk/pull/4226
More information about the hotspot-gc-dev
mailing list