RFR: 8342580: GenShen: TestChurnNotifications fails executing in unintended test-id modes with ShenandoahGCMode=generational [v2]
Y. Srinivas Ramakrishna
ysr at openjdk.org
Wed Oct 23 17:01:55 UTC 2024
> Generational mode should not be tested with explicit heuristics settings, which are silently ignored. The test failed in these settings because it needs to be told which memory pool to check for notifications, which needs to be specified differently for generational. Ideally, we'd have the test figure this out rather than needing this to be explicitly specified. For now I just tightened the `@requires` clause to run only in the intended modes.
>
> Related cleanups will follow separately (such as emitting a warning when an incompatible heuristics option is specified, code which may have been lost in a previous refactor). See details in the ticket.
>
> **Testing** : tested with no specific collector, with Shenandoah, withGenShen, and with other collectors with test explicitly specified. As intended, in the first and second case 6 test-ids are run, in the thirdd case 1 test-id is run, and in the remaining cases, 0 test-ids are run.
Y. Srinivas Ramakrishna 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 four additional commits since the last revision:
- Merge branch 'master' into test_requires
- Merge branch 'master' into test_requires
- Don't test generational with heuristics it ignores. Separately, we'll
fix the code for generational such that the choice of a specific
heuristic elicits a warning.
- TestChurnNotifications fails when jtreg run with GenShen
-------------
Changes:
- all: https://git.openjdk.org/shenandoah/pull/520/files
- new: https://git.openjdk.org/shenandoah/pull/520/files/9cf9504f..3744a7f3
Webrevs:
- full: https://webrevs.openjdk.org/?repo=shenandoah&pr=520&range=01
- incr: https://webrevs.openjdk.org/?repo=shenandoah&pr=520&range=00-01
Stats: 24445 lines in 544 files changed: 20218 ins; 1645 del; 2582 mod
Patch: https://git.openjdk.org/shenandoah/pull/520.diff
Fetch: git fetch https://git.openjdk.org/shenandoah.git pull/520/head:pull/520
PR: https://git.openjdk.org/shenandoah/pull/520
More information about the shenandoah-dev
mailing list