RFR (M): JDK-8065993: Merge OneContigSpaceCardGeneration with TenuredGeneration
bengt.rutisson at oracle.com
Mon Dec 1 13:39:11 UTC 2014
On 2014-12-01 14:36, Mikael Gerdin wrote:
> 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.
:) Thanks for reviewing this, Mikael!
I've fixed the alignment in TenuredGeneration. Thanks for spotting that!
>> 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
>>> 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
>>> now also mentioned the removed OneContigSpaceCardGeneration. Rather
>>> updating/re-writing these comments I removed them. They did not add
>>> 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
>>> which now belongs to TenuredGeneration.
More information about the hotspot-gc-dev