On Mon, Jun 16, 2014 at 9:46 AM, Daniele Di Proietto
<ddiproie...@vmware.com> wrote:
> rte_eal_init() returns the number of parsed dpdk arguments to skip.
> dpdk_init() should add 1 to that number, because it has already skipped the
> "--dpdk" argument itself
>
> Signed-off-by: Daniele Di Proietto <ddiproie...@vmware.com>
Looks good.

Acked-by: Pravin B Shelar <pshe...@nicira.com>

> ---
>  lib/netdev-dpdk.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/lib/netdev-dpdk.c b/lib/netdev-dpdk.c
> index fbdb6b3..1ae5217 100644
> --- a/lib/netdev-dpdk.c
> +++ b/lib/netdev-dpdk.c
> @@ -1193,7 +1193,8 @@ dpdk_init(int argc, char **argv)
>      rte_memzone_dump();
>      rte_eal_init_ret = 0;
>
> -    return result;
> +    /* We need to skip 'result' arguments plus the "--dpdk" argument itself 
> */
> +    return result + 1;
>  }
>
>  void
> --
> 2.0.0
>
> _______________________________________________
> dev mailing list
> dev@openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to