<div dir="ltr"><br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<div class="im"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
- instead of checking whether _eden_chunk_array is NULL to detect<br>
whether you can sample I would prefer if the same predicate as in the<br>
initialization (gch->supports_inline_contig_<u></u>alloc()) were used.<br>
(I think the change just copied what was done in the previous version of<br>
concurrentMarkSweepGeneration.<u></u>cpp:4515 has been done)<br>
<br>
Maybe create a new predicate for that, used everywhere.<br>
<br>
(btw, throughout this code there is a lot of checking whether some<br>
NEW_C_HEAP_ARRAY returned NULL or not, and does elaborate recovery -<br>
however by default NEW_C_HEAP_ARRAY fails with an OOM anyway... - but<br>
that is certainly out of scope)<br>
</blockquote>
<br></div>
Thomas, could you file a CR for fixing that?  Thanks.<div class="im"><br></div></blockquote><div><br></div><div style>Thomas, by a new predicate, do you mean checking </div><div style><span style="color:rgb(80,0,80)"><br>
</span></div><div style><span style="color:rgb(80,0,80)">  if (gch->supports_inline_contig_</span><u style="color:rgb(80,0,80)"></u><span style="color:rgb(80,0,80)">a</span><span style="color:rgb(80,0,80)">lloc())</span></div>
<div style><span style="color:rgb(80,0,80)"><br></span></div><div style><span style="color:rgb(80,0,80)">as opposed to</span></div><div style><br></div><div style><font color="#500050">  if (_eden_chunk_array != NULL)</font></div>
<div style><font color="#500050"><br></font></div><div style><font color="#500050">assuming that the NEW_C_HEAP_ARRAY succeeded?</font></div><div style><font color="#500050"><br></font></div><div style>You're right that the code checks the null-ness of <span style="color:rgb(80,0,80)">_eden_chunk_array as </span><font color="#500050">CMSCollector::sample_eden() did so.</font></div>
<div style><font color="#500050"><br></font></div><div style><font color="#500050">I see that CR </font><span style="color:rgb(80,0,80);font-family:arial,sans-serif;font-size:13px">8016276 </span><font color="#500050">was filed. Given this, but without knowing the details of the CR, it seems OK for this patch to keep the null check against </font><span style="color:rgb(80,0,80)">_eden_chunk_array as it is? </span></div>
<div><br></div></div></div></div>