RFR: 8362561: Remove diagnostic option AllowArchivingWithJavaAgent [v2]

Ioi Lam iklam at openjdk.org
Thu Sep 18 04:46:33 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

I have merged manually to fix conflicts in TEST.groups. No other changes.
Please re-review. Thanks!

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

PR Comment: https://git.openjdk.org/jdk/pull/27304#issuecomment-3305406551


More information about the serviceability-dev mailing list