dev.dpdk.org archive mirror
 help / color / mirror / Atom feed
* Vmxnet3 pmd
@ 2014-08-13 10:13 Alex Markuze
       [not found] ` <CAKfHP0V4xx3mwg0dYMDFcQOVdN1BfBzuHfnT59KKFVy9LAz70Q-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
  0 siblings, 1 reply; 4+ messages in thread
From: Alex Markuze @ 2014-08-13 10:13 UTC (permalink / raw)
  To: dev-VfR2kkLFssw@public.gmane.org

Hi, I Have a simple dpdk app - basically a KNI interface with the dpdk
layer serving only as a pipeline.

This allows me to ping between vEth0 on different VM's works great with ixgbevf.
Now I moved to ESXi5.5 , Ubuntu14.4 VM (Dpdk 1.7).
When running the same code*  I've discovered that the polling doesn't
retrieve any packets after vEth0 gents an IP. I've resolved this issue
by removing the dev restart calls I had in the callback.

        //      rte_eth_dev_stop(port_id);
        //      ret = rte_eth_dev_start(port_id);

Is this a know issue? how can I report a BUG if its not.

Thanks
Alex.

*Except this line I needed to add to setup the TXQ.
tx_conf.txq_flags |= (ETH_TXQ_FLAGS_NOMULTSEGS | ETH_TXQ_FLAGS_NOOFFLOADS);

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Vmxnet3 pmd
       [not found] ` <CAKfHP0V4xx3mwg0dYMDFcQOVdN1BfBzuHfnT59KKFVy9LAz70Q-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
@ 2014-08-13 16:58   ` Patel, Rashmin N
  2014-08-26  0:41   ` [PATCH] vmxnet3: fix crash on stop Stephen Hemminger
  1 sibling, 0 replies; 4+ messages in thread
From: Patel, Rashmin N @ 2014-08-13 16:58 UTC (permalink / raw)
  To: Alex Markuze, dev-VfR2kkLFssw@public.gmane.org

Hi Alex,

This is known issue with dev_stop of vmxnet3_pmd. Vmxnet3_cmd_ring_release() routine calls rte_free(ring->buf_info), which should be reused instead. Because it's not being reallocated at dev_start, otherwise reallocation is needed. I've a patch that works for me, I'll submit it after some testing.

Thanks,
Rashmin

-----Original Message-----
From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Alex Markuze
Sent: Wednesday, August 13, 2014 3:14 AM
To: dev@dpdk.org
Subject: [dpdk-dev] Vmxnet3 pmd

Hi, I Have a simple dpdk app - basically a KNI interface with the dpdk layer serving only as a pipeline.

This allows me to ping between vEth0 on different VM's works great with ixgbevf.
Now I moved to ESXi5.5 , Ubuntu14.4 VM (Dpdk 1.7).
When running the same code*  I've discovered that the polling doesn't retrieve any packets after vEth0 gents an IP. I've resolved this issue by removing the dev restart calls I had in the callback.

        //      rte_eth_dev_stop(port_id);
        //      ret = rte_eth_dev_start(port_id);

Is this a know issue? how can I report a BUG if its not.

Thanks
Alex.

*Except this line I needed to add to setup the TXQ.
tx_conf.txq_flags |= (ETH_TXQ_FLAGS_NOMULTSEGS | ETH_TXQ_FLAGS_NOOFFLOADS);

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [PATCH] vmxnet3: fix crash on stop
       [not found] ` <CAKfHP0V4xx3mwg0dYMDFcQOVdN1BfBzuHfnT59KKFVy9LAz70Q-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
  2014-08-13 16:58   ` Patel, Rashmin N
@ 2014-08-26  0:41   ` Stephen Hemminger
  2014-08-29 10:41     ` Thomas Monjalon
  1 sibling, 1 reply; 4+ messages in thread
From: Stephen Hemminger @ 2014-08-26  0:41 UTC (permalink / raw)
  To: Alex Markuze; +Cc: dev-VfR2kkLFssw@public.gmane.org

The cmd_ring_release can be called twice if queue has already
been released. This cause crash on shutdown.

Signed-off-by: Stephen Hemminger <stephen-OTpzqLSitTUnbdJkjeBofR2eb7JE58TQ@public.gmane.org>


--- a/lib/librte_pmd_vmxnet3/vmxnet3_rxtx.c	2014-08-25 17:39:04.544119911 -0700
+++ b/lib/librte_pmd_vmxnet3/vmxnet3_rxtx.c	2014-08-25 17:39:04.544119911 -0700
@@ -173,6 +173,7 @@ vmxnet3_cmd_ring_release(vmxnet3_cmd_rin
 		vmxnet3_cmd_ring_adv_next2comp(ring);
 	}
 	rte_free(ring->buf_info);
+	ring->buf_info = NULL;
 }
 
 void

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] vmxnet3: fix crash on stop
  2014-08-26  0:41   ` [PATCH] vmxnet3: fix crash on stop Stephen Hemminger
@ 2014-08-29 10:41     ` Thomas Monjalon
  0 siblings, 0 replies; 4+ messages in thread
From: Thomas Monjalon @ 2014-08-29 10:41 UTC (permalink / raw)
  To: Stephen Hemminger; +Cc: dev-VfR2kkLFssw

> The cmd_ring_release can be called twice if queue has already
> been released. This cause crash on shutdown.
> 
> Signed-off-by: Stephen Hemminger <stephen-OTpzqLSitTUnbdJkjeBofR2eb7JE58TQ@public.gmane.org>
> 
> @@ -173,6 +173,7 @@ vmxnet3_cmd_ring_release(vmxnet3_cmd_rin
>  		vmxnet3_cmd_ring_adv_next2comp(ring);
>  	}
>  	rte_free(ring->buf_info);
> +	ring->buf_info = NULL;
>  }

Acked-by: Thomas Monjalon <thomas.monjalon-pdR9zngts4EAvxtiuMwx3w@public.gmane.org>

Applied for version 1.7.1.

Thanks
-- 
Thomas

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2014-08-29 10:41 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-08-13 10:13 Vmxnet3 pmd Alex Markuze
     [not found] ` <CAKfHP0V4xx3mwg0dYMDFcQOVdN1BfBzuHfnT59KKFVy9LAz70Q-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2014-08-13 16:58   ` Patel, Rashmin N
2014-08-26  0:41   ` [PATCH] vmxnet3: fix crash on stop Stephen Hemminger
2014-08-29 10:41     ` Thomas Monjalon

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).