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

[Andrew John Hughes]

On Mon, 29 Aug 2022 01:47:58 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 five additional commits since the last revision:
> 
>  - Merge branch 'master' into 8067941
>  - Merge branch '8173339' into 8067941
>  - Merge branch 'master' into 8173339
>  - Backport 8e2df5f543522866e7c27ff95ea6fb6458393682
>  - Backport 540ec375c30e973ceb1a944d5cc60cc8532e88ec

8u-dev is currently closed for rampdown.
I'll sponsor this once it reopens.

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

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