RFR: 8255070: Shenandoah: Use single thread for concurrent CLD liveness test [v3]

Roman Kennke rkennke at openjdk.java.net
Wed Oct 21 18:05:16 UTC 2020


On Wed, 21 Oct 2020 16:43:33 GMT, Zhengyu Gu <zgu at openjdk.org> wrote:

>> Since the concurrent liveness test does not touch any oops, there is no point to use multi-thread for the task, they just duplicate the work.
>> 
>> Also, multi-thread version requires to acquire ClassLoaderGraph_lock early, which is deadlock prone.
>
> Zhengyu Gu has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Replaced naked lock/unlock with MutexLocker

Looks good to me! Thank you!

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

Marked as reviewed by rkennke (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/777



More information about the hotspot-gc-dev mailing list