RFR(XXS): 8223397: gc+promotion log lines are missing the GC id

[Tony Printezis]

Thomas and Per,

Thanks! I’ll submit it to the submit repo, just in case (and to refresh my
memory on how to do that), and I’ll push it after I get the results...

Tony


—————
Tony Printezis | @TonyPrintezis | tprintezis at twitter.com


On May 7, 2019 at 2:20:25 AM, Per Liden (per.liden at oracle.com) wrote:

Hi Tony,

On 5/6/19 7:27 PM, Tony Printezis wrote:
> Trivial one-line change:
>
> http://cr.openjdk.java.net/~tonyp/8223397/webrev.0/
>
> Was there a reason why gc+promotion was not added to LOG_PREFIX_LIST in
> the first place?

I'm pretty sure that was just a mistake.

Patch looks good.

cheers,
Per

>
>
> Before:
>
> [59.899s][info ][gc,task          ] GC(17) Using 18 workers of 18 for
> evacuation
>
> [61.934s][info ][gc,promotion     ] Promotion failed
>
> [61.934s][info ][gc               ] GC(17) Pause Young (Allocation
> Failure) 3948M->4555M(4557M) 2035.171ms
>
>
>
> After:
>
> [55.339s][info ][gc,task          ] GC(17) Using 18 workers of 18 for
> evacuation
>
> [57.383s][info ][gc,promotion     ] GC(17) Promotion failed
>
> [57.383s][info ][gc               ] GC(17) Pause Young (Allocation
> Failure) 3944M->4556M(4557M) 2044.354ms
>
>
>
> Thanks,
>
> Tony
>
> —————
> Tony Printezis | @TonyPrintezis | tprintezis at twitter.com
> <mailto:tprintezis at twitter.com>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.org/pipermail/hotspot-gc-dev/attachments/20190507/8e7bb7a0/attachment.htm>