<div dir="ltr">Hi Poonam --<div><br></div><div>Awesome, thanks for doing the backport (and thanks to Yasumasa for the original fixes)!  Are there performance numbers to share? (Unfortunately the original jira's aren't publicly visible on openjdk, in case performance data is attached there.)<div>
<br></div><div>Which version of 7u is this slated to make an appearance?</div><div><br></div><div>thanks!</div><div>-- ramki</div><div><br></div></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Feb 12, 2014 at 6:14 PM, Poonam Bajaj <span dir="ltr"><<a href="mailto:poonam.bajaj@oracle.com" target="_blank">poonam.bajaj@oracle.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
  
    
  
  <div bgcolor="#FFFFFF" text="#000000">
    Hi,<br>
    <br>
    Could I have some reviews on the following backport changes, please.<br>
    <a href="http://cr.openjdk.java.net/%7Epoonam/6412968_6990419/webrev.01/" target="_blank">http://cr.openjdk.java.net/~poonam/6412968_6990419/webrev.01/</a><br>
    <br>
    Thanks,<br>
    Poonam<div><div class="h5"><br>
    <br>
    On 2/11/2014 10:30 AM, Poonam Bajaj wrote:
    <blockquote type="cite">
      
      Hi,<br>
      <br>
      As per Jon Masamitsu's suggestion, we would be having the
      CMSParallelInitialMarkEnabled and CMSEdenChunksRecordAlways flags'
      value 'false' by default for 7u so as not to introduce any
      behavior changes in 7 update release. The updated webrev is here:<br>
      <a href="http://cr.openjdk.java.net/%7Epoonam/6412968_6990419/webrev.01/" target="_blank">http://cr.openjdk.java.net/~poonam/6412968_6990419/webrev.01/</a><br>
      <br>
      Thanks,<br>
      Poonam<br>
      <br>
      On 2/9/2014 5:13 PM, Poonam Bajaj wrote:
      <blockquote type="cite">
        
        Hi All,<br>
        <br>
        Could I have some reviews for the backport of the following two
        fixes from jdk8 to 7u:<br>
        <a href="https://bugs.openjdk.java.net/browse/JDK-6412968" target="_blank">6412968

        </a>: CMS: Long initial mark pauses <br>
        <a href="https://bugs.openjdk.java.net/browse/JDK-6990419" target="_blank">6990419</a>:
        CMS: Remaining work for 6572569: consistently skewed work
        distribution in (long) re-mark pauses <br>
        <br>
        These changes parallelize the initial-mark phase of the CMS
        collector and improve the work distribution among parallel
        worker threads.<br>
        Webrev: <a href="http://cr.openjdk.java.net/%7Epoonam/6412968_6990419/webrev.00/" target="_blank">http://cr.openjdk.java.net/~poonam/6412968_6990419/webrev.00/</a><br>
        <br>
        Original fixes:<br>
        6412968 : <a href="http://hg.openjdk.java.net/hsx/hsx25/hotspot/rev/ca9dedeebdec" target="_blank">http://hg.openjdk.java.net/hsx/hsx25/hotspot/rev/ca9dedeebdec</a><br>
        6990419: <a href="http://hg.openjdk.java.net/hsx/hsx25/hotspot/rev/7b06ae405d7b" target="_blank">http://hg.openjdk.java.net/hsx/hsx25/hotspot/rev/7b06ae405d7b</a><br>
        <br>
        Testing:<br>
        JPRT, Customer verification testing<br>
        <br>
        Thanks,<br>
        Poonam<br>
        <br>
        <br>
        <br>
      </blockquote>
    </blockquote>
  </div></div></div>

</blockquote></div><br></div>