<div dir="ltr"><br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">
> Now, if I comment out the assert, it seems to work (though I haven't<br>
> tested it very long.) This may be good if in fact sneak won't happen<br>
> with try_lock()/unlock() only.<br>
><br>
> I haven't tried this, but another potential approach might be to give<br>
> up sampling (just return) if it's called by the VM thread at a<br>
> safepoint, though the VM thread might allocate a large object, and the<br>
> evenness of the sample distribution could suffer to some extent.<br>
<br>
</div>I'd just remove the assert :)<br>
<br>
As already mentioned multiple times, try_lock() does never actually<br>
block the thread, so the vm thread cannot actually sneak, as you already<br>
noticed in your question to David, and he confirmed. The guarded code<br>
can also never block, it's just a few assignments.<br></blockquote><div><br></div><div style>Removing the assert sounds good to me. </div><div style><br></div><div style>Jon, what do you think?</div><div style><br></div>
</div></div></div>