<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 all,</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="margin: 0px;"><font face="Helvetica">I was advised to get the ParNew changes reviewed separately from the G1 changes (see 8151556). Here are just the PreservedMarks* changes + the ParNew change:</font></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;"><a href="http://cr.openjdk.java.net/~tonyp/8152312/webrev.0/">http://cr.openjdk.java.net/~tonyp/8152312/webrev.0/</a><span class="Apple-tab-span" style="white-space:pre"> </span></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;">I’ll redo the G1 changes for <span style="font-family: Helvetica;">8151556 to depend on this change.</span></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;"><span style="font-family: Helvetica;"><br></span></div><div id="bloop_customfont" style="color: rgb(0, 0, 0); margin: 0px;"><font face="Helvetica">I also piggy-backed the following change (for DefNew):</font></div><div id="bloop_customfont" style="color: rgb(0, 0, 0); margin: 0px;"><font face="Helvetica"><br></font></div><div id="bloop_customfont" style="margin: 0px;"><p style="color: rgb(0, 0, 0); margin: 0px;"><font face="Helvetica">log_debug(gc)("Promotion failed”);</font></p><p style="color: rgb(0, 0, 0); margin: 0px;"><font face="Helvetica">-></font></p><p style="color: rgb(0, 0, 0); margin: 0px;"><font face="Helvetica">log_info(gc, promotion)("Promotion failed”);</font></p><p style="color: rgb(0, 0, 0); margin: 0px;"><font face="Helvetica"><br></font></p><p style="margin: 0px;"><font face="Helvetica">so that it’s consistent with what ParNew does. Any objections?</font></p></div><div id="bloop_customfont" style="color: rgb(0, 0, 0); margin: 0px;"><font face="Helvetica"><br></font></div><div id="bloop_customfont" style="color: rgb(0, 0, 0); margin: 0px;"><font face="Helvetica">Ton</font><span style="font-family: Helvetica; font-size: 13px;">y</span></div><br><div id="bloop_sign_1458575302003209984" 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>