On Tue, 7 Nov 2023 14:59:48 GMT, Roman Kennke <rken...@openjdk.org> wrote:
>> See JBS issue for details. >> >> Testing: >> - [x] gc/logging/TestUnifiedLoggingSwitchStress.java -XX:+UseParallelGC >> - [x] tier1 -XX:+UseParallelGC >> - [ ] tier2 -XX:+UseParallelGC >> - [ ] hotspot_gc > > Roman Kennke has updated the pull request incrementally with one additional > commit since the last revision: > > Review comments; expose self-forwarded objects src/hotspot/share/gc/parallel/mutableSpace.cpp line 242: > 240: // so we can encounter objects with "forwarded" markword. > 241: // They are essentially dead, so skipping them > 242: if (!obj->is_forwarded() || obj->forwardee() == obj) { Self-forwarded ones should already be handled by `PSPromotionManager::restore_preserved_marks`, so the second condition will always be false. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/16494#discussion_r1385076675