RFR: 8220465: Use shadow regions for faster ParallelGC full GCs
Stefan Johansson
stefan.johansson at oracle.com
Fri Dec 6 11:48:55 UTC 2019
Hi Haoyu,
Your contribution has now been pushed and it will run through our testing over the weekend. I’ll let you know if we see any problems but I don’t think we will, both me and Thomas have to some significant testing already.
If you are interested in becoming more involved in OpenJDK you are now just one more contribution away from being able to request an OpenJDK username and become an Author. More information can be found here:
https://openjdk.java.net/projects/
Cheers,
Stefan
> 5 dec. 2019 kl. 15:59 skrev Haoyu Li <leihouyju at gmail.com>:
>
> 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
> >
>
More information about the hotspot-gc-dev
mailing list