RFR (L): JDK-8061308: Remove iCMS
martijnverburg at gmail.com
Fri Oct 17 13:33:52 UTC 2014
As an outsider... nice work!! Anything that makes the GC subsystem more
On 17 October 2014 12:43, Bengt Rutisson <bengt.rutisson at oracle.com> wrote:
> Hi everyone,
> Can I have a couple of reviews for this change?
> HotSpot changes:
> JDK changes:
> (Just a small update to a test library class.)
> JBS bug:
> The change removes the incremental mode of CMS. This work is part of JEP
> 214 (http://openjdk.java.net/jeps/214). There are quite a few lines
> changed. Mostly it is just removal of code and it should hopefully be
> fairly straight forward to review.
> One interesting side effect of removing iCMS was that the ConcEdenSpace (a
> subclass in the Space hierarchy) could be removed. It implemented the
> "soft-end" feature that only iCMS was taking advantage of. When I had
> removed ConcEdenSpace it turned out that its superclass, EdenSpace, was
> actually an unnecessary class that did not add anything so I removed that
> too. When I then went to update the comment in space.hpp that describes the
> Space hierarchy I noticed that the description of the hierarchy was
> completely out-dated. Rather than bringing it up-to-date I removed the
> comment. I think that type of information will only get out-dated quickly
> again and it is much easier to let your IDE create the correct type
> hierarchy view dynamically for you.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the hotspot-gc-dev