[jdk8u-dev] Integrated: JDK-8319405: [s390] [jdk8] Increase javac default stack size for s390x zero

[Thomas Stuefe]

On Fri, 3 Nov 2023 14:49:24 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:

> This is a JDK8 - only patch to fix failing TCK tests for javac on s390.
> 
> s390x, by default, needs a lot of stack space for C++ frames since the s390x ABI requires a 160-byte register save area per frame. That affects JDK 8 in particular since we run Zero, and the C++ interpreter builds up C-frames for Java frames; furthermore, it affects javac since it recurses a lot.
> 
> This causes several javac TCK tests to fail with SOE (stmt33002mxxx).
> 
> To pass TCK, the default stack size of javac should be increased for s390 on JDK 8.
> 
> Testing:
> 
> - manually tested on s390 to see if the issue was resolved, which it was.
> - manually tested on x64 to check that nothing changed
> - GHAs

This pull request has now been integrated.

Changeset: dd43bdd6
Author:    Thomas Stuefe <stuefe at openjdk.org>
URL:       https://git.openjdk.org/jdk8u-dev/commit/dd43bdd6c4fae298d401222ddfaf848b0d458aa3
Stats:     8 lines in 1 file changed: 7 ins; 0 del; 1 mod

8319405: [s390] [jdk8] Increase javac default stack size for s390x zero

Reviewed-by: phh, sgehwolf

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

PR: https://git.openjdk.org/jdk8u-dev/pull/385