<html>
<head>
<meta http-equiv="content-type" content="text/html; charset=ISO-8859-1">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<br>
Hi everyone,<br>
<br>
Could I have a couple of reviews for this small change?<br>
<br>
<a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~brutisso/8025605/webrev.00/">http://cr.openjdk.java.net/~brutisso/8025605/webrev.00/</a><br>
<br>
Bug:<br>
<a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8025605">https://bugs.openjdk.java.net/browse/JDK-8025605</a><br>
<br>
Background:<br>
<br>
<meta http-equiv="content-type" content="text/html;
charset=ISO-8859-1">
When running G1 with -XX:+PrintAdaptiveSizePolicy some information
about the heap sizing is logged. When the heap is fully expanded but
the ergonomics would have liked to expand even more the following
message is logged:
<br>
<br>
[G1Ergonomics (Heap Sizing) did not expand the heap, reason: heap
expansion operation failed]
<br>
<br>
It sounds like something failed, but in reality we just hit the -Xmx
value. We should have a more informative message that does not look
like a serious failure. <br>
<br>
With my suggested patch the message will instead be:<br>
<br>
<meta http-equiv="content-type" content="text/html;
charset=ISO-8859-1">
[G1Ergonomics (Heap Sizing) did not expand the heap, reason: heap
already fully expanded] <br>
<br>
Thanks,<br>
Bengt<br>
<br>
<br>
</body>
</html>