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

[Andrew John Hughes]

On Thu, 30 Jun 2022 03:36:49 GMT, Dongbo He <dongbohe at openjdk.org> wrote:

>> 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

SKARA seems to be confused by the closure of #66 due to timeout, not integration. Please re-open that one so we can correctly integrate it as a dependency for this one.

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

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