RFR 8249837: Avoid direct or implicit Thread::current() calls when we already have a current thread variable

David Holmes david.holmes at oracle.com
Thu Jul 30 06:24:38 UTC 2020


Hi Coleen,

Thanks for taking on this one. :)

On 30/07/2020 1:54 am, coleen.phillimore at oracle.com wrote:
> Summary: Add current thread and/or moved HandleMark to the scope that it 
> is needed
> 
> This patch removes HandleMarks from the GC code.  I added a NoHandleMark 
> to these places and found the locations in the verification code that 
> needs the HandleMarks, and moved them there.

Sounds good.

I looked at all the other changes and they all seemed fine too.

Thanks,
David

> Tested with tier1-6 with no failures.
> 
> open webrev at http://cr.openjdk.java.net/~coleenp/2020/8249837.01/webrev
> bug link https://bugs.openjdk.java.net/browse/JDK-8249837
> 
> Thanks,
> Coleen


More information about the hotspot-dev mailing list