RFR: 8361725: Do not load Java agent with "-Xshare:dump -XX:+AOTClassLinking"

[Ioi Lam]

On Fri, 18 Jul 2025 15:41:04 GMT, Matias Saavedra Silva <matsaave at openjdk.org> wrote:

>> When `-XX:+AOTClassLinking` is enabled when dumping CDS archives with `java -Xshare:dump`, we have more stringent restriction of what Java code can be executed -- if arbitrary Java code is executed, it may produce side effects that cannot be handled when archiving Java heap objects. This usually leads to cdsHeapVerifier.cpp reporting suspicious references to static fields that are not known to be safe.
>> 
>> We already avoid loading Java agents when dumping AOT caches (which are just enhanced CDS archives with more optimizations), we should do the same thing for `java -Xshare:dump -XX:+AOTClassLinking`.
>> 
>> After this PR, we still allow Java agents for `java -Xshare:dump -XX:-AOTClassLinking`, as that is required by some CDS tests. In a subsequent RFE, we will fix these CDS tests so Java agents are always disabled for `java -Xshare:dump`. See [JDK-8362561](https://bugs.openjdk.org/browse/JDK-8362561)
>
> LGTM!

Thanks @matias9927 @calvinccheung for the review

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

PR Comment: https://git.openjdk.org/jdk/pull/26374#issuecomment-3090763537