RFR: 8367848: Parallel: Use NMethodToOopClosure during Young GC [v2]

Albert Mingkun Yang ayang at openjdk.org
Wed Sep 17 14:54:57 UTC 2025


On Wed, 17 Sep 2025 14:41:11 GMT, Stefan Karlsson <stefank at openjdk.org> wrote:

>> Albert Mingkun Yang has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   review
>
> src/hotspot/share/gc/parallel/psScavenge.cpp line 244:
> 
>> 242:       Threads::assert_all_threads_claimed();
>> 243:     }
>> 244:   };
> 
> Could this class be placed in Threads together with these functions:
> 
>   static uintx thread_claim_token()
>   static void change_thread_claim_token();
>   static void assert_all_threads_claimed() NOT_DEBUG_RETURN;
>   static void possibly_parallel_oops_do(bool is_par, OopClosure* f, NMethodClosure* cf);
> 
> 
> There's other GC code that could benefit from using this helper.

Moved to `threads.hpp`

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27336#discussion_r2355810260


More information about the hotspot-gc-dev mailing list