[jdk20] RFR: 8293824: gc/whitebox/TestConcMarkCycleWB.java failed "RuntimeException: assertTrue: expected true, was false" [v2]
Kim Barrett
kbarrett at openjdk.org
Wed Jan 4 03:27:19 UTC 2023
> Please review this change to WhiteBox and some tests involving G1 concurrent GCs.
>
> Some tests currently use WhiteBox.g1StartConcMarkCycle() to trigger a
> concurrent GC. Many of them follow it with a loop waiting for a concurrent
> cycle to not be in progress. A few also preceed that call with a similar
> loop, since that call does nothing and returns false if a concurrent cycle is
> already in progress. Those tests typically want to ensure there was a
> concurrent cycle that was started after some setup.
>
> The failing test calls that function, asserting that it returned true, e.g. a
> new concurrent cycle was started.
>
> There are various problems with this, due to races with concurrent cycles
> started automatically, and due to possibly aborted (by full GCs) concurrent cycles,
> making some of these tests unreliable in some configurations.
>
> For example, the test failure associated with this bug intermittently arises
> when running with `-Xcomp`, triggering a concurrent cycle before the explicit
> request by the test, causing the explicit request to fail (because there is
> already one in progress), failing the assert. Waiting for there not to be an
> in-progress cycle before the explicit request just narrows the race window.
>
> We have a different mechanism in WhiteBox for controlling concurrent cycles,
> the concurrent GC breakpoint mechanism. By adding a counter specifically for
> such cycles, we can use GC breakpoints to ensure only the concurrent cycles
> the test wants are occurring, and can verify they completed successfully.
>
> So we change tests using WhiteBox.g1StartConcMarkCycle() to instead use GC
> breakpoints, along with the new WhiteBox.g1CompletedConcurrentMarkCycles() to
> avoid racing request problems and to detect aborted cycles. Since it is no
> longer used, WhiteBox.g1StartConcMarkCycle() is removed. The test that began
> this adventure (TestConcMarkCycleWB.java) is also removed, since it no longer
> serves any purpose, having been purely a test of that removed function.
>
> Testing:
> mach5 tier1-7 - running the changed tests on a variety of platforms with a
> variety of configurations.
Kim Barrett 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 'master' into test-concmark20
- use gc breakpoints to perform conc gc
-------------
Changes:
- all: https://git.openjdk.org/jdk20/pull/71/files
- new: https://git.openjdk.org/jdk20/pull/71/files/39abc3dc..3b837273
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk20&pr=71&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk20&pr=71&range=00-01
Stats: 786 lines in 35 files changed: 563 ins; 95 del; 128 mod
Patch: https://git.openjdk.org/jdk20/pull/71.diff
Fetch: git fetch https://git.openjdk.org/jdk20 pull/71/head:pull/71
PR: https://git.openjdk.org/jdk20/pull/71
More information about the serviceability-dev
mailing list