<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">
> > Same for checking whether _survivor_chunk_array is NULL - at the moment<br>
> > sometimes _survivor_chunk_array is checked against NULL to avoid<br>
> > entering code (in print_eden_and_survivor_chunk_arrays()).<br>
> ><br>
> > The condition that could be used is the one that triggers allocation of<br>
> > _survivor_chunk_array: (CMSParallelRemarkEnabled &&<br>
> > CMSParallelSurvivorRemarkEnabled) in<br>
> > concurrentMarkSweepGeneration.cpp:728.<br>
><br>
> Personally, I like a check like if (_eden_chunk_array != NULL)<br>
> better. I think it is more future proof.   For whatever reason we don't<br>
> try to dereference _eden_chunk_array if it is NULL.  An assertion<br>
> checking that<br>
><br>
> (CMSParallelRemarkEnabled &&<br>
> CMSParallelSurvivorRemarkEnabled)<br>
><br>
> is true would we be appropriate.<br>
<br>
</div>Also good.<br></blockquote><div><br></div><div style>Do you mean adding an assert in CMSCollector::print_eden_and_survivor_chunk_arrays() like the following:</div><div style><br></div><div style>  if (_survivor_chunk_array != NULL) {<br>
</div><div style>     assert(<span style="color:rgb(80,0,80)">CMSParallelRemarkEnabled && </span><span style="color:rgb(80,0,80)">CMSParallelSurvivorRemarkEnabl</span><span style="color:rgb(80,0,80)">ed, "Error");</span></div>
<div style>     ...</div><div style>  }</div><div style><br></div><div style>?</div><div style><br></div><div style>That can be easily inserted.</div><div><br></div></div></div></div>