[8] 2020-03-02, Bulk backports to sh/jdk8

Roman Kennke rkennke at redhat.com
Mon Mar 2 11:45:41 UTC 2020


Hi Aleksey,

In:

+void ShenandoahHeuristics::adjust_penalty(intx step) {
+  assert(0 <= _gc_time_penalties && _gc_time_penalties <= 100,
+          err_msg("In range before adjustment: " INTX_FORMAT,
_gc_time_penalties));


There is one too many spaces before err_msg, I think. Happens in the
assert below too.

Other than that, it's looking good! Thank you!

Roman


> This backports a round of relatively safe changes for 8u:
>   [backport] 8240070: Shenandoah: remove obsolete ShenandoahCommonGCStateLoads
>   [backport] 8239868: Shenandoah: ditch C2 node limit adjustments
>   [backport] 8239904: Shenandoah: accumulated penalties should not be over 100% of capacity
>   [backport] 8240069: Shenandoah: turn more flags diagnostic
>   [backport] 8240076: Shenandoah: pacer should cover reset and preclean phases
>   [backport] 8240215: Shenandoah: remove ShenandoahAllocationTrace
> 
> 8u webrev:
>   https://cr.openjdk.java.net/~shade/shenandoah/backports/jdk8u-20200302/
> 
> Testing: hotspot_gc_shenandoah {fastdebug,release}
> 



More information about the shenandoah-dev mailing list