For this series:

Reviewed-and-tested-by: Bill Fischofer <[email protected]>

Note that part 2 mentions Coverity by name in the short log. A simple
reword during merge should be doable.

On Fri, May 20, 2016 at 10:17 AM, Barry Spinney <[email protected]>
wrote:

> Add the PRIu64 and PRIX64 macros to some of the ODP_DBG calls.
>
> Signed-off-by: Barry Spinney <[email protected]>
> ---
>  platform/linux-generic/odp_pkt_queue.c   | 8 ++++----
>  platform/linux-generic/odp_sorted_list.c | 7 ++++---
>  2 files changed, 8 insertions(+), 7 deletions(-)
>
> diff --git a/platform/linux-generic/odp_pkt_queue.c
> b/platform/linux-generic/odp_pkt_queue.c
> index 64f4930..949cf74 100644
> --- a/platform/linux-generic/odp_pkt_queue.c
> +++ b/platform/linux-generic/odp_pkt_queue.c
> @@ -348,15 +348,15 @@ void
> _odp_pkt_queue_stats_print(_odp_int_queue_pool_t queue_pool)
>         queue_pool_t *pool;
>
>         pool = (queue_pool_t *)(uintptr_t)queue_pool;
> -       ODP_DBG("pkt_queue_stats - queue_pool=0x%lX\n", queue_pool);
> +       ODP_DBG("pkt_queue_stats - queue_pool=0x%" PRIX64 "\n",
> queue_pool);
>         ODP_DBG("  max_queue_num=%u max_queued_pkts=%u
> next_queue_num=%u\n",
>                 pool->max_queue_num, pool->max_queued_pkts,
>                 pool->next_queue_num);
> -       ODP_DBG("  total pkt appends=%lu total pkt removes=%lu "
> -               "bad removes=%lu\n",
> +       ODP_DBG("  total pkt appends=%" PRIu64 " total pkt removes=%"
> PRIu64
> +               " bad removes=%" PRIu64 "\n",
>                 pool->total_pkt_appends, pool->total_pkt_removes,
>                 pool->total_bad_removes);
> -       ODP_DBG("  free_list size=%u  min size=%u peak size=%u\n",
> +       ODP_DBG("  free_list size=%u min size=%u peak size=%u\n",
>                 pool->free_list_size, pool->min_free_list_size,
>                 pool->peak_free_list_size);
>  }
> diff --git a/platform/linux-generic/odp_sorted_list.c
> b/platform/linux-generic/odp_sorted_list.c
> index cbe0223..554494b 100644
> --- a/platform/linux-generic/odp_sorted_list.c
> +++ b/platform/linux-generic/odp_sorted_list.c
> @@ -258,11 +258,12 @@ void
> _odp_sorted_list_stats_print(_odp_int_sorted_pool_t sorted_pool)
>         sorted_pool_t *pool;
>
>         pool = (sorted_pool_t *)(uintptr_t)sorted_pool;
> -       ODP_DBG("sorted_pool=0x%lX\n", sorted_pool);
> +       ODP_DBG("sorted_pool=0x%" PRIX64 "\n", sorted_pool);
>         ODP_DBG("  max_sorted_lists=%u next_list_idx=%u\n",
>                 pool->max_sorted_lists, pool->next_list_idx);
> -       ODP_DBG("  total_inserts=%lu total_deletes=%lu
> total_removes=%lu\n",
> -               pool->total_inserts, pool->total_deletes,
> pool->total_removes);
> +       ODP_DBG("  total_inserts=%" PRIu64 " total_deletes=%" PRIu64
> +               " total_removes=%" PRIu64 "\n", pool->total_inserts,
> +               pool->total_deletes, pool->total_removes);
>  }
>
>  void _odp_sorted_pool_destroy(_odp_int_sorted_pool_t sorted_pool)
> --
> 2.7.2
>
> _______________________________________________
> lng-odp mailing list
> [email protected]
> https://lists.linaro.org/mailman/listinfo/lng-odp
>
_______________________________________________
lng-odp mailing list
[email protected]
https://lists.linaro.org/mailman/listinfo/lng-odp

Reply via email to