RFR(S): 8192936: RI does not follow the JVMTI RedefineClasses spec that is too strict in the definition
coleen.phillimore at oracle.com
coleen.phillimore at oracle.com
Thu Apr 18 18:30:11 UTC 2019
Hi Serguei,
You made additional fixes for tests affected by this change to add
-XX:+AllowRedefinitionToAddOrDeleteMethods. These changes look good to me.
Thanks,
Coleen
On 4/16/19 5:40 AM, serguei.spitsyn at oracle.com wrote:
> Please, review the fix of:
> https://bugs.openjdk.java.net/browse/JDK-8192936
>
>
> Webrev (fix from Coleen):
> http://cr.openjdk.java.net/~sspitsyn/webrevs/2019/8192936-redef-add-delete.1/
>
> I've already reviewed and updated the webrev with my suggestions.
>
>
> Reviewed and approved CSR:
> https://bugs.openjdk.java.net/browse/JDK-8221528
>
>
> Summary:
> The fix introduces new VM option
> -XX:AllowRedefinitionToAddOrDeleteMethods
> for compatibility with previous releases.New option enables old behavior
> and allows the JVM TI RedefineClasses and RetransformClasses to
> add/delete
> private static and private final instance methods in the new class
> versions.
> Without this option the old behavior is disabled.
>
> New option is deprecated right away.
> The plan is to keep this option for several releases to allow customers
> (tool vendors) to remove dependency on old behavior from their tools.
>
>
> Testing:
> Added new test to verify that class redefinitions which add or
> delete methods
> return expected JVMTI error codes:
> test/hotspot/jtreg/serviceability/jvmti/RedefineClasses/TestAddDeleteMethods.java
>
> Several jvmti, com/sun/jdi and java/lang/instrument tests which need
> old behavior are updated to use new flag.
>
> Run locally on Linux-x64 the following test suites in release and
> fastdebug mode:
> - open/test/hotspot/jtreg/serviceability/jvmti/RedefineClasses/
> - vmTestbase_nsk_jvmti
> - vmTestbase_nsk_jdi
> - vmTestbase_nsk_jdb
> - vmTestbase_nsk_jdwp
> - jdk_jdi
> - jdk_instrument
>
> Submission of corresponding mach5 jobs is in progress.
>
> Thanks,
> Serguei
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.java.net/pipermail/serviceability-dev/attachments/20190418/82eeaba9/attachment.html>
More information about the serviceability-dev
mailing list