JDK-8330973 : Test8009761 fails on ppcle_linux: init recursive calls: 38. After deopt 37
  • Type: Bug
  • Component: hotspot
  • Sub-Component: compiler
  • Affected Version: openjdk8u412
  • Priority: P4
  • Status: Open
  • Resolution: Unresolved
  • OS: linux_ubuntu
  • CPU: ppc
  • Submitted: 2024-04-23
  • Updated: 2024-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
8-poolUnresolved
Related Reports
Relates :  
JDK-8021775 - compiler/8009761/Test8009761.java "Failed: init recursive calls: 51. After deopt 50"
Description
In some versions of ppcle Linux (e.g. Ubuntu 1804), Test8009761 fails with this message:

-----
[2023-07-19T16:35:21.776Z] CompilerOracle: exclude Test8009761.m2
[2023-07-19T16:35:21.776Z] Failed: init recursive calls: 38. After deopt 37
-----

One solution to this could be to backport the fix created for JDK9 here: https://bugs.openjdk.org/browse/JDK-8021775

It appears that this fix was intended to be deferred until 8u20, and then backported. However, I don't see any indication that this was ever done.

If we don't feel the need to backport the full fix, we can resolve this problem by at least backporting the portion of the fix that increases the variance tolerance for the deopt value:

-----
        // Allow number of recursive calls to vary by 1
        if ((c1 < (count - 1)) || (c1 > (count + 1))) {
-----

Will create a commit shortly for review.
Comments
Update: Severin reviewed PR 487 and requested a full backport rather than the stripped-down version. I've complied with his request, and created a new PR here: https://github.com/openjdk/jdk8u-dev/pull/545 This PR also include the Xss change from an interim commit, and has been tested here: https://ci.adoptium.net/job/Grinder/10554/ As per the backporting guide linked below, I'm currently on Step 9, awaiting a review from a JDK8u reviewer. https://wiki.openjdk.org/display/jdk8u
10-07-2024
I've been asked to replace the PR above with a full backport of JDK-8021775 (mentioned in the description). The PR has been closed, and this bug will be closed as well once we have a solid response on the request for the backport approval.
24-04-2024
A pull request was submitted for review. URL: https://git.openjdk.org/jdk8u-dev/pull/487 Date: 2024-04-23 13:02:30 +0000
23-04-2024