RFR (XS): 8232779: G1 current collection parallel time does not include optional evacuation

[Thomas Schatzl]

Hi Sangheon, Kim

On 22.10.19 22:50, sangheon.kim at oracle.com wrote:
> Hi Thomas,
> 
> On 10/22/19 11:05 AM, Thomas Schatzl wrote:
>> Hi all,
>>
>>   can I have reviews for this change that fixes the calculation of 
>> G1GCPhaseTimes::cur_collection_par_time_ms(): we forgot to consider 
>> the optional evacuation time.
>>
>> This causes too long Other time, having minor effects on pause time 
>> prediction.
>>
>> CR:
>> https://bugs.openjdk.java.net/browse/JDK-8232779
>> Webrev:
>> http://cr.openjdk.java.net/~tschatzl/8232779/webrev/
> Looks good.
> 

   thanks for your review.

Thomas