RFR: Refactor worker timings into ShenandoahPhaseTimings

Christine Flood cflood at redhat.com
Tue Sep 26 11:52:10 UTC 2017


I would classify this as a one-peer-reviewer level patch for future
reference, but it's fine by me anyway.

On Tue, Sep 26, 2017 at 5:10 AM, Roman Kennke <rkennke at redhat.com> wrote:

> This looks useful and good to me.
>
> Roman
>
>
> Am 26. September 2017 10:50:13 MESZ schrieb Aleksey Shipilev <
> shade at redhat.com>:
> >http://cr.openjdk.java.net/~shade/shenandoah/stats-worker-
> times/webrev.01/
> >
> >Noticed we now have shenandoahPhaseTimings.(hpp|cpp) and
> >shenandoahPhaseTimes.{cpp|hpp}. The latter
> >does the per-worker accounting, and merges it back to total counters.
> >Moved that into our recently
> >added file, and did a few renames to capture those bits are indeed
> >about workers.
> >
> >Does not change the output:
> >http://cr.openjdk.java.net/~shade/shenandoah/stats-worker-
> times/before.txt
> >http://cr.openjdk.java.net/~shade/shenandoah/stats-worker-times/after.txt
> >
> >Testing: hotspot_gc_shenandoah, eyeballing gc+stats
> >
> >Thanks,
> >-Aleksey
>
> --
> Diese Nachricht wurde von meinem Android-Gerät mit K-9 Mail gesendet.
>


More information about the shenandoah-dev mailing list