The high level idea looks correct. I'll look at the details in a bit (seriously this time; sorry it dropped off my plate last time I promised).<br>Does anyone have a pointer to the related discussion thread on this aias from earlier in the year, by chance, so one could refresh one's<br>
memory of that discussion?<br><br>Also you might want to check this with a very long CMSWaitDuration value and +ExplicitGCInvokesConcurrent, just to further sanity check.<br><br>thanks!<br>-- ramki<br><br><div class="gmail_quote">
On Mon, Oct 29, 2012 at 9:37 AM, John Cuthbertson <span dir="ltr"><<a href="mailto:john.cuthbertson@oracle.com" target="_blank">john.cuthbertson@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 Everyone,<br>
<br>
Can I have a couple of volunteers review this change that was submitted by Michal Frajt? The webrev can be found at: <a href="http://cr.openjdk.java.net/%7Ejohnc/7189971/webrev.0/" target="_blank">http://cr.openjdk.java.net/~<u></u>johnc/7189971/webrev.0/</a><br>

<br>
In the original patch, Michal was returning (if the CMS thread was terminating or if a foreground collection was scheduled) after setting the CMS flag. Michal incorporated my suggestion to change that and return before setting the CMS flag.<br>

<br>
I'm not an expert in the inner workings of CMS so I'd appreciate further reviews from those more familiar with  CMS' operation.<br>
<br>
Testing:<br>
Functional testing was performed using the GC test suite in both incremental and non-incremental modes with various values of CMSWaitDuration and checking the log files.  Additional tests: nsk, jprt.<br>
<br>
Thanks,<br>
<br>
JohnC<br>
</blockquote></div><br>