<div dir="ltr">Here's an updated webrev: <a href="http://cr.openjdk.java.net/~hiroshi/webrevs/edenchunks/webrev.01/">http://cr.openjdk.java.net/~hiroshi/webrevs/edenchunks/webrev.01/</a><div><br></div><div style>Let me know if there are more comments.</div>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Jun 21, 2013 at 3:50 PM, Jon Masamitsu <span dir="ltr"><<a href="mailto:jon.masamitsu@oracle.com" target="_blank">jon.masamitsu@oracle.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Yes, remove the assert.<span class="HOEnZb"><font color="#888888"><br>
<br>
Jon</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
On 6/19/2013 10:39 AM, Hiroshi Yamauchi wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

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>
</blockquote>
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>
<br>
</blockquote>
Removing the assert sounds good to me.<br>
<br>
Jon, what do you think?<br>
<br>
</blockquote>
<br>
</div></div></blockquote></div><br></div>