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