<div dir="ltr">I will fix this. Thanks.<br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jun 27, 2013 at 2:15 AM, Thomas Schatzl <span dir="ltr"><<a href="mailto:thomas.schatzl@oracle.com" target="_blank">thomas.schatzl@oracle.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Hiroshi,<br>
<div class="im"><br>
On Tue, 2013-06-25 at 19:05 -0700, Hiroshi Yamauchi wrote:<br>
> Here's an updated<br>
> webrev: <a href="http://cr.openjdk.java.net/~hiroshi/webrevs/edenchunks/webrev.01/" target="_blank">http://cr.openjdk.java.net/~hiroshi/webrevs/edenchunks/webrev.01/</a><br>
><br>
> Let me know if there are more comments.<br>
<br>
</div>Minor nitpick:<br>
<br>
concurrentMarkSweepGeneration.cpp:<br>
<br>
4516   if (_eden_chunk_array && !CMSEdenChunksRecordAlways) {<br>
<br>
The != NULL has been forgotten here.<br>
<br>
Thanks for considering to use the mutex in<br>
CMSCollector::sample_eden_chunk(). It looks a lot cleaner now.<br>
<br>
Looks good.<br>
<br>
Thanks,<br>
  Thomas<br>
<br>
<br>
<br>
</blockquote></div><br></div></div>