Request for review: 8007763: Refactoring: split up compute_generation_free_space() into two functions for class PSAdaptiveSizePolicy

[Thomas Schatzl]

Hi Tao,

On Mon, 2013-05-06 at 14:46 -0700, Tao Mao wrote:
> 8007763: Refactoring: split up compute_generation_free_space() into
> two functions for class PSAdaptiveSizePolicy
> http://bugs.sun.com/view_bug.do?bug_id=8007763 
> 
> webrev: 
> http://cr.openjdk.java.net/~tamao/8007763/webrev.00/ 

The change looks okay after thorough visual inspection and given your
testing there seem to be no regressions.

One note: A large part of the code surrounding the change in
psMarkSweep/psParallelCompact.cpp looks very much the same. Is there a
way to factor out something here? (Probably not as there is no common
ancestor...).
Consider it okay from me anyway, that has been the case before too.

> changeset: 
> Split up compute_generation_free_space() into two functions:
> compute_eden_space_size() + compute_old_gen_free_space(), each of
> which (if needed, e.g.
> src/share/vm/gc_implementation/parallelScavenge/psScavenge.cpp) can be
> reused without executing an overhead of the other.
> 
> The changeset is also related to JDK-8007763 (Rename a bunch of
> methods in size policy across collectors).

I will look at 8007762 too.

Sorry for the delay,
Thomas