<div dir="ltr"><div dir="ltr">Thanks for your response Kim!<div><br></div><div>For the process issues you mentioned, I have already been covered by an Oracle Contributor Agreement, and I will right away port this patch into OpenJDK 13 to follow the current line, then submit the updated patch to this mail list once I finish.</div><div><br></div><div>Also as you mentioned, my change need additional testing for both correctness and performance. Can I please get some specific suggestions (like what benchmark or method I need to test the patch with)? Currently I test it with the benchmarks from DaCapo, SPECjvm2008, and JOlden suits on a machine with an Intel x86-64 CPU and a Linux OS. I can only find a suggestion from the page <a href="https://openjdk.java.net/contribute/">https://openjdk.java.net/contribute/</a> which says a patch need a unit test with jreg harness, but it also says the unit test is not pratical for performance improvement changes. It really helps if you can offer me some tips about tests.</div><div><br></div><div>Last but not the least, I really appreciate that you help me find a sponsor or play the role of the sponsor.</div><div><br></div><div>Best,</div><div>Haoyu Li</div><div><br></div><div class="gmail_quote"><div dir="ltr">Kim Barrett <<a href="mailto:kim.barrett@oracle.com" target="_blank">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></div></div>