RFR (M): JDK-8065993: Merge OneContigSpaceCardGeneration with TenuredGeneration

Bengt Rutisson bengt.rutisson at oracle.com
Mon Dec 1 12:42:24 UTC 2014


Hi everyone,

Could I have a couple of reviews to this cleanup?

http://cr.openjdk.java.net/~brutisso/8065993/webrev.00/

https://bugs.openjdk.java.net/browse/JDK-8065993

During the review of "8064702: Remove the CMS foreground collector" 
Mikael made the comment that we can merge OneContigSpaceCardGeneration 
with TenuredGeneration. This is the change to address that comment.

The motivation to have OneContigSpaceCardGeneration was that the PermGen 
implementation inherited from that class too. When the PermGen was 
removed TenuredGenteration was the only implementation that inherited 
OneContigSpaceCardGeneration.

These classes were exposed in the serviceability agent, so I've made 
changes there too. In the SA the classes are use by for example the 
"jmap -heap" command. Thus, I have been using that command for SA testing.

I also found a couple  of comments that still mentioned PermGen and that 
now also mentioned the removed OneContigSpaceCardGeneration. Rather than 
updating/re-writing these comments I removed them. They did not add much 
to the understanding of the code anymore in my opinion.

Note that I renamed generation.inline.hpp to tenuredGeneration.inline 
since it only contained inline fuctions for OneContigSpaceCardGeneration 
which now belongs to TenuredGeneration.

Thanks,
Bengt



More information about the hotspot-gc-dev mailing list