RFR: 8306738: Select num workers for safepoint ParallelCleanupTask [v3]

Fri May 5 07:10:19 UTC 2023

On Thu, 4 May 2023 18:44:02 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> Axel Boldt-Christmas has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Add expected_num_workers
>
> src/hotspot/share/runtime/safepoint.cpp line 550:
> 
>> 548:                    Universe::heap()->uses_stack_watermark_barrier()) {}
>> 549: 
>> 550:   uint expected_num_workers() const {
> 
> should this be 'static' for internal linkage?

It is a method and uses the non static member field `_do_lazy_roots`. Not a free function.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/13616#discussion_r1185763031