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

[Haoyu Li]

Thanks Stefan,

If there are any issues in the test, please feel free to contact me!

Best Regards,
Haoyu Li


Stefan Johansson <stefan.johansson at oracle.com> 于2019年12月5日周四 下午9:24写道：

> 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
> >
>
>