RFR (S): 8204082: Make names of Young GCs more uniform in logs
Thomas Schatzl
thomas.schatzl at oracle.com
Wed Jun 20 11:27:19 UTC 2018
Hi,
some changes need to be done for the gtests too, so here are new
webrevs:
http://cr.openjdk.java.net/~tschatzl/8204082/webrev.3_to_4/
http://cr.openjdk.java.net/~tschatzl/8204082/webrev.4/
Thanks,
Thomas
On Fri, 2018-06-15 at 15:47 +0200, Thomas Schatzl wrote:
> Hi,
>
> during some more discussion about the messages there was the
> concern
> that the "Concurrent End" message is not really an indication of the
> concurrent marking end (this happens asynchronuosly), also that pause
> may not occur (if no mixed gc starts after marking), so we came up
> with
> "Prepare Mixed" for it.
> "Concurrent End" would probably fit better for the current "Cleanup"
> pause.
>
> I.e.
>
> Pause Young (Normal) ...
> Pause Young (Concurrent Start) ...
> Pause Young (Prepare Mixed) ...
> Pause Young (Mixed) ...
>
>
> http://cr.openjdk.java.net/~tschatzl/8204082/webrev.2_to_3 (diff)
> http://cr.openjdk.java.net/~tschatzl/8204082/webrev.3 (full)
>
> Thanks,
> Thomas
>
> On Thu, 2018-06-14 at 15:02 +0200, Thomas Schatzl wrote:
> > Hi all,
> >
> > another round of reviews after some more internal remarks :P
> >
> > I also changed the title of the CR.
> >
> > The set of "final" tags would be:
> >
> > Pause Young (Normal) ...
> > Pause Young (Concurrent Start) ...
> > Pause Young (Concurrent End) ...
> > Pause Young (Mixed) ...
> >
> > I also adapted the strings in the GCVerifyType functionality.
> >
> > http://cr.openjdk.java.net/~tschatzl/8204082/webrev.1_to_2 (diff)
> > http://cr.openjdk.java.net/~tschatzl/8204082/webrev.2 (full)
> >
> > Testing:
> > running through all gc tests locally
> >
> > Thanks,
> > Thomas
> >
>
>
More information about the hotspot-gc-dev
mailing list