RFR: JDK-8256155: os::Linux Populate all large_page_sizes, select smallest page size in reserve_memory_special_huge_tlbfs* [v25]

Marcus G K Williams github.com+168222+mgkwill at openjdk.java.net
Tue Mar 16 20:59:27 UTC 2021


> When using LargePageSizeInBytes=1G, os::Linux::reserve_memory_special_huge_tlbfs* cannot select large pages smaller than 1G. Code heap usually uses less than 1G, so currently the code precludes code heap from using
> Large pages in this circumstance and when os::Linux::reserve_memory_special_huge_tlbfs* is called page sizes fall back to Linux::page_size() (usually 4k).
> 
> This change allows the above use case by populating all large_page_sizes present in /sys/kernel/mm/hugepages in _page_sizes upon calling os::Linux::setup_large_page_size().
> 
> In os::Linux::reserve_memory_special_huge_tlbfs* we then select the largest large page size available in _page_sizes that is smaller than bytes being reserved.

Marcus G K Williams has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 36 commits:

 - Merge branch 'master' into update_hlp
 - Addressed kstefanj review suggestions
   
   Signed-off-by: Marcus G K Williams <marcus.williams at intel.com>
 - Use SIZE_FORMAT in logging
   
   Signed-off-by: Marcus G K Williams <marcus.williams at intel.com>
 - Fix logging issues 2
   
   Signed-off-by: Marcus G K Williams <marcus.williams at intel.com>
 - Fix logging issues
   
   Signed-off-by: Marcus G K Williams <marcus.williams at intel.com>
 - Fix reserve_memory_special_huge_tlbfs_mixed, remove logging
   
   Signed-off-by: Marcus G K Williams <marcus.williams at intel.com>
 - Fix whitespace error
   
   Signed-off-by: Marcus G K Williams <marcus.williams at intel.com>
 - Fix first set of TestTracePageSizes.java issues
   
   Signed-off-by: Marcus G K Williams <marcus.williams at intel.com>
 - Update LargePage Setup per review comments
   
   Signed-off-by: Marcus G K Williams <marcus.williams at intel.com>
 - Merge remote-tracking branch 'upstream/master' into update_hlp
 - ... and 26 more: https://git.openjdk.java.net/jdk/compare/9cb9af68...08bf288c

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

Changes: https://git.openjdk.java.net/jdk/pull/1153/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=1153&range=24
  Stats: 154 lines in 3 files changed: 76 ins; 47 del; 31 mod
  Patch: https://git.openjdk.java.net/jdk/pull/1153.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/1153/head:pull/1153

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



More information about the hotspot-gc-dev mailing list