<html>
<head>
<meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
Ok. Thanks Jon!<br>
<br>
regards,<br>
Poonam<br>
<br>
On 4/18/2014 7:36 AM, Jon Masamitsu wrote:
<blockquote cite="mid:535088A5.2040501@oracle.com" type="cite">
<meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
<br>
<div class="moz-cite-prefix">On 4/17/2014 6:53 PM, Poonam Bajaj
wrote:<br>
</div>
<blockquote cite="mid:53508590.6060201@oracle.com" type="cite">
<meta content="text/html; charset=UTF-8"
http-equiv="Content-Type">
Hi Jon,<br>
<br>
On 4/17/2014 10:07 PM, Jon Masamitsu wrote:
<blockquote cite="mid:53500340.4050702@oracle.com" type="cite">
<meta content="text/html; charset=UTF-8"
http-equiv="Content-Type">
Poonam,<br>
<br>
In jdk7 I think there is a <br>
<br>
GenCollectedHeap::print_perm_heap_change()<br>
<br>
Your changes have added in G1CollectorPolicy<br>
<pre><span class="new">1168 void G1CollectorPolicy::print_perm_heap_change(size_t perm_prev_used) const {
</span><span class="new"></span>
</pre>
Is there still a GenCollectedHeap::print_perm_heap_change()
in the jdk7<br>
you're fixing and could you have used that instead of adding
the<br>
print_perm_heap_change() to G1CollectorPolicy.<br>
</blockquote>
<br>
Yes, there is print_perm_heap_change() in GenCollectedHeap
class. But according to the following hierarchy of classes, I
could not use it in G1CollectedHeap<br>
<br>
//<br>
// CollectedHeap<br>
// SharedHeap<br>
// GenCollectedHeap<br>
// G1CollectedHeap<br>
// ParallelScavengeHeap<br>
//<br>
<br>
So I added the similar function in G1CollectorPolicy. I can move
it to G1CollectedHeap too. And if we don't want duplication, I
can explore the possibility of moving it to SharedHeap and then
use it in both the derived classes.<br>
</blockquote>
<br>
I would recommend against moving things to SharedHeap. That might
complicate other backports<br>
and since perm gen is gone in jdk9, I think your solution is fine.<br>
<br>
Reviewed.<br>
<br>
Jon<br>
<br>
<blockquote cite="mid:53508590.6060201@oracle.com" type="cite"> <br>
Thanks,<br>
Poonam<br>
<br>
<blockquote cite="mid:53500340.4050702@oracle.com" type="cite">
<br>
<br>
<pre><span class="new"></span></pre>
Jon<br>
<br>
<br>
<br>
<div class="moz-cite-prefix">On 4/16/2014 6:03 PM, Poonam
Bajaj wrote:<br>
</div>
<blockquote cite="mid:534F2877.50303@oracle.com" type="cite">
<meta http-equiv="content-type" content="text/html;
charset=UTF-8">
Hi,<br>
<br>
Could I have reviews for the following fix:<br>
<br>
<a moz-do-not-send="true"
href="https://bugs.openjdk.java.net/browse/JDK-8010738">JDK-8010738</a>:
G1: Output for full GCs with +PrintGCDetails should contain
perm gen/meta data size change info<br>
webrev: <a moz-do-not-send="true"
class="moz-txt-link-freetext"
href="http://cr.openjdk.java.net/%7Epoonam/8010738/webrev.00/">http://cr.openjdk.java.net/~poonam/8010738/webrev.00/</a><br>
<br>
With JDK7 G1GC when we use -XX:+PrintGCDetails option,
PermGen information does not get printed in the Full GC logs
as shown below:<br>
[Full GC 16M->16M(20M), 0.7055536 secs]<br>
[Eden: 0.0B(1024.0K)->0.0B(1024.0K) Survivors:
0.0B->0.0B Heap: 16.7M(20.0M)->16.7M(20.0M)]<br>
<br>
PermGen info should also get printed with Full GC info
similar to what other garbage collectors report. These
changes are backport of the fix in jdk8 where MetaSpace info
gets printed in the Full GC logs.<br>
<br>
With the fix the output looks like:<br>
[Full GC 16M->16M(20M), 0.6763478 secs]<br>
[Eden: 0.0B(1024.0K)->0.0B(1024.0K) Survivors:
0.0B->0.0B Heap: 16.7M(20.0M)->16.7M(20.0M)], [Perm:
1725K->1725K(16384K)]<br>
<br>
<br>
Thanks,<br>
Poonam<br>
<br>
</blockquote>
<br>
</blockquote>
</blockquote>
<br>
</blockquote>
</body>
</html>