RFR (M): 8060697: Improve G1 Heap Growth Heuristics

[Tom Benson]

Hi Thomas,
OK, thanks very much.
Now looking for a sponsor to push the changes...  Thomas, Kim, or Jon?
Tnx,
Tom

On 12/9/2015 4:57 AM, Thomas Schatzl wrote:
> Hi,
>
> On Mon, 2015-12-07 at 20:35 -0500, Tom Benson wrote:
>> Hi Kim,
>> Yes, that's it exactly.  I'm not bothered by it, but am open to
>> suggestions if someone feels strongly.
>> Tnx,
>> Tom
>>
>> On 12/7/2015 7:30 PM, Kim Barrett wrote:
>>> On Dec 3, 2015, at 11:40 AM, Tom Benson <tom.benson at oracle.com> wrote:
>>>> Updated webrevs (including the suggestions from Kim and Jon) are in:
>>>>
>>>>     http://cr.openjdk.java.net/~tbenson/8060697/webrev.01.vs.00/    - incremental
>>>> http://cr.openjdk.java.net/~tbenson/8060697/webrev.01/ - full
>>> I'd previously overlooked the removal of the const qualifier from
>>> G1CollectorPolicy::expansion_amount. This was done to permit some
>>> member variables to be updated to accumulate additional information
>>> (specifically, the fields that are reset by the new
>>> clear_ratio_check_data). But the function name "expansion_amount"
>>> looks to me like an accessor or compute function, and not something
>>> that modifies the policy object.
>>>
>>> I don't have a good alternative (either structure or naming) to offer
>>> right now, but maybe you or someone else might think of something.  Or
>>> maybe I'm the only one (somewhat) bothered by this.
>>>
>    I already noticed that, but I have not good advice for that either.
>
> Looked at latest changes again, looks good.
>
> No preference about the (double) casts in the other email.
>
> Thanks,
>    Thomas
>