<html><head><style>body{font-family:Helvetica,Arial;font-size:13px}</style></head><body style="word-wrap:break-word;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">Thomas and Per,</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">Thanks! I’ll submit it to the submit repo, just in case (and to refresh my memory on how to do that), and I’ll push it after I get the results...</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> <div id="bloop_sign_1557255821270884096" class="bloop_sign"><div><br></div><div><div>—————</div><div>Tony Printezis | @TonyPrintezis | <a href="mailto:tprintezis@twitter.com">tprintezis@twitter.com</a></div></div><div><br></div></div> <br><p class="airmail_on">On May 7, 2019 at 2:20:25 AM, Per Liden (<a href="mailto:per.liden@oracle.com">per.liden@oracle.com</a>) wrote:</p> <blockquote type="cite" class="clean_bq"><span><div><div></div><div>Hi Tony,
<br>
<br>On 5/6/19 7:27 PM, Tony Printezis wrote:
<br>> Trivial one-line change:
<br>>
<br>> <a href="http://cr.openjdk.java.net/~tonyp/8223397/webrev.0/">http://cr.openjdk.java.net/~tonyp/8223397/webrev.0/</a>
<br>>
<br>> Was there a reason why gc+promotion was not added to LOG_PREFIX_LIST in
<br>> the first place?
<br>
<br>I'm pretty sure that was just a mistake.
<br>
<br>Patch looks good.
<br>
<br>cheers,
<br>Per
<br>
<br>>
<br>>
<br>> Before:
<br>>
<br>> [59.899s][info ][gc,task ] GC(17) Using 18 workers of 18 for
<br>> evacuation
<br>>
<br>> [61.934s][info ][gc,promotion ] Promotion failed
<br>>
<br>> [61.934s][info ][gc ] GC(17) Pause Young (Allocation
<br>> Failure) 3948M->4555M(4557M) 2035.171ms
<br>>
<br>>
<br>>
<br>> After:
<br>>
<br>> [55.339s][info ][gc,task ] GC(17) Using 18 workers of 18 for
<br>> evacuation
<br>>
<br>> [57.383s][info ][gc,promotion ] GC(17) Promotion failed
<br>>
<br>> [57.383s][info ][gc ] GC(17) Pause Young (Allocation
<br>> Failure) 3944M->4556M(4557M) 2044.354ms
<br>>
<br>>
<br>>
<br>> Thanks,
<br>>
<br>> Tony
<br>>
<br>> —————
<br>> Tony Printezis | @TonyPrintezis | <a href="mailto:tprintezis@twitter.com">tprintezis@twitter.com</a>
<br>> <mailto:<a href="mailto:tprintezis@twitter.com">tprintezis@twitter.com</a>>
<br>>
<br></div></div></span></blockquote></body></html>