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

Stefan Johansson stefan.johansson at oracle.com
Thu Dec 5 13:21:50 UTC 2019


Thanks Haoyu,

I will probably push this tomorrow, running it through some more testing.

Cheers,
Stefan

> 5 dec. 2019 kl. 13:30 skrev Haoyu Li <leihouyju at gmail.com>:
> 
> Hi Stefan,
> 
> I used leihouyju at gmail.com when signing my OCA, and it's the same address on these lists. Thanks very much for all your efforts reviewing this patch!
> 
> Best Regards,
> Haoyu Li
> 
> 
> Stefan Johansson <stefan.johansson at oracle.com> 于2019年12月5日周四 下午6:52写道:
> 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