8372860: TestCodeCacheUnloadDuringConcCycle fails on ARM32

[wkia]

This PR contains clean backports of 8372860 and 8373429.
These changes are intended to make TestCodeCacheUnloadDuringConcCycle more stable on different platforms including linux-arm32.
Test changes only. No risk.

---

Progress

• Change must be properly reviewed (1 review required, with at least 1 Reviewer)
• Change must not contain extraneous whitespace
• JDK-8372860 needs maintainer approval
• Commit message must refer to an issue
• JDK-8373429 needs maintainer approval

Issues

• JDK-8372860: TestCodeCacheUnloadDuringConcCycle fails on ARM32 (Bug - P4 - Approved)
• JDK-8373429: gc/g1/TestCodeCacheUnloadDuringConcCycle fails on various platforms (Bug - P4 - Approved)

Reviewers

• Aleksey Shipilev (@shipilev - Reviewer)

Reviewing

Using git

Checkout this PR locally:
$ git fetch https://git.openjdk.org/jdk25u-dev.git pull/152/head:pull/152
$ git checkout pull/152

Update a local copy of the PR:
$ git checkout pull/152
$ git pull https://git.openjdk.org/jdk25u-dev.git pull/152/head

Using Skara CLI tools

Checkout this PR locally:
$ git pr checkout 152

View PR using the GUI difftool:
$ git pr show -t 152

Using diff file

Download this PR as a diff file:
https://git.openjdk.org/jdk25u-dev/pull/152.diff

Using Webrev

Link to Webrev Comment

[bridgekeeper]

👋 Welcome back rmarchenko! A progress list of the required criteria for merging this PR into master will be added to the body of your pull request. There are additional pull request commands available for use with this pull request.

[openjdk]

@wkia This change now passes all automated pre-integration checks.

ℹ️ This project also has non-automated pre-integration requirements. Please see the file CONTRIBUTING.md for details.

After integration, the commit message for the final commit will be:

8372860: TestCodeCacheUnloadDuringConcCycle fails on ARM32
8373429: gc/g1/TestCodeCacheUnloadDuringConcCycle fails on various platforms

Reviewed-by: shade

You can use pull request commands such as /summary, /contributor and /issue to adjust it as needed.

At the time when this comment was updated there had been 55 new commits pushed to the master branch:

• 38a616f: 8375530: PPC64: incorrect quick verify_method_data_pointer check causes poor performance in debug build
• 3973384: 8366082: Improve queue size computation in CPU-time sampler
• cc69ed7: 8374872: Cleanup outdated SAP AG copyright header info
• ... and 52 more: https://git.openjdk.org/jdk25u-dev/compare/bdf4939dcd050e0493e55ea819daaaead8200e34...master

As there are no conflicts, your changes will automatically be rebased on top of these commits when integrating. If you prefer to avoid this automatic rebasing, please check the documentation for the /integrate command for further details.

As you do not have Committer status in this project an existing Committer must agree to sponsor your change. Possible candidates are the reviewers of this PR (@shipilev) but any other Committer may sponsor as well.

➡️ To flag this PR as ready for integration with the above commit message, type /integrate in a new comment. (Afterwards, your sponsor types /sponsor in a new comment to perform the integration).

[openjdk]

This backport pull request has now been updated with issue from the original commit.

[openjdk]

⚠️ @wkia This change is now ready for you to apply for maintainer approval. This can be done directly in each associated issue or by using the /approval command.

[wkia]

/approval request I'd like to backport this to 25u-dev as jdk25 is affected as well. Clean backport. Test changes only. No risk.

[openjdk]

@wkia
8372860: The approval request has been created successfully.

[mlbridge]

Webrevs

• 01: Full - Incremental (8e0a940f)
• 00: Full (be08b035)

[openjdk]

@wkia
Your change (at version be08b03) is now ready to be sponsored by a Committer.

[wkia]

/integrate

[openjdk]

@wkia
Your change (at version be08b03) is now ready to be sponsored by a Committer.

[shipilev]

There is a bugtail: https://bugs.openjdk.org/browse/JDK-8373429 -- are you planning to backport that as well?

[wkia]

@shipilev

https://bugs.openjdk.org/browse/JDK-8373429

I'm not going to backport this to 25, as far as I see it's labeled as "affected 26", not 25.

[shipilev]

I'm not going to backport this to 25, as far as I see it's labeled as "affected 26", not 25.

Well, sure it does not affect 25, because it does not affect 25 until you do this backport! JDK-8373429 is the bugtail for this PR.

[wkia]

@shipilev No problem then, I will backport JDK-8373429 as well with the next PR.
Or do you want me to add that change to the current PR?

[shipilev]

I think it is worth mixing up two commits in this PR to avoid breakages in between commits.

[wkia]

/issue add 8373429

[openjdk]

@wkia
Adding additional issue to issue list: 8373429: gc/g1/TestCodeCacheUnloadDuringConcCycle fails on various platforms.

[shipilev]

This looks right, thanks.

[wkia]

/approval JDK-8373429 request I'd like to backport this to 25u-dev as a part of fixing this test on linux-arm32. Clean backport. Test changes only. No risk.

[openjdk]

@wkia
JDK-8373429: The approval request has been created successfully.

[wkia]

/integrate

[openjdk]

@wkia
Your change (at version 8e0a940) is now ready to be sponsored by a Committer.