RFR(S) 8038404: Move object_iterate_mem from Space to CMS since it is only ever used by CMS
Thomas Schatzl
thomas.schatzl at oracle.com
Thu Mar 27 07:55:55 UTC 2014
Hi,
On Wed, 2014-03-26 at 17:21 +0100, Mikael Gerdin wrote:
> Hi,
>
> I'm doing some cleanups in the "Space" classes to simplify the code for the G1
> class unloading work[1].
> Here's a change to move CMS specific code to CompactibleFreeListSpace:
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8038404
> Webrev: http://cr.openjdk.java.net/~mgerdin/8038404/webrev.0
- Could you also move the documentation of Space::object_iterate_mem()
to CompactibleFreeListSpace::object_iterate_mem()?
- the UpwardsObjectClosure also seems to be only used by
Space::object_iterate_mem() too, and hence CMS. Is there some place to
move it from the generic iterator.hpp to a CMS file?
Thanks for the cleanup,
Thomas
More information about the hotspot-gc-dev
mailing list