[dpdk-dev,v2] virtio: Update max RX packet length

Message ID CEFFBC011B3590479939EA0B9311D21526916D4B@ILMB1.corp.radware.com (mailing list archive)
State Not Applicable, archived
Headers

Commit Message

Yan Freedland Oct. 28, 2014, 2:38 p.m. UTC
  Currently I have an issue in a pass-thorugh mode. I am running in multi process (2 processes) via ixgbe_rxtx. When running with a single process I have a perfect traffic while with 2 processes I have no traffic at all. 
Perhaps there is a special configuration for that, can you please assist ?

Thanks
Yan

-----Original Message-----
From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Ouyang Changchun
Sent: Monday, October 13, 2014 6:46 AM
To: dev@dpdk.org
Subject: [dpdk-dev] [PATCH v2] virtio: Update max RX packet length

Update max RX packet length since virtio PMD has the capability of receiving and transmitting jumbo frame.

This following patch provides the above capability:
[dpdk-dev,v3] virtio: Support mergeable buffer in virtio pmd
  Submitter Ouyang Changchun
  Date Aug. 14, 2014, 8:54 a.m. 
  Message ID <1408006475-17606-1-git-send-email-changchun.ouyang@intel.com>
  Permalink http://dpdk.org/dev/patchwork/patch/159/

Signed-off-by: Changchun Ouyang <changchun.ouyang@intel.com>
Tested-by: Jingguo Fu <jingguox.fu@intel.com>

---
 lib/librte_pmd_virtio/virtio_ethdev.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
  

Comments

Thomas Monjalon Oct. 28, 2014, 2:48 p.m. UTC | #1
2014-10-28 14:38, Yan Freedland:
> Currently I have an issue in a pass-thorugh mode.
> I am running in multi process (2 processes) via ixgbe_rxtx.
> When running with a single process I have a perfect traffic
> while with 2 processes I have no traffic at all. 
> Perhaps there is a special configuration for that, can you please assist ?

Excuse me, I don't understand the relation between your question and the
patch below. If there is no relation, please open a new thread by clicking
on "new mail" instead of "reply all".

Maybe that some kind developers will help you but you'd have more chances
by providing your command line and more debug details showing that you tried
to debug it by yourself and you are not a lazy developer.

Reminder: some interesting links about mailing practices are in this page:
	http://dpdk.org/ml

> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Ouyang Changchun
> Sent: Monday, October 13, 2014 6:46 AM
> To: dev@dpdk.org
> Subject: [dpdk-dev] [PATCH v2] virtio: Update max RX packet length
> 
> Update max RX packet length since virtio PMD has the capability of receiving and transmitting jumbo frame.
> 
> This following patch provides the above capability:
> [dpdk-dev,v3] virtio: Support mergeable buffer in virtio pmd
>   Submitter Ouyang Changchun
>   Date Aug. 14, 2014, 8:54 a.m. 
>   Message ID <1408006475-17606-1-git-send-email-changchun.ouyang@intel.com>
>   Permalink http://dpdk.org/dev/patchwork/patch/159/
> 
> Signed-off-by: Changchun Ouyang <changchun.ouyang@intel.com>
> Tested-by: Jingguo Fu <jingguox.fu@intel.com>
> 
> ---
>  lib/librte_pmd_virtio/virtio_ethdev.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/lib/librte_pmd_virtio/virtio_ethdev.h b/lib/librte_pmd_virtio/virtio_ethdev.h
> index d2e1eed..1da3c62 100644
> --- a/lib/librte_pmd_virtio/virtio_ethdev.h
> +++ b/lib/librte_pmd_virtio/virtio_ethdev.h
> @@ -53,7 +53,7 @@
>  #define VIRTIO_MAX_TX_QUEUES 128
>  #define VIRTIO_MAX_MAC_ADDRS 1
>  #define VIRTIO_MIN_RX_BUFSIZE 64
> -#define VIRTIO_MAX_RX_PKTLEN  1518
> +#define VIRTIO_MAX_RX_PKTLEN  9728
>  
>  /* Features desired/implemented by this driver. */  #define VTNET_FEATURES \
> --
> 1.8.4.2
  
Yan Freedland Oct. 28, 2014, 4:26 p.m. UTC | #2
Hi, 

I mistakenly pressed on reply all instead of reply and sent my question to the whole dev. Sorry about that.
In fact I am not lazy and looks like I solved the problem by myself. I will gladly share my knowledge with you or anyone in dev and of course open a new mail thread with an appropriate subject for further appeals.

Thank you,
Yan

-----Original Message-----
From: Thomas Monjalon [mailto:thomas.monjalon@6wind.com] 
Sent: Tuesday, October 28, 2014 4:49 PM
To: Yan Freedland
Cc: dev@dpdk.org
Subject: Re: [dpdk-dev] [PATCH v2] virtio: Update max RX packet length

2014-10-28 14:38, Yan Freedland:
> Currently I have an issue in a pass-thorugh mode.
> I am running in multi process (2 processes) via ixgbe_rxtx.
> When running with a single process I have a perfect traffic while with 
> 2 processes I have no traffic at all.
> Perhaps there is a special configuration for that, can you please assist ?

Excuse me, I don't understand the relation between your question and the patch below. If there is no relation, please open a new thread by clicking on "new mail" instead of "reply all".

Maybe that some kind developers will help you but you'd have more chances by providing your command line and more debug details showing that you tried to debug it by yourself and you are not a lazy developer.

Reminder: some interesting links about mailing practices are in this page:
	http://dpdk.org/ml

> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Ouyang Changchun
> Sent: Monday, October 13, 2014 6:46 AM
> To: dev@dpdk.org
> Subject: [dpdk-dev] [PATCH v2] virtio: Update max RX packet length
> 
> Update max RX packet length since virtio PMD has the capability of receiving and transmitting jumbo frame.
> 
> This following patch provides the above capability:
> [dpdk-dev,v3] virtio: Support mergeable buffer in virtio pmd
>   Submitter Ouyang Changchun
>   Date Aug. 14, 2014, 8:54 a.m. 
>   Message ID <1408006475-17606-1-git-send-email-changchun.ouyang@intel.com>
>   Permalink http://dpdk.org/dev/patchwork/patch/159/
> 
> Signed-off-by: Changchun Ouyang <changchun.ouyang@intel.com>
> Tested-by: Jingguo Fu <jingguox.fu@intel.com>
> 
> ---
>  lib/librte_pmd_virtio/virtio_ethdev.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/lib/librte_pmd_virtio/virtio_ethdev.h 
> b/lib/librte_pmd_virtio/virtio_ethdev.h
> index d2e1eed..1da3c62 100644
> --- a/lib/librte_pmd_virtio/virtio_ethdev.h
> +++ b/lib/librte_pmd_virtio/virtio_ethdev.h
> @@ -53,7 +53,7 @@
>  #define VIRTIO_MAX_TX_QUEUES 128
>  #define VIRTIO_MAX_MAC_ADDRS 1
>  #define VIRTIO_MIN_RX_BUFSIZE 64
> -#define VIRTIO_MAX_RX_PKTLEN  1518
> +#define VIRTIO_MAX_RX_PKTLEN  9728
>  
>  /* Features desired/implemented by this driver. */  #define 
> VTNET_FEATURES \
> --
> 1.8.4.2
  

Patch

diff --git a/lib/librte_pmd_virtio/virtio_ethdev.h b/lib/librte_pmd_virtio/virtio_ethdev.h
index d2e1eed..1da3c62 100644
--- a/lib/librte_pmd_virtio/virtio_ethdev.h
+++ b/lib/librte_pmd_virtio/virtio_ethdev.h
@@ -53,7 +53,7 @@ 
 #define VIRTIO_MAX_TX_QUEUES 128
 #define VIRTIO_MAX_MAC_ADDRS 1
 #define VIRTIO_MIN_RX_BUFSIZE 64
-#define VIRTIO_MAX_RX_PKTLEN  1518
+#define VIRTIO_MAX_RX_PKTLEN  9728
 
 /* Features desired/implemented by this driver. */  #define VTNET_FEATURES \
--
1.8.4.2