8314139: TEST_BUG: runtime/os/THPsInThreadStackPreventionTest.java could fail on machine with large number of cores#3212
8314139: TEST_BUG: runtime/os/THPsInThreadStackPreventionTest.java could fail on machine with large number of cores#3212tabata-d wants to merge 2 commits into
Conversation
|
👋 Welcome back dtabata! A progress list of the required criteria for merging this PR into |
|
❗ This change is not yet ready to be integrated. |
|
This backport pull request has now been updated with issue from the original commit. |
Webrevs
|
tstuefe
left a comment
There was a problem hiding this comment.
Okay. Please make sure that Github Actions run through successfully
|
The parent pull request that this pull request depends on has been closed without being integrated and the target branch of this pull request has been updated as the previous branch was deleted. This means that changes from the parent pull request will start to show up in this pull request. If closing the parent pull request was done in error, it will need to be re-opened and this pull request will need to manually be retargeted again. |
|
After the discussion in #3210, I have decided to close this PR. |
This fix addresses a test reliability issue in the HotSpot VM where the
runtime/os/THPsInThreadStackPreventionTest.javatest could fail with "RSS+Swap larger than expected" errors on machines with a large number of cores.This is a clean backport from JDK 22 to JDK 11.
Testing:
THPsInThreadStackPreventionTest.javatest, including both#ENABLEDand#DISABLEDconfigurations, passed successfully.Progress
Integration blocker
Issue
Reviewers
Reviewing
Using
gitCheckout this PR locally:
$ git fetch https://git.openjdk.org/jdk11u-dev.git pull/3212/head:pull/3212$ git checkout pull/3212Update a local copy of the PR:
$ git checkout pull/3212$ git pull https://git.openjdk.org/jdk11u-dev.git pull/3212/headUsing Skara CLI tools
Checkout this PR locally:
$ git pr checkout 3212View PR using the GUI difftool:
$ git pr show -t 3212Using diff file
Download this PR as a diff file:
https://git.openjdk.org/jdk11u-dev/pull/3212.diff
Using Webrev
Link to Webrev Comment