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

[Tao Mao]

Thank you for review, Thomas.

Tao

On 5/14/13 5:06 AM, Thomas Schatzl wrote:
> 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
>
>