<p dir="ltr">Hi John,</p>
<p dir="ltr">I think the following assert can be strengthened a bit to be:</p>
<p dir="ltr">assert((_parallel_workers == NULL && parallel_marking_threads() == 0)|| <br>
 513            parallel_marking_threads() > 0,<br>
 514            "parallel workers not set up correctly");</p>
<p dir="ltr">Don't know whether to also verify max_parallel_marking_ threads() == 0 there - may be too paranoid.</p>
<p dir="ltr">Looks good otherwise.</p>
<p dir="ltr">Thanks</p>
<p dir="ltr">Sent from my phone</p>
<div class="gmail_quote">On Jan 31, 2013 5:09 PM, "John Cuthbertson" <<a href="mailto:john.cuthbertson@oracle.com">john.cuthbertson@oracle.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Everyone,<br>
<br>
Here's a new webrev based upon feedback from Vitaly and Bengt: <a href="http://cr.openjdk.java.net/~johnc/8005875/webrev.1/" target="_blank">http://cr.openjdk.java.net/~<u></u>johnc/8005875/webrev.1/</a><br>
<br>
I've wrapped the check with the asserts suggested by Bengt into a small routine.<br>
<br>
Testing:<br>
GC test suite with ConcGCThreads=3<br>
Kitchensink with ParallelGCThreads=0:<br>
<br>
Thanks,<br>
<br>
JohnC<br>
<br>
On 1/8/2013 2:13 PM, John Cuthbertson 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 please have a couple of volunteers look over the fix for this CR - the webrev can be found at: <a href="http://cr.openjdk.java.net/~johnc/8005875/webrev.0/" target="_blank">http://cr.openjdk.java.net/~<u></u>johnc/8005875/webrev.0/</a><br>

<br>
Summary:<br>
One of the modules in the Kitchensink test generates a VM_PrintThreads vm operation. The JVM crashes when it tries to print out G1's concurrent marking worker threads when ParallelGCThreads=0 because the work gang has not been created. The fix is to add the same check that's used elsewhere in G1's concurrent marking.<br>

<br>
Testing:<br>
Kitchensink with ParallelGCThreads=0<br>
<br>
Thanks,<br>
<br>
JohnC<br>
</blockquote>
<br>
</blockquote></div>