<div dir="ltr">Hi Goetz,<div><br></div><div><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Aug 22, 2017 at 3:19 PM, Lindenmaier, Goetz <span dir="ltr"><<a href="mailto:goetz.lindenmaier@sap.com" target="_blank">goetz.lindenmaier@sap.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi,<br>
<br>
Could I please get reviews for this small change? I also please need a sponsor.<br>
<a href="http://cr.openjdk.java.net/~goetz/wr17/8186437-compThrLock/" rel="noreferrer" target="_blank">http://cr.openjdk.java.net/~<wbr>goetz/wr17/8186437-<wbr>compThrLock/</a><br>
<br>
When the VM is aborted because compiler thread creation fails (seen in<br>
TestOprionsWithRanges with huge stack size) the Thread_lock was not<br>
released.<br>
<br></blockquote><div><br></div><div>Seems fine, if a bit difficult to read. Alternatively, one could have unlocked manually<span style="color:rgb(0,0,0)"> - after all, we are never leaving this function, so the ~MutexLocker. That would have preserved the control.</span></div><div><br></div><div>Cheers, Thomas</div><div><br></div><div><br></div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Best regards,<br>
  Goetz.<br>
</blockquote></div><br></div></div>