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