Hi,<br><br>I would like few volunteers to review changes for <a href="http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7068625" target="_blank">http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7068625</a><br>WebRev: <a href="http://cr.openjdk.java.net/%7Ebrutisso/7068625/webrev.00/" target="_blank">http://cr.openjdk.java.net/~brutisso/7068625/webrev.00/</a><br>
<br>Change summary<br>For large heaps (I was focusing on 8GiB and above) it is common to have long continuous ranges of clean cards.<br>Patch is introducing a short path for skipping ranges of clean cards using word aligned memory access instead of byte aligned.<br>
<br>Patch affects serial and CMS collectors. For CMS collector stride size should be increase to see any performance gains (I was using <br><div class="MsoNormal" style="line-height:normal;margin-bottom:.0001pt;margin-bottom:0cm">
<span style="font-family:"Courier New"">-XX:+UnlockDiagnosticVMOptions   </span></div><div class="MsoNormal" style="line-height:normal;margin-bottom:.0001pt;margin-bottom:0cm"><span style="font-family:"Courier New"">-XX:ParGCCardsPerStrideChunk=4096)<br>
<br></span>For testing I was mainly using synthetic benchmark randomly modifying hash tables in heap, thus uniformly touching cards across heaps.<br>Average duration of young GC pause were used as KPI.<br>More details about testing can be found at <br>
<a href="http://blog.ragozin.info/2011/07/openjdk-patch-cutting-down-gc-pause.html">http://blog.ragozin.info/2011/07/openjdk-patch-cutting-down-gc-pause.html</a><br>Though article is referring jdk6, my resent tests with trunk jdk7 show no difference.<br>
I was also tested patch with real application (Oracle Coherence storage node).<br>With 16GiB of heap and CMS/ParNew GC, enabling patch have shortened GC pauses roughly in 2 times.<br><br>Source code of benchmark used in test are available at<br>
<a href="https://gridkit.googlecode.com/svn/branches/aragozin-sandbox/young-gc-bench">https://gridkit.googlecode.com/svn/branches/aragozin-sandbox/young-gc-bench</a><br>Main class YoungGCPauseBenchmark<br></div><br>Regards,<br>
Alexey<br>