[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-serviceability-dev
Subject:    Re: RFR: JDK-8163581: Heap Parameters in HSDB cannot handle G1CollectedHeap
From:       Yasumasa Suenaga <yasuenag () gmail ! com>
Date:       2016-08-16 12:31:47
Message-ID: c480e946-a0cd-d3b2-0dac-47723c06b711 () gmail ! com
[Download RAW message or body]

Hi all,

Have you checked changes for this issue?

   http://cr.openjdk.java.net/~ysuenaga/JDK-8163581/webrev.01/hotspot/
   http://cr.openjdk.java.net/~ysuenaga/JDK-8163581/webrev.01/jdk/

For example, ParallelScavengeHeap.java overrides printOn() method.
This method shows addresses of each memory regions.

My proposal shows memory region of G1CollectedHeap and region size.
Please review and sponsor for it. If it is not enough, please tell me.


Thanks,

Yasumasa


On 2016/08/11 23:15, Yasumasa Suenaga wrote:
> Hi Jini,
> 
> I added this check in new webrev:
> 
> http://cr.openjdk.java.net/~ysuenaga/JDK-8163581/webrev.01/jdk/
> 
> 
> Thanks,
> 
> Yasumasa
> 
> 
> On 2016/08/11 15:33, Jini Susan George wrote:
> > Thank you for this fix, Yasumasa. This issue was manifested through the \
> > 'universe' command in clhsdb also, and your change fixes that. It would be great \
> > if you could modify BasicLauncherTest.java to have some testing done for this \
> > also. 
> > Thank you,
> > Jini (Not a Reviewer).
> > 
> > > -----Original Message-----
> > > From: David Holmes
> > > Sent: Thursday, August 11, 2016 10:03 AM
> > > To: Yasumasa Suenaga; serviceability-dev@openjdk.java.net; hotspot-gc-
> > > dev@openjdk.java.net
> > > Subject: Re: RFR: JDK-8163581: Heap Parameters in HSDB cannot handle
> > > G1CollectedHeap
> > > 
> > > Hi Yasumasa,
> > > 
> > > Adding in GC folk.
> > > 
> > > I agree with the need to override the printOn method, but can't comment
> > > on the details of what you are actually printing.
> > > 
> > > Thanks,
> > > David
> > > 
> > > On 10/08/2016 10:20 PM, Yasumasa Suenaga wrote:
> > > > Hi all,
> > > > 
> > > > When I chose "Heap Parameters" menu in HSDB, I got following value:
> > > > --------------------
> > > > Heap Parameters:
> > > > unknown subtype of CollectedHeap @ 0x00007f4d4c030510
> > > > (0x00000006c6e00000,0x00000007c0000000)
> > > > --------------------
> > > > 
> > > > I think it should not be "unknown subtype".
> > > > 
> > > > This value is shown by sun.jvm.hotspot.gc.shared.CollectedHeap#printOn().
> > > > This method is overrided in ParallelScavengeHeap and GenCollectedHeap.
> > > > However, G1CollectedHeap does not override.
> > > > 
> > > > I think G1CollectedHeap should override this method.
> > > > 
> > > > I uploaded a webrev for this issue.
> > > > Could you review it?
> > > > 
> > > > http://cr.openjdk.java.net/~ysuenaga/JDK-8163581/webrev.00/
> > > > 
> > > > I'm jdk 9 committer, but I cannot access JPRT.
> > > > So I need a sponsor.
> > > > 
> > > > 
> > > > Thanks,
> > > > 
> > > > Yasumasa
> > > > 


[prev in list] [next in list] [prev in thread] [next in thread]