<html><head><style>body{font-family:Helvetica,Arial;font-size:13px}</style></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;">Hi Thomas,</div><div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;"><br></div><div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;">Cool, thanks. So, what else should we do on this front? Do you want me to also refactor G1 to use the new abstractions? Or are you guys happy with the way G1 is right now (last time I looked at the code I thought there was some opportunity for re-use). Also, there’s an additional change to cut down on the number of preserved marks that are pushed on he stacks (given that, as we discussed on a separate e-mail thread, when biased locking is enabled most objects do not have 0x1 as their mark word). Interested in both?</div><div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;"><br></div><div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;">Tony</div> <br><p class="airmail_on">On March 4, 2016 at 11:52:34 AM, Thomas Schatzl (<a href="mailto:thomas.schatzl@oracle.com">thomas.schatzl@oracle.com</a>) wrote:</p> <blockquote type="cite" class="clean_bq"><span><div><div></div><div>Hi,
<br>
<br>On Fri, 2016-03-04 at 11:14 -0500, Tony Printezis wrote:
<br>> Thomas,
<br>>
<br>> I confirmed that the code below was incorrect and doing the addition
<br>> before calling restore() fixes the issue. Latest webrev here:
<br>>
<br>> http://cr.openjdk.java.net/~tonyp/8146991/webrev.2/
<br>>
<br>
<br> looks good to me. JPRT has been fine, although I messed up running
<br>the other tests - I forgot to specify ParallelGC as collector to use.
<br>
<br>I will redo that on Monday. I do not expect any issues.
<br>
<br>Thanks,
<br> Thomas
<br>
<br></div></div></span></blockquote> <div id="bloop_sign_1457127455789350912" class="bloop_sign"><div style="font-family:helvetica,arial;font-size:13px"><div>-----</div><div><br></div><div>Tony Printezis | JVM/GC Engineer / VM Team | Twitter</div><div><br></div><div>@TonyPrintezis</div><div><a href="mailto:tprintezis@twitter.com">tprintezis@twitter.com</a></div><div><br></div></div></div></body></html>