<div dir="ltr">Andrew,<div><br></div><div>I realize this is a correctness fix, but isn't this going to possibly defeat any perf gain from using conditional card marking in the first place (for CMS)? Didn't someone suggest a different approach that allows store-store to be used still?</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, May 5, 2015 at 10:52 AM, Andrew Haley <span dir="ltr"><<a href="mailto:aph@redhat.com" target="_blank">aph@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I've added StoreLoad barriers everywhere they're needed.<br>
<br>
<a href="http://cr.openjdk.java.net/~aph/8079315/" target="_blank">http://cr.openjdk.java.net/~aph/8079315/</a><br>
<br>
This patch depends on the patch for 8078438, which is still not<br>
committed.  <a href="http://cr.openjdk.java.net/~shade/8078438/webrev.02/" target="_blank">http://cr.openjdk.java.net/~shade/8078438/webrev.02/</a><br>
<br>
This is x86 only, will do AArch64 parts in a separate patch once<br>
we've agreed on this.<br>
<span class="HOEnZb"><font color="#888888"><br>
Andrew.<br>
</font></span></blockquote></div><br></div>