RFR (XXS): 8152400: Enabling TASK_STATS_ONLY filters out just enabled messages anyway

Tom Benson tom.benson at oracle.com
Tue Mar 22 20:18:39 UTC 2016


Looks fine to me.
Tom

On 3/22/2016 4:05 PM, Thomas Schatzl wrote:
> Hi all,
>
>    can I have reviews for the following one-liner?
>
> If you enable TASK_QUEUE_STATS for a product build for analysis, you
> can't immediately use them. You also need to change another line of
> code from log_develop_is_enabled() to log_is_enabled().
>
> This seems strange: if I want the task queue stats, I surely intend to
> use it, and since that latter condition only filters out the printing
> itself, the overhead has already been incurred anyway.
>
> This change changes the log_develop_is_enabled() check to
> log_is_enabled() to avoid an additional round of changes after noticing
> that task queue stats are not printed.
>
> There is no difference in performance for product builds, as the whole
> code is compiled out, in non-product builds there is an extra trivial
> condition to be checked per gc. I do not think this is noticeable at
> all.
>
> CR:
> https://bugs.openjdk.java.net/browse/JDK-8152400
> Webrev:
> http://cr.openjdk.java.net/~tschatzl/8152400/webrev/
> Testing:
> local testing, local building
>
> Thanks,
>    Thomas




More information about the hotspot-gc-dev mailing list