RFR: 8280761: UseCompressedOops should be set after limit_heap_by_allocatable_memory [v4]

Albert Mingkun Yang ayang at openjdk.java.net
Tue Apr 12 07:52:33 UTC 2022


On Tue, 12 Apr 2022 06:50:32 GMT, KIRIYAMA Takuya <duke at openjdk.java.net> wrote:

>> I fixed to set UseCompressedOops flag after limit_heap_by_allocatable_memory(). 
>> So when ulimit -v is called and -XX:MaxRAM is set, UseCompressedOops does not become false. 
>> And all hotspot tier1 test are passed.
>> Would you please review this fix?
>
> KIRIYAMA Takuya has updated the pull request incrementally with one additional commit since the last revision:
> 
>   8280761 UseCompressedOops should be set after limit_heap_by_allocatable_memory

test/hotspot/jtreg/gc/arguments/TestUseCompressedOopsFlagsWithUlimit.java line 85:

> 83: 
> 84:     // Args: MaxRAM , MaxRAMPercentage, expect coop
> 85:     // Having MaxRAMPercentage set explicitly instead of relying on its default value indeed makes the test more resilient.

Slight rewording, `// Setting MaxRAMPercentage explicitly to make the test more resilient.`

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

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



More information about the hotspot-gc-dev mailing list