[jdk11u-dev] RFR: 8257993: vmTestbase/nsk/jvmti/RedefineClasses/StressRedefine/TestDescription.java crash intermittently

[Martin Doerr]

On Thu, 20 Jul 2023 14:28:35 GMT, Lutz Schmidt <lucy at openjdk.org> wrote:

> This backport is for parity with 11.0.21-oracle
> 
> Patch (from jdk16) did not apply at all. Basically, it is a fresh implementation, guided by the original fix. Had to introduce new method "Method* get_new_method() const;" in method.hpp. Did not exist in 11.
> 
> Local testing successful.
> SAP internal nightly build and test did not show any issues.
> Reviews pending.

LGTM. Thanks for backporting!

src/hotspot/share/interpreter/interpreterRuntime.cpp line 864:

> 862:                                  last_frame.get_index_u2_cpcache(bytecode), bytecode,
> 863:                                  CHECK);
> 864: 

Original change doesn't introduce this empty line. I suggest to remove it before integrating.

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

Marked as reviewed by mdoerr (Reviewer).

PR Review: https://git.openjdk.org/jdk11u-dev/pull/2051#pullrequestreview-1541032717
PR Review Comment: https://git.openjdk.org/jdk11u-dev/pull/2051#discussion_r1270600440