RFR: 8221785: Let possibly_parallel_threads_do cover the same threads as threads_do [v2]

Thomas Schatzl tschatzl at openjdk.org
Thu Jan 26 15:43:53 UTC 2023


On Thu, 26 Jan 2023 14:02:20 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> Thomas Schatzl has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   coleenp review
>
> src/hotspot/share/runtime/threads.cpp line 267:
> 
>> 265:     if (current->claim_threads_do(is_par, claim_token)) {
>> 266:       tc->do_thread(current);
>> 267:     }
> 
> If I understand correctly, if this is only OK in a safepoint, can you add the safepoint assert here?

Done. Another tier1-3 is almost done, also did local testing of Shenandoah tests to see whether there is any issue, none found.

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

PR: https://git.openjdk.org/jdk/pull/12201


More information about the shenandoah-dev mailing list