RFR: 8236569: -Xss not multiple of 4K does not work for the main thread on macOS [v2]

Henry Jen henryjen at openjdk.java.net
Fri Jun 4 19:07:19 UTC 2021


> …d on macOS
> 
> This patch simply round up the specified stack size to multiple of the system page size. 
> 
> Test is trivial, simply run java with -Xss option against following code. On MacOS, before the fix, running with `-Xss159k` and `-Xss160k` would get `7183` and `649` respectively. After fix, both would output `649`, while `-Xss161k` would be same as `-Xss164k` and see 691 as the output.
> 
> ```code:java
> public class StackLeak {
>     public int depth = 0;
>     public void stackLeak() {
>         depth++;
>         stackLeak();
>     }
> 
>     public static void main(String[] args) {
>         var test = new StackLeak();
>         try {
>             test.stackLeak();
>         } catch (Throwable e) {
>             System.out.println(test.depth);
>         }
>     }
> }

Henry Jen has updated the pull request incrementally with one additional commit since the last revision:

  Avoid overflow on page size

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

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/4256/files
  - new: https://git.openjdk.java.net/jdk/pull/4256/files/fb9b22d5..39b041d7

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=4256&range=01
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=4256&range=00-01

  Stats: 15 lines in 2 files changed: 8 ins; 0 del; 7 mod
  Patch: https://git.openjdk.java.net/jdk/pull/4256.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/4256/head:pull/4256

PR: https://git.openjdk.java.net/jdk/pull/4256


More information about the core-libs-dev mailing list