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