RFR: 8167679: G1 phase logging is messy

[Erik Helin]

On 2016-10-25, Thomas Schatzl wrote:
> Hi Erik,
> 
> On Tue, 2016-10-25 at 15:17 +0200, Erik Helin wrote:
> > On 2016-10-25, Stefan Johansson wrote:
> > > 
> > > Hi Erik,
> > > 
> > > On 2016-10-14 16:00, Erik Helin wrote:
> > > > 
> > > > Hi all,
> > > > 
> > > > 
> [...]
> > > * The timing _recorded_clear_claimed_marks_time_ms is not added to
> > > sum_ms in
> > > print_post_evacuate_collection_set.
> > Agree, fixed.
> > 
> > Please see new patches at:
> > - incremental: http://cr.openjdk.java.net/~ehelin/8167679/00-01/
> > - full: http://cr.openjdk.java.net/~ehelin/8167679/01/
> > 
> 
>   looks good.
> 
> One suggestion: please move the sub-sections of the Free Collection Set
> phase into trace level. They look odd at the same level as the main
> category.

I agree, please see new patches at:
- incremental: http://cr.openjdk.java.net/~ehelin/8167679/01-02/
- full: http://cr.openjdk.java.net/~ehelin/8167679/02/

Thanks,
Erik

> Thanks,
>   Thomas
>