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