On Fri, 10 Jul 2026 06:52:03 GMT, Serguei Spitsyn <[email protected]> wrote:
>> This update fixes the test bug which was initially filed as a Valhalla bug >> but then found to be NOT Valhalla specific. >> The static variable `objectCount` is used for two different purposes: >> - to count current walked objects number >> - as a number to tag objects >> >> This leads to mismatches in indexes calculations. >> For instance the `ObjectFree` event callback decrements the `objectCount` >> number. Then the `objectCountMax` can be smaller than some of the actual >> indexes. >> The fix does: >> - adds a fragment to force GC's with the JVMTI `ForceGarbageCollection` to >> make the failure well reproducible >> - split `objectCount` into: `objectCount` and `objectTagCount` >> - `objectTagCount` is used to set the object tags and value of the >> `objectCountMax` >> - a couple of the `ind` boundary checks are also corrected >> >> Testing: >> - Executed the test locally: >> `jvmti/IterateOverReachableObjects/iterreachobj002` >> - TBD: Submit mach5 tiers 1-3 to be safe >> >> --------- >> - [x] I confirm that I make this contribution in accordance with the >> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai). > > Serguei Spitsyn 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 three additional > commits since the last revision: > > - Merge > - review: moved to the right place the line which sets the objectCountMax > - 8384882: jvmti/IterateOverReachableObjects/iterreachobj002 fails with Not > a valid malloc pointer PING! Please, need at least one reviewer for this test issue. ------------- PR Comment: https://git.openjdk.org/jdk/pull/31585#issuecomment-4943934294
