Integrated: 8295476: Split G1 cost per byte predictor on gc phase

[Thomas Schatzl]

On Tue, 25 Oct 2022 20:00:32 GMT, Thomas Schatzl <tschatzl at openjdk.org> wrote:

> Hi,
> 
>   please review this change to the cost-per-byte predictor to be split on gc phase (young-only/mixed) instead of during marking/outside of marking.
> 
> The reason is that the distinction between young-only/mixed of these values is higher than marking/outside of marking, although the latter degrades somewhat to the former in case of continuous concurrent marking where both are almost equivalent anyway. I believe that has been the original reason to do the marking/not-marking distinction.
> 
> I also removed the implied 10% extra cost for copies during marking (i.e. during the young-only phase). I could not find any particular reason why copying would be more expensive then with the current code base.
> 
> The distinction in general makes sense as the data copied during young-only and mixed is different (only short-lived data vs. short+long-lived data).
> 
> Testing: gha, examining predictions with some benchmarks
> 
> Thanks,
>   Thomas

This pull request has now been integrated.

Changeset: 8de3edac
Author:    Thomas Schatzl <tschatzl at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/8de3edac6db5bdb0a60bb68b15b81ce9cdf2be02
Stats:     37 lines in 3 files changed: 4 ins; 24 del; 9 mod

8295476: Split G1 cost per byte predictor on gc phase

Reviewed-by: ayang, iwalulya, kbarrett

-------------

PR: https://git.openjdk.org/jdk/pull/10862