On Thu, Mar 30, 2017 at 2:41 AM, Matias Elo <[email protected]> wrote:
> Zero is a valid return value from the packet headroom/tailroom functions.
>
> Signed-off-by: Matias Elo <[email protected]>

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

> ---
>  test/common_plat/performance/odp_bench_packet.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/test/common_plat/performance/odp_bench_packet.c 
> b/test/common_plat/performance/odp_bench_packet.c
> index 7a3a004..c4cd613 100644
> --- a/test/common_plat/performance/odp_bench_packet.c
> +++ b/test/common_plat/performance/odp_bench_packet.c
> @@ -618,7 +618,7 @@ static int bench_packet_headroom(void)
>         for (i = 0; i < TEST_REPEAT_COUNT; i++)
>                 ret += odp_packet_headroom(gbl_args->pkt_tbl[i]);
>
> -       return ret;
> +       return i;
>  }
>
>  static int bench_packet_tailroom(void)
> @@ -629,7 +629,7 @@ static int bench_packet_tailroom(void)
>         for (i = 0; i < TEST_REPEAT_COUNT; i++)
>                 ret += odp_packet_tailroom(gbl_args->pkt_tbl[i]);
>
> -       return ret;
> +       return i;
>  }
>
>  static int bench_packet_tail(void)
> --
> 2.7.4
>

Reply via email to