On Wed, Jan 27, 2016 at 11:44 AM, Zoltan Kiss <[email protected]>
wrote:

> 'j' is incremented twice, therefore every second element of hdr_tbl is
> garbage. Introduced in f3c1e77f "linux-generic: pktio: optimize
> pktin_deq_multi".
>
> Signed-off-by: Zoltan Kiss <[email protected]>
>

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


> ---
> diff --git a/platform/linux-generic/odp_packet_io.c
> b/platform/linux-generic/odp_packet_io.c
> index cda1900..540cdb8 100644
> --- a/platform/linux-generic/odp_packet_io.c
> +++ b/platform/linux-generic/odp_packet_io.c
> @@ -640,7 +640,7 @@ int pktin_deq_multi(queue_entry_t *qentry,
> odp_buffer_hdr_t *buf_hdr[], int num)
>         /* Queue the rest for later */
>         for (j = 0; i < pkts; i++, j++) {
>                 buf        = _odp_packet_to_buffer(pkt_tbl[i]);
> -               hdr_tbl[j++] = odp_buf_to_hdr(buf);
> +               hdr_tbl[j] = odp_buf_to_hdr(buf);
>         }
>
>         if (j)
> _______________________________________________
> 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