JDK-8228718 : Revert incorrect backport of JDK-8129757 to 8-aarch64
  • Type: Bug
  • Component: hotspot
  • Sub-Component: runtime
  • Affected Version: 8-aarch64
  • Priority: P4
  • Status: Resolved
  • Resolution: Fixed
  • Submitted: 2019-07-29
  • Updated: 2021-02-01
  • Resolved: 2019-07-30
The Version table provides details related to the release that this issue/RFE will be addressed.

Unresolved : Release in which this issue/RFE will be addressed.
Resolved: Release in which this issue/RFE has been resolved.
Fixed : Release in which this issue/RFE has been fixed. The release containing this fix may be available for download as an Early Access Release or a General Availability Release.

To download the current JDK release, click here.
JDK 8 Other
8-aarch64Fixed openjdk8u292Fixed
Related Reports
Relates :  
JDK-8257192 - Integrate AArch64 JIT port into 8u
Relates :  
JDK-8129757 - ppc/aarch: Fix passing thread to runtime after "8073165: Contended Locking fast exit bucket."
Description
In current aarch64-port code, there is a backport of JDK-8129757:
 https://hg.openjdk.java.net/aarch64-port/jdk8u-shenandoah/hotspot/rev/f39e296fb4c5

The backport is there to fit the changed signature of SharedRuntime::complete_monitor_unlocking_C that is introduced by JDK-8073165, that is not available in 8u. Current signature of SharedRuntime::complete_monitor_unlocking_C in 8-aarch is:

src/share/vm/opto/runtime.hpp
180:  static void complete_monitor_unlocking_C(oopDesc* obj, BasicLock* lock);

src/share/vm/runtime/sharedRuntime.hpp
504:  static void complete_monitor_unlocking_C(oopDesc* obj, BasicLock* lock);

This means the backport is actually incorrect, and we should revert it.
Comments
https://hg.openjdk.java.net/aarch64-port/jdk8u-shenandoah/hotspot/rev/f82f2008d4cf
30-07-2019
RFR: https://mail.openjdk.java.net/pipermail/aarch64-port-dev/2019-July/007736.html
29-07-2019