Integrated: 8261636: The test mapping in hugetlbfs_sanity_check should consider LargePageSizeInBytes
Stefan Johansson
sjohanss at openjdk.java.net
Mon Mar 1 12:05:51 UTC 2021
On Tue, 23 Feb 2021 17:29:44 GMT, Stefan Johansson <sjohanss at openjdk.org> wrote:
> When explicit large pages are enabled we do a sanity check to test if there are large pages configured. This sanity check just tests if there are any pages available of the systems default large page size. When the flag `LargePageSizeInBytes` is specified the JVM will try to use this large page size for all its mapping, except for the first sanity mapping. This can lead to large pages being disabled because the sanity mapping failed, while there are still plenty of pages available of the size specified by `LargePageSizeInBytes`.
>
> The fix is to simply take the large page size into consideration in the sanity-check, making sure the correct flags are passed when needed.
>
> **Testing**
> * Manual testing verifying the problem when for example only 1g pages are available. Before the fix these large pages could not be used (unless the default large page size was changed to 1g or there were also 2m pages configured).
This pull request has now been integrated.
Changeset: 0a7fff46
Author: Stefan Johansson <sjohanss at openjdk.org>
URL: https://git.openjdk.java.net/jdk/commit/0a7fff46
Stats: 23 lines in 2 files changed: 12 ins; 7 del; 4 mod
8261636: The test mapping in hugetlbfs_sanity_check should consider LargePageSizeInBytes
Reviewed-by: tschatzl, iwalulya
-------------
PR: https://git.openjdk.java.net/jdk/pull/2695
More information about the hotspot-runtime-dev
mailing list