RFR: 8367371: Remove @requires vm.opt.UseLargePages from InternSharedString.java test [v2]

Paul Hübner phubner at openjdk.org
Thu Sep 11 14:00:33 UTC 2025


On Thu, 11 Sep 2025 13:53:04 GMT, Joel Sikström <jsikstro at openjdk.org> wrote:

>> Paul Hübner has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Update test/hotspot/jtreg/runtime/cds/appcds/sharedStrings/InternSharedString.java
>>   
>>   Co-authored-by: Joel Sikström <joel.sikstrom at oracle.com>
>
> test/hotspot/jtreg/runtime/cds/appcds/sharedStrings/InternSharedString.java line 30:
> 
>> 28:  * @requires vm.cds.write.archived.java.heap
>> 29:  * @requires vm.gc == null
>> 30:  * @comment CDS archive heap mapping is not supported with large pages
> 
> We should remove the comment as well.
> Suggestion:

Good point, I missed that 🤦 .

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27218#discussion_r2341007120


More information about the hotspot-runtime-dev mailing list