RFR: 8337318: Deoptimization::relock_objects fails assert(monitor->owner() == Thread::current()) failed: must be

[Richard Reingruber]

On Mon, 12 Aug 2024 16:12:20 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

> The assert is wrong.  During deoptimization, the locking_thread is not the current thread, and that's the thread we should assert is the owner.
> Reproduced in tier8 with com/sun/jdi/EATests.java, with LockStack CAPACITY=1 and not reproduced with the fix.

Looks good to me. The reproducer succeeds with the fix.

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

Marked as reviewed by rrich (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/20553#pullrequestreview-2233620114