[jdk8u-dev] RFR: 8067941: [TESTBUG] Fix tests for OS with 64K page size. [v2]

Dongbo He dongbohe at openjdk.org
Thu Jun 30 03:36:49 UTC 2022


> Hi,
> 
> I would like to backport this patch to fix tests for OS with 64K page size.
> Patch does not apply cleanly: 
> JDK-8062854[1] move StackOverflowBug.java and Test8009761.java to corresponding subfolders.
> Test8009761.java context is different because JDK-8021775[2] and JDK-8011397[3] doesn't in jdk8u.
> Change to TestGCLogMessages.java is excluded because it was added in JDK-8027962[4].
> Chnage to WBStackSize.java is excluded because JDK-8032970[5] does not exist in 8.
> 
> I changed Xss in StackOverflowBug.java, Test8009761.java, TestRecursiveReplacedException.java, 
> StackOverflowGuardPagesOff.java, TestStackBangMonitorOwned.java and TestStackBangRbp.java from
> 392k to 512k according to JDK-8159335[6], because JDK-8173339[7] changed StackShadowPages to 20, xss needs at least 456k.
> 
> Testing: Performed full jtreg test aarch64-linux-gnu platforms with 64k page size.
> Checked that StackOverflowBug.java, Test8009761.java, TestRecursiveReplacedException.java,
> StackOverflowGuardPagesOff.java, TestStackBangMonitorOwned.java, TestStackBangRbp.java,
> TestHumongousAllocInitialMark.java, TestCMSHeapSizeFlags.java, TestG1HeapSizeFlags.java,
> TestParallelHeapSizeFlags.java, TestSerialHeapSizeFlags.java fails without the patch
> and passes with the patch on Aarch64 with 64K page size.
> 
> 
> [1] https://bugs.openjdk.java.net/browse/JDK-8062854
> [2] https://bugs.openjdk.java.net/browse/JDK-8021775
> [3] https://bugs.openjdk.java.net/browse/JDK-8011397
> [4] https://bugs.openjdk.java.net/browse/JDK-8027962
> [5] https://bugs.openjdk.java.net/browse/JDK-8032970
> [6] https://bugs.openjdk.java.net/browse/JDK-8159335
> [7] https://bugs.openjdk.java.net/browse/JDK-8173339

Dongbo He has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains two additional commits since the last revision:

 - Merge branch '8173339' into 8067941
 - Backport 8e2df5f543522866e7c27ff95ea6fb6458393682

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

Changes:
  - all: https://git.openjdk.org/jdk8u-dev/pull/71/files
  - new: https://git.openjdk.org/jdk8u-dev/pull/71/files/b2717683..565302ee

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=71&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=71&range=00-01

  Stats: 1077 lines in 21 files changed: 966 ins; 51 del; 60 mod
  Patch: https://git.openjdk.org/jdk8u-dev/pull/71.diff
  Fetch: git fetch https://git.openjdk.org/jdk8u-dev pull/71/head:pull/71

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


More information about the jdk8u-dev mailing list