<div dir="ltr">Hi Kim,<div><br></div><div>I have ported my patch to OpenJDK 13 according to your instructions in your last mail, and the patch is attached in this mail. The patch does not change much since PSGC is indeed pretty stable.</div><div><br></div><div>Also, I evaluate the correctness and performance of PS full GC with benchmarks from DaCapo, SPECjvm2008, and JOlden suits on a machine with dual Intel Xeon E5-2618L v3 CPUs(16 physical cores), 64G DRAM and linux kernel 4.17. The evaluation result, indicating 1.9X GC throughput improvement on average, is attached, too.</div><div><br></div><div>However, I have no idea how to further test this patch for both correctness and performance. Can I please get any guidance from you or some sponsor?</div><div><br clear="all"><div><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div style="font-size:12px"><div><font face="黑体" color="#000000">Best Regrads,</font></div><div><font face="黑体" color="#000000">Haoyu Li,</font></div><div><font face="黑体" color="#000000">Institute of Parallel and Distributed Systems(IPADS),</font></div><div><font face="黑体" color="#000000">School of Software,</font></div><div><font face="黑体" color="#000000">Shanghai Jiao Tong University</font></div></div></div></div></div></div></div></div></div></div></div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Kim Barrett <<a href="mailto:kim.barrett@oracle.com">kim.barrett@oracle.com</a>> 于2019年1月16日周三 上午12:33写道：<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">> On Dec 23, 2018, at 12:38 PM, Haoyu Li <<a href="mailto:leihouyju@gmail.com" target="_blank">leihouyju@gmail.com</a>> wrote:<br>
> <br>
> Hi all， <br>
> I have developed a patch to enhance the full GC performance of Parallel Scavenge on OpenJDK 11, may I have some reviews? The patch is described as follows and attached in this mail.<br>
> <br>
> […]<br>
> <br>
> The patch and evaluation result are attached.<br>
> <br>
> Best Regrads,<br>
> Haoyu Li,<br>
> Institute of Parallel and Distributed Systems(IPADS),<br>
> School of Software,<br>
> Shanghai Jiao Tong University<br>
<br>
Thanks for offering to contribute to the OpenJDK project.  Sorry it's<br>
taken a while to respond, but holidays &etc intervened.<br>
<br>
The idea of the patch sounds interesting.  However, before we can<br>
proceed, there may be some process issues that need to be addressed.<br>
<br>
The first, and most critical: are you covered by an Oracle Contributor<br>
Agreement (either individually or via an employer who is sponsoring<br>
this work)?  If not, that needs to be dealt with first.  For details,<br>
see <a href="https://openjdk.java.net/contribute/" rel="noreferrer" target="_blank">https://openjdk.java.net/contribute/</a>.<br>
<br>
Another is that current development is going into the jdk repo<br>
<a href="http://hg.openjdk.java.net/jdk/jdk/" rel="noreferrer" target="_blank">http://hg.openjdk.java.net/jdk/jdk/</a> (which is currently targeting JDK<br>
13), while your email suggests your patch is against the OpenJDK 11<br>
release.  This kind of change should be made in the current line of<br>
development, and then perhaps considered for backporting.  So the<br>
patch will likely need to be updated.  (It might not require all that<br>
much modification though; ParallelGC has been pretty stable for a<br>
while.)<br>
<br>
Since you don't appear to be an OpenJDK committer/author, you'll need<br>
a sponsor to help with some of the other pieces of the puzzle:<br>
<br>
Each change requires an associated bug ID, from the OpenJDK bug<br>
database.  Without a username there (which requires OpenJDK author<br>
status), your sponsor can create the CR there.<br>
<br>
Also, changes of more than some trivial extent generally get packaged<br>
for review using the webrev tool, and published on <a href="http://cr.openjdk.java.net" rel="noreferrer" target="_blank">cr.openjdk.java.net</a>.<br>
<a href="https://openjdk.java.net/guide/codeReview.html" rel="noreferrer" target="_blank">https://openjdk.java.net/guide/codeReview.html</a><br>
Your sponsor can help you with that.<br>
<br>
Your change may need additional testing (both for correctness and<br>
performance).  For example, you don't mention what platforms you've<br>
tested on, there are a lot of functional tests for OpenJDK, and there<br>
are probably other performance tests that should be applied to a<br>
change like this.<br>
<br>
For a change like this, your sponsor should be someone currently<br>
involved in OpenJDK GC development, preferably in an area related to<br>
the patch.  It's not required that they be an Oracle employee, though<br>
for some things that might help.  I can help you find someone.  (It<br>
might be me, but I don't want to commit to that today.)<br>
<br>
<br>
</blockquote></div>