On Wed, 13 May 2026 15:19:06 GMT, Aleksey Shipilev <[email protected]> wrote:
>> Xiaolong Peng has updated the pull request with a new target base due to a >> merge or a rebase. The pull request now contains 386 commits: >> >> - Merge branch 'openjdk:master' into cas-alloc-1 >> - Document _collector_allocator_reserved ordering contract >> - Tame TestCASAllocContention: fewer threads, shorter run, smaller retention >> - Comment: pair reserve-time inflation with read-time compensation >> >> Cross-reference ShenandoahFreeSet::reserve_alloc_regions_internal at >> every 'bytes_allocated - mutator_allocator_remaining' subtraction site. >> No behavior change. >> - Include mutator allocator remaining bytes in unsafe_max_tlab_alloc >> - Reserve mutator alloc regions after abbreviated degenerated GC >> - fix: new jtreg tests miss -XX:+UnlockDiagnosticVMOptions >> - fix: sync _top before CAS in unset_active_alloc_region >> - fix: sync _top once on CAS success and reset_age after the loop in >> unset_active_alloc_region >> - Add jtreg tests for CAS allocator flag combinations and contention >> - ... and 376 more: https://git.openjdk.org/jdk/compare/ebb3d688...ce68d1fa > > src/hotspot/share/gc/shenandoah/shenandoahAffiliation.hpp line 28: > >> 26: #define SHARE_GC_SHENANDOAH_SHENANDOAHAFFILIATION_HPP >> 27: >> 28: #include "utilities/debug.hpp" > > Do you need it? If you don't use anything from the header in this file, move > the include where it is actually needed. I will fix this, there is no change in this file, I don't think I need the include for utilities/debug.hpp, I may reverted this changes in file but forgot to remove the header. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/26171#discussion_r3236676524
