RFR: 8298783: java/lang/ref/FinalizerHistogramTest.java failed with "RuntimeException: MyObject is not found in test output" [v5]

Brent Christian bchristi at openjdk.org
Fri Apr 25 21:30:10 UTC 2025


> I propose some cleanups to `FinalizerHistogramTest.java` to hopefully clear up the intermittent failures:
> 
> * run with `othervm`: this test blocks the (global) finalizer thread, and also requires the (global) finalizer thread to enter the test's `finalize()` method
> * The test uses `volatile` ints, but sets them based on their current value, which is not reliable; convert to `AtomicInteger`
> * use `PhantomReference`s to ensure that at least two `MyObject`s have become unreachable. If one is stuck in `finalize()`, at least one is still waiting to be finalized and should show up in the histogram.

Brent Christian 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 8298783
 - convert to WeakRefs, use a RefQ, print ForceGC results
 - 'return' not needed in lambda
 - rename counter variables to use 'Count'
 - test reliability improvements

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/24143/files
  - new: https://git.openjdk.org/jdk/pull/24143/files/d1041073..cfbb4230

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=24143&range=04
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=24143&range=03-04

  Stats: 353256 lines in 4198 files changed: 101602 ins; 237660 del; 13994 mod
  Patch: https://git.openjdk.org/jdk/pull/24143.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/24143/head:pull/24143

PR: https://git.openjdk.org/jdk/pull/24143


More information about the core-libs-dev mailing list