RFR: 8362561: Remove diagnostic option AllowArchivingWithJavaAgent [v2]

Serguei Spitsyn sspitsyn at openjdk.org
Thu Sep 18 14:31:46 UTC 2025


On Thu, 18 Sep 2025 03:35:19 GMT, Ioi Lam <iklam at openjdk.org> wrote:

>> Goal: simplify AOT implementation and removed unnecessary tests:
>> 
>> This PR removes old CDS tests that use java agents during CDS dump (to increase test coverage by triggering conditions, such as GCs, that were not happening during usual `java -Xshare:dump` operations).
>> 
>> Since AOT assembly is much more complex now, we have enough coverage. These old tests are no longer necessary.
>> 
>> With these tests removed, we no longer need the `-XX:-AllowArchivingWithJavaAgent` diagnostic flag, which has been abused by users despite the warning that this flag should not be used in production.
>
> Ioi Lam has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains three commits:
> 
>  - Merge branch 'master' into 8362561-remove-diagnostic-option-AllowArchivingWithJavaAgent
>  - Added test case for dynamic archive with -javaagent
>  - 8362561: Remove diagnostic option AllowArchivingWithJavaAgent

Marked as reviewed by sspitsyn (Reviewer).

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

PR Review: https://git.openjdk.org/jdk/pull/27304#pullrequestreview-3240125838


More information about the serviceability-dev mailing list