RFR (M): JDK-8065993: Merge OneContigSpaceCardGeneration with TenuredGeneration
mikael.gerdin at oracle.com
Mon Dec 1 13:36:13 UTC 2014
On 2014-12-01 14:31, Mikael Gerdin wrote:
> Hi Bengt,
> On 2014-12-01 13:42, Bengt Rutisson wrote:
>> Hi everyone,
>> Could I have a couple of reviews to this cleanup?
> in concurrentMarkSweepGeneration.inline.hpp you've removed the include
> of genCollectedHeap.hpp, is that intentional? There are still references
> to GenCollectedHeap in that file.
Disregard this comment, for some reason I saw this change backwards.
Adding it is of course correct.
> in tenuredGeneration.cpp
> TenuredGeneration::collect and expand_and_allocate the parameters are
> not properly aligned.
> Otherwise the change looks good.
> I don't need to re-review the above mentioned changes.
>> 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
>> 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
>> 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.
More information about the hotspot-gc-dev