<html>
<head>
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<font face="Times New Roman, Times, serif">Thomas,<br>
<br>
Change looks good.<br>
<br>
Would this edit of the Description<br>
<br>
"<big>This leads to PLAB size adjustment appearing to be for the<br>
next GC</big>."<br>
<br>
be better than the current<br>
<br>
"</font>This leads to the PLAB size adjustment being one GC too
late."<br>
<br>
Jon<br>
<br>
<br>
<div class="moz-cite-prefix">On 10/28/2015 01:44 PM, Thomas Schatzl
wrote:<br>
</div>
<blockquote cite="mid:1446065057.2078.19.camel@oracle.com"
type="cite">
<pre wrap="">Hi all,
can I have reviews for this bug M. Chernov reported a few days ago?
PLAB statistics flushing (i.e. updating the G1EvacStats data structure)
is done too late so that the PrintPLAB printout shows and uses the
previous' GCs values, which may negatively affect PLAB statistics
measurements.
CR:
<a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8140585">https://bugs.openjdk.java.net/browse/JDK-8140585</a>
Webrev:
<a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~tschatzl/8140585/webrev">http://cr.openjdk.java.net/~tschatzl/8140585/webrev</a>
Testing:
jtreg, vm.gc, new jtreg test
Thanks,
Thomas
</pre>
</blockquote>
<br>
</body>
</html>