RFR: JDK-8153186: Convert TraceGCTaskThread to use unified logging

Thomas Schatzl thomas.schatzl at oracle.com
Thu Mar 31 13:14:21 UTC 2016


Hi Bengt,

On Thu, 2016-03-31 at 14:59 +0200, Bengt Rutisson wrote:
> Hi Thomas,
> 
> Thanks for looking at this!
> 
> On 2016-03-31 14:50, Thomas Schatzl wrote:
> > Hi,
> > 
> > On Thu, 2016-03-31 at 14:23 +0200, Bengt Rutisson wrote:
> > > Hi everyone,
> > > 
> > > Could I have a couple of reviews for this change?
> > > 
> > > http://cr.openjdk.java.net/~brutisso/8153186/webrev.00/
> > > https://bugs.openjdk.java.net/browse/JDK-8153186
> > > 
> > > The logging guarded by TraceGCTaskThread should be converted to
> > > use
> > > the
> > > unified logging framework.
> >    - these log messages seem to be gc specific, so maybe add the gc
> > -id
> > decoration.
> 
> This logging is all done at initilization, before we start a GC so I 
> don't think there is a point in adding the GC id decoration.
> 
> > 
> >    - this change simply removes the flag from the available
> > options.
> > Shouldn't it be added to the table we have that translates old
> > flags to
> > new logging somewhere? (I do not have an opinion about this, just
> > asking).
> 
> There are some internal table for such conversion. I can look in to 
> adding it there. For the public documentation we have selected a
> subset 
> of all flags to document. I don't think this flag is used widely
> enough 
> to qualify for that list.

  okay. Ship it.

Thomas




More information about the hotspot-gc-dev mailing list