RFR: 8220465: Use shadow regions for faster ParallelGC full GCs

Stefan Johansson stefan.johansson at oracle.com
Thu Dec 5 10:52:10 UTC 2019


Hi Haoyu,

Both me and Thomas are good with the current state of the patch and we think it is ready to get pushed. 

I will push the patch with a "Contributed-by: <email>" line to indicate you are the author of the patch. For that I need to know which email you used when signing the OCA? Is it the same one you are using on these lists: leihouyju at gmail.com

Thanks,
Stefan


> 4 dec. 2019 kl. 17:27 skrev Stefan Johansson <stefan.johansson at oracle.com>:
> 
> 
> 
>> 4 dec. 2019 kl. 16:55 skrev Thomas Schatzl <thomas.schatzl at oracle.com>:
>> 
>> Hi all,
>> 
>> On 04.12.19 10:47, Haoyu Li wrote:
>>> Hi Thomas and Stefan,
>>> Thanks for your quick replies. The changes also look great to me, but I noticed that a few more statics and assertions could be improved, so I have updated them in the attached patches.  I could have finished these updates earlier. However, there was a network issue that I cannot pull the latest code. Should you have any further suggestions, I am happy to adopt them.
>>> Best Regards,
>>> Haoyu Li
>> 
>> Merged with your changes.
>> 
>> http://cr.openjdk.java.net/~tschatzl/8220465/webrev.7_to_8/ (diff)
>> http://cr.openjdk.java.net/~tschatzl/8220465/webrev.8/ (full)
>> 
>> Tests of the previous version were good afaict.
>> 
>> Ready to go imho :)
> I agree,
> Stefan
> 
>> 
>> Thomas




More information about the hotspot-gc-dev mailing list