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

[sangheon.kim at oracle.com]

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,
Sangheon


> Testing:
> local compilation
>
> Thanks,
>   Thomas