RFR: 8368104: Parallel: Refactor PSThreadRootsTaskClosure [v2]

[Albert Mingkun Yang]

On Fri, 19 Sep 2025 17:42:10 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 295:
> 
>> 293:     {
>> 294:       PSScavengeRootsClosure root_closure(pm);
>> 295:       _oop_storage_strong_par_state.oops_do(&root_closure);
> 
> The pm variable isn't local to this scope anymore so it would probably be nicer to move the following part out of the scope:
> 
>       // Do the real work
>       pm->drain_stacks(false);

Well, it's there to be consistent with other kinds of roots -- old-to-young pointers, inside `scavenge_roots_work`, and `thread_closure`.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27393#discussion_r2364062470