RFR: 8146990: Convert CollectorPolicy to use log_warning instead of warning

[Erik Helin]

On 2016-01-13, Stefan Johansson wrote:
> Hi Erik,
> 
> Thanks for doing this improvement.
> 
> On 2016-01-13 15:24, Erik Helin wrote:
> >Hi all,
> >
> >this patchs updates CollectorPolicy to use log_warning(gc, ergo) instead of
> >warning (as part of updating the GC code to use unified logging).
> >
> >Enhancement:
> >https://bugs.openjdk.java.net/browse/JDK-8146990
> >
> >Webrev:
> >http://cr.openjdk.java.net/~ehelin/8146990/00/webrev/
> src/share/vm/gc/shared/collectorPolicy.cpp:
> 700           log_warning(gc, ergo)("GenCollectorPolicy::mem_allocate_work
> retries %d times \n\t"
> 701                                 " size=" SIZE_FORMAT " %s", try_count,
> size, is_tlab ? "(TLAB)" : "");
> 
> 873       log_warning(gc, ergo)("satisfy_failed_metadata_allocation()
> retries %d times \n\t"
> 874                             " size=" SIZE_FORMAT, loop_count,
> word_size);
> 
> After UL we avoid having multi-line logging statements. So I suggest you
> either re-format these to only use a single line each.

Nice catch, I missed the \t and the \n characters. New patches are
available at:
- full: http://cr.openjdk.java.net/~ehelin/8146990/01/
- inc: http://cr.openjdk.java.net/~ehelin/8146990/01/inc/

Thanks,
Erik

> ---
> 
> Otherwise the change looks good.
> 
> Thanks,
> Stefan
> 
> >Testing:
> >- JPRT
> >
> >Thanks,
> >Erik
>