<div>
                    Looks good.
                </div>
                <div><div><br></div><div><br></div><div>igor</div><div><br></div></div>
                 
                <p style="color: #A0A0A8;">On Wednesday, February 22, 2012 at 12:10 PM, John Cuthbertson wrote:</p>
                <blockquote type="cite" style="border-left-style:solid;border-width:1px;margin-left:0px;padding-left:10px;">
                    <span><div><div><div>Hi Everyone,</div><div><br></div><div>Can I have a couple of volunteers to review the changes for this CR? The </div><div>webrev can be found at: <a href="http://cr.openjdk.java.net/~johnc/7147806/webrev.0/">http://cr.openjdk.java.net/~johnc/7147806/webrev.0/</a></div><div><br></div><div>The issues here was that some updates to the G1 performance counter </div><div>variables were not guarded. In the other collectors, the generation and </div><div>collector performance counters are allocated/initialized </div><div>unconditionally. The performance counter variables that these counters </div><div>contain, however, are allocated and updated only if UsePerfData is </div><div>enabled. In G1, some of these updates were not suitably guarded and so </div><div>we tried to update some unallocated performance counter variables.</div><div><br></div><div>Testing:</div><div>* Small test case (gcbasher) with the offending command line flag </div><div>combination and with just G1.</div><div><br></div><div>Thanks,</div><div><br></div><div>JohnC</div></div></div></span>
                 
                 
                 
                 
                </blockquote>
                 
                <div>
                    <br>
                </div>