[dpdk-dev] [PATCH v2 10/13] mbuf: generic support for TCP segmentation offload

2014-11-17 Thread Ananyev, Konstantin
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Olivier Matz > Sent: Friday, November 14, 2014 5:03 PM > To: dev at dpdk.org > Cc: jigsaw at gmail.com > Subject: [dpdk-dev] [PATCH v2 10/13] mbuf: generic support for TCP > segmentation offload > > Some of t

[dpdk-dev] [PATCH v2 03/13] mbuf: move vxlan_cksum flag definition at the proper place

2014-11-17 Thread Thomas Monjalon
2014-11-14 18:03, Olivier Matz: > The tx mbuf flags are ordered from the highest value to the > the lowest. Move the PKT_TX_VXLAN_CKSUM at the right place. Please, could you reorder them from the lowest to the highest? It will be simpler to understand. There is already a comment to explain the rev

[dpdk-dev] Performance impact with QoS

2014-11-17 Thread Dumitrescu, Cristian
Hi Satish, The QoS traffic manager has a large memory footprint due to large number of packet queues (e.g. 64K queues of 64 packets each) and large tables (e.g. 4K pipes with one cache line of context per pipe) that far exceeds the amount of CPU cache physically available. There are a lot of da

[dpdk-dev] [PATCH] lib: include rte_memory.h for __rte_cache_aligned

2014-11-17 Thread Jia Yu
Thanks for reviewing this change. No error message was found without this patch. This patch follows a good coding practice to include the header files when it?s needed, rather than through some other header files. In the past, we found some data structure was not cache line aligned due to missing

[dpdk-dev] [PATCH v2 06/13] mbuf: add functions to get the name of an ol_flag

2014-11-17 Thread Ananyev, Konstantin
Hi Oliver, > -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Olivier Matz > Sent: Friday, November 14, 2014 5:03 PM > To: dev at dpdk.org > Cc: jigsaw at gmail.com > Subject: [dpdk-dev] [PATCH v2 06/13] mbuf: add functions to get the name of > an ol_flag > >

[dpdk-dev] [PATCH v2 3/4] hash: add fallback to software CRC32 implementation

2014-11-17 Thread Yerden Zhumabekov
17.11.2014 18:34, Ananyev, Konstantin ?: > Hi Yerden, > >> +static inline void >> +rte_hash_crc_set_alg(enum crc32_alg_t alg) >> +{ >> +int sse42_supp = rte_cpu_get_flag_enabled(RTE_CPUFLAG_SSE4_2); >> +enum crc32_alg_t alg_supp = sse42_supp ? CRC32_SSE42 : CRC32_SW; >> + >> +if (a

[dpdk-dev] [PATCH v2 02/13] ixgbe: fix remaining pkt_flags variable size to 64 bits

2014-11-17 Thread Thomas Monjalon
2014-11-17 18:03, Olivier MATZ: > Hi Miroslaw, > > On 11/17/2014 05:47 PM, Walukiewicz, Miroslaw wrote: > > > > > >> -Original Message- > >> From: Olivier Matz [mailto:olivier.matz at 6wind.com] > >> Sent: Friday, November 14, 2014 6:03 PM > >> To: dev at dpdk.org > >> Cc: olivier.matz a

[dpdk-dev] [PATCH v2 11/13] ixgbe: support TCP segmentation offload

2014-11-17 Thread Ananyev, Konstantin
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Olivier Matz > Sent: Friday, November 14, 2014 5:03 PM > To: dev at dpdk.org > Cc: jigsaw at gmail.com > Subject: [dpdk-dev] [PATCH v2 11/13] ixgbe: support TCP segmentation offload > > Implement TSO (TCP segm

[dpdk-dev] [PATCH v2 09/13] mbuf: introduce new checksum API

2014-11-17 Thread Ananyev, Konstantin
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Olivier Matz > Sent: Friday, November 14, 2014 5:03 PM > To: dev at dpdk.org > Cc: jigsaw at gmail.com > Subject: [dpdk-dev] [PATCH v2 09/13] mbuf: introduce new checksum API > > Introduce new functions to cal

[dpdk-dev] [PATCH v2 02/13] ixgbe: fix remaining pkt_flags variable size to 64 bits

2014-11-17 Thread Olivier MATZ
Hi Miroslaw, On 11/17/2014 05:47 PM, Walukiewicz, Miroslaw wrote: > > >> -Original Message- >> From: Olivier Matz [mailto:olivier.matz at 6wind.com] >> Sent: Friday, November 14, 2014 6:03 PM >> To: dev at dpdk.org >> Cc: olivier.matz at 6wind.com; Walukiewicz, Miroslaw; Liu, Jijiang; Li

[dpdk-dev] [PATCH v2 0/4] rte_hash_crc reworked to be platform-independent

2014-11-17 Thread Yerden Zhumabekov
17.11.2014 17:31, Neil Horman ?: > On Sun, Nov 16, 2014 at 11:59:16PM +0600, Yerden Zhumabekov wrote: >> This is a rework of my previous patches improving performance of >> rte_hash_crc. In addition, this revision brings a fallback mechanism to >> ensure that CRC32 hash is calculated regardl

[dpdk-dev] [PATCH] eal: allow virtual devices to be white/black listed

2014-11-17 Thread Bruce Richardson
On Mon, Nov 17, 2014 at 04:40:19PM +, Nicolas Pernas Maradei wrote: > > On 17/11/14 16:00, Neil Horman wrote: > >I get that, I was more asking, why those values? They seem a bit magic to > >me, > >and might benefit from some descriptive macros or comments so they make more > >sense > >Neil >

[dpdk-dev] [PATCH v2 02/13] ixgbe: fix remaining pkt_flags variable size to 64 bits

2014-11-17 Thread Walukiewicz, Miroslaw
> -Original Message- > From: Olivier Matz [mailto:olivier.matz at 6wind.com] > Sent: Friday, November 14, 2014 6:03 PM > To: dev at dpdk.org > Cc: olivier.matz at 6wind.com; Walukiewicz, Miroslaw; Liu, Jijiang; Liu, Yong; > jigsaw at gmail.com; Richardson, Bruce > Subject: [PATCH v2 02/13

[dpdk-dev] [PATCH] eal: allow virtual devices to be white/black listed

2014-11-17 Thread Nicolas Pernas Maradei
On 17/11/14 16:00, Neil Horman wrote: > I get that, I was more asking, why those values? They seem a bit magic to me, > and might benefit from some descriptive macros or comments so they make more > sense > Neil OK, I get you now. Maybe the diff is not very clear. I just left the original calls

[dpdk-dev] [PATCH] eal: allow virtual devices to be white/black listed

2014-11-17 Thread Neil Horman
On Mon, Nov 17, 2014 at 04:52:50PM +, Bruce Richardson wrote: > On Mon, Nov 17, 2014 at 04:40:19PM +, Nicolas Pernas Maradei wrote: > > > > On 17/11/14 16:00, Neil Horman wrote: > > >I get that, I was more asking, why those values? They seem a bit magic to > > >me, > > >and might benefit

[dpdk-dev] vhost-user technical isssues

2014-11-17 Thread Tetsuya Mukawa
Hi Xie, (2014/11/14 19:59), Xie, Huawei wrote: > I tested with latest qemu(with offset fix) in vhost app(not with test case), > unmap succeeds only when the size is aligned to 1GB(hugepage size). I appreciate for your testing. > Another important thing is could we do mmap(0, region[i].memory_si

[dpdk-dev] [PATCH RFC] lib/librte_vhost: vhost-user

2014-11-17 Thread Tetsuya Mukawa
Hi Xie, (2014/11/17 15:04), Tetsuya Mukawa wrote: > Hi Xie, > > > (2014/11/15 10:14), Huawei Xie wrote: >> implement socket server >> fd event dispatch mechanism >> vhost sock message handling >> memory map for each region >> VHOST_USER_SET_VRING_KICK_FD as the indicator that vring is available >

[dpdk-dev] [RFC PATCH 2/2] lib/librte_vhost: Add device abstraction layer

2014-11-17 Thread Tetsuya Mukawa
--- lib/librte_vhost/Makefile | 6 +- lib/librte_vhost/rte_virtio_net.h | 22 - lib/librte_vhost/vhost-cuse/vhost-net-cdev.c | 6 +- lib/librte_vhost/vhost-cuse/vhost-net-cdev.h | 40 + lib/librte_vhost/vhost-cuse/virtio-net-cdev.c | 1 + lib/l

[dpdk-dev] [RFC PATCH 1/2] lib/librte_vhost: change macro name of include guard.

2014-11-17 Thread Tetsuya Mukawa
This patch changes include macro name like following. - "_VIRTIO_NET_H_" > "_RTE_VIRTIO_NET_H_" --- lib/librte_vhost/rte_virtio_net.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/librte_vhost/rte_virtio_net.h b/lib/librte_vhost/rte_virtio_net.h index 7d7d001..a095

[dpdk-dev] [RFC PATCH] lib/librte_vhost: cleanup white spaces, tabs and indents

2014-11-17 Thread Tetsuya Mukawa
--- lib/librte_vhost/rte_virtio_net.h | 4 +-- lib/librte_vhost/vhost-cuse/vhost-net-cdev.c | 4 +-- lib/librte_vhost/vhost-cuse/virtio-net-cdev.c | 8 ++--- lib/librte_vhost/vhost-user/fd_man.c | 13 lib/librte_vhost/vhost-user/fd_man.h | 2 +- lib/libr

[dpdk-dev] [PATCH RFC] lib/librte_vhost: vhost-user

2014-11-17 Thread Tetsuya Mukawa
Hi Xie, (2014/11/15 10:14), Huawei Xie wrote: > implement socket server > fd event dispatch mechanism > vhost sock message handling > memory map for each region > VHOST_USER_SET_VRING_KICK_FD as the indicator that vring is available > VHOST_USER_GET_VRING_BASE as the message that vring should be

[dpdk-dev] [PATCH] eal: allow virtual devices to be white/black listed

2014-11-17 Thread Nicolas Pernas Maradei
On 17/11/14 14:10, Neil Horman wrote: > What are the above pci bus/device/function tuples? > > Neil Hi, I'm not sure if I understand the question. If you mean what kind of devices those are, as in physical or virtual, they are physical. rte_eal_devargs_add() would parse the string as a PCI ID.

[dpdk-dev] [PATCH v2 08/13] testpmd: rework csum forward engine

2014-11-17 Thread Olivier MATZ
Hi Jijiang, On 11/17/2014 09:11 AM, Liu, Jijiang wrote: >> +/* Calculate the checksum of outer header (only vxlan is supported, >> + * meaning IP + UDP). The caller already checked that it's a vxlan >> + * packet */ >> +static uint64_t >> +process_outer_cksums(void *outer_l3_hdr, uint16_t outer_et

[dpdk-dev] [PATCH v2 06/13] mbuf: add functions to get the name of an ol_flag

2014-11-17 Thread Olivier MATZ
Hi Bruce, On 11/17/2014 11:39 AM, Bruce Richardson wrote: >> +/* >> + * Get the name of a TX offload flag >> + */ >> +const char *rte_get_tx_ol_flag_name(uint64_t mask) >> +{ >> +switch (mask) { >> +case PKT_TX_VLAN_PKT: return "PKT_TX_VLAN_PKT"; >> +case PKT_TX_IP_CKSUM: return "PKT_T

[dpdk-dev] [PATCH v6 0/8] support of multiple sizes of redirection table

2014-11-17 Thread Ananyev, Konstantin
> -Original Message- > From: Zhang, Helin > Sent: Saturday, November 15, 2014 4:04 PM > To: dev at dpdk.org > Cc: Cao, Waterman; Cao, Min; Wu, Jingjing; Liu, Jijiang; Chen, Erlu; Ananyev, > Konstantin; Zhang, Helin > Subject: [PATCH v6 0/8] support of multiple sizes of redirection table

[dpdk-dev] [PATCH v2 3/4] hash: add fallback to software CRC32 implementation

2014-11-17 Thread Ananyev, Konstantin
Hi Yerden, > -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Yerden Zhumabekov > Sent: Sunday, November 16, 2014 5:59 PM > To: dev at dpdk.org > Subject: [dpdk-dev] [PATCH v2 3/4] hash: add fallback to software CRC32 > implementation > > Initially, SSE4.2 su

[dpdk-dev] [PATCH v2] librte_pmd_packet: add PMD for AF_PACKET-based virtual devices

2014-11-17 Thread Thomas Monjalon
2014-11-17 06:19, Neil Horman: > On Thu, Nov 13, 2014 at 12:57:25PM +0100, Thomas Monjalon wrote: > > 2014-11-13 06:14, Neil Horman: > > > Do you have an example set of tests that the other pmd's have followed > > > for this? > > > > You can check this: > > http://dpdk.org/browse/tools/dts/tr

[dpdk-dev] [PATCH v8 10/10] app/testpmd:test VxLAN Tx checksum offload

2014-11-17 Thread Olivier MATZ
Hi Jijiang, On 11/17/2014 07:52 AM, Liu, Jijiang wrote: > Anyway, I explain the checksum mechanism here again. > > In my VXLAN patch set, for an VXLAN packet TX checksum offload, the logics > below: > > 1. only set outer L3/L4 header TX checksum > tx_checksum set 0x3(0r 0x1) 0 > In this

[dpdk-dev] [RFC PATCH] Enable uio_pci_generic support

2014-11-17 Thread Bruce Richardson
On Tue, Oct 28, 2014 at 01:48:02AM +0800, Danny Zhou wrote: > Linux kernel provides UIO as well as VFIO mechanism to support writing user > space device driver. Comparing to UIO which is available since 2.6.32 kernel, > the VFIO is introduced into kernel since version 3.6.0 with better interrupt >

[dpdk-dev] [PATCH 2/4] doc: Corrected info for tx_checksum set mask function, in testpmd UG

2014-11-17 Thread Olivier MATZ
Hi Pablo, On 11/15/2014 08:13 PM, Pablo de Lara wrote: > tx_checksum set mask function now allows 4 extra bits in the mask > for TX checksum offload > > Signed-off-by: Pablo de Lara > --- > doc/guides/testpmd_app_ug/testpmd_funcs.rst | 10 +- > 1 files changed, 9 insertions(+), 1 dele

[dpdk-dev] [PATCH] eal: allow virtual devices to be white/black listed

2014-11-17 Thread Neil Horman
On Mon, Nov 17, 2014 at 02:52:18PM +, Nicolas Pernas Maradei wrote: > > On 17/11/14 14:10, Neil Horman wrote: > >What are the above pci bus/device/function tuples? > Hi, > > I'm not sure if I understand the question. If you mean what kind of devices > those are, as in physical or virtual, the

[dpdk-dev] [PATCH v3] librte_pmd_af_packet: add PMD for AF_PACKET-based virtual devices

2014-11-17 Thread John W. Linville
This is a Linux-specific virtual PMD driver backed by an AF_PACKET socket. This implementation uses mmap'ed ring buffers to limit copying and user/kernel transitions. The PACKET_FANOUT_HASH behavior of AF_PACKET is used for frame reception. In the current implementation, Tx and Rx queues are alw

[dpdk-dev] one lightwight rte_eal_init() for SECONDARY processes which only use sharedmemory

2014-11-17 Thread Bruce Richardson
On Sun, Nov 16, 2014 at 03:06:15AM +, Chi, Xiaobo (NSN - CN/Hangzhou) wrote: > Hi, Bruce and Neil, > Do you indicate that to add one program arg, such as "--mem-only", to let the > rte_eal_init()only to do those memory initializing things? I am worrying that > this would make rte_eal_init() m

[dpdk-dev] [PATCH 2/4] doc: Corrected info for tx_checksum set mask function, in testpmd UG

2014-11-17 Thread De Lara Guarch, Pablo
Hi Olivier, > -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Olivier MATZ > Sent: Monday, November 17, 2014 10:39 AM > To: De Lara Guarch, Pablo; dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH 2/4] doc: Corrected info for tx_checksum set > mask function, in

[dpdk-dev] [PATCH v2 06/13] mbuf: add functions to get the name of an ol_flag

2014-11-17 Thread Bruce Richardson
On Fri, Nov 14, 2014 at 06:03:22PM +0100, Olivier Matz wrote: > In test-pmd (rxonly.c), the code is able to dump the list of ol_flags. > The issue is that the list of flags in the application has to be > synchronized with the flags defined in rte_mbuf.h. > > This patch introduces 2 new functions r

[dpdk-dev] [PATCH v2 05/13] mbuf: remove too specific PKT_TX_OFFLOAD_MASK definition

2014-11-17 Thread Bruce Richardson
On Fri, Nov 14, 2014 at 06:03:21PM +0100, Olivier Matz wrote: > This definition is specific to Intel PMD drivers and its definition > "indicate what bits required for building TX context" shows that it > should not be in the generic rte_mbuf.h but in the PMD driver. > > Signed-off-by: Olivier Matz

[dpdk-dev] Load-balancing position field in DPDK load_balancer sample app vs. Hash table

2014-11-17 Thread Chilikin, Andrey
Fortville can calculate hash for packets encapsulated into different tunnels: GRE/NVGRE, VXLAN, QinQ (S-Tag + C-Tag), but at the moment current version of DPDK supports only VXLAN. Regards, Andrey -Original Message- From: Yerden Zhumabekov [mailto:e_zhumabe...@sts.kz] Sent: Friday, Nov

[dpdk-dev] [PULL REQUEST] doc: programmers guide.

2014-11-17 Thread Bernard Iremonger
These changes are a conversion of the Programmers Guide from an MSWord file to Sphinx rst files. The following changes since commit 07db4a9750940bbf95299a89afd12d178556e064: examples/distributor: new sample app (2014-11-16 22:54:56 +0100) are available in the git repository at: git://dpdk.o

[dpdk-dev] [PATCH] llib/ibrte_net: workaround to avoid macro conflict

2014-11-17 Thread Neil Horman
On Mon, Nov 03, 2014 at 08:41:53AM +0100, Thomas Monjalon wrote: > 2014-10-09 07:29, Neil Horman: > > On Thu, Oct 09, 2014 at 05:20:31AM +, Wu, Jingjing wrote: > > > Hi, Neil > > > > > > To have rte_ip.h include netinet/in.h directly is also a choice. > > > > > > But netinet/in.h contains a l

[dpdk-dev] [PATCH] eal: allow virtual devices to be white/black listed

2014-11-17 Thread Neil Horman
On Sun, Nov 16, 2014 at 09:26:55PM +, Nicol?s Pernas Maradei wrote: > From: Nicol?s Pernas Maradei > > Virtual and physical devices are now treated the same in terms of > white/black listing. Virtual devices can be defined using --vdev as > before and also whitelisted (using -w vdev_name) or

[dpdk-dev] [PATCH v2 3/4] hash: add fallback to software CRC32 implementation

2014-11-17 Thread Neil Horman
On Mon, Nov 17, 2014 at 12:34:04PM +, Ananyev, Konstantin wrote: > > Hi Yerden, > > > -Original Message- > > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Yerden Zhumabekov > > Sent: Sunday, November 16, 2014 5:59 PM > > To: dev at dpdk.org > > Subject: [dpdk-dev] [PATCH v2

[dpdk-dev] [PATCH v2 08/13] testpmd: rework csum forward engine

2014-11-17 Thread Liu, Jijiang
> -Original Message- > From: Olivier Matz [mailto:olivier.matz at 6wind.com] > Sent: Saturday, November 15, 2014 1:03 AM > To: dev at dpdk.org > Cc: olivier.matz at 6wind.com; Walukiewicz, Miroslaw; Liu, Jijiang; Liu, Yong; > jigsaw at gmail.com; Richardson, Bruce > Subject: [PATCH v2 08/

[dpdk-dev] [PATCH v8 10/10] app/testpmd:test VxLAN Tx checksum offload

2014-11-17 Thread Liu, Jijiang
> -Original Message- > From: Olivier MATZ [mailto:olivier.matz at 6wind.com] > Sent: Friday, November 14, 2014 5:10 PM > To: Liu, Jijiang; Thomas Monjalon > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH v8 10/10] app/testpmd:test VxLAN Tx checksum > offload > > Hi Jijiang, > > On

[dpdk-dev] [PATCH v2 0/4] rte_hash_crc reworked to be platform-independent

2014-11-17 Thread Neil Horman
On Sun, Nov 16, 2014 at 11:59:16PM +0600, Yerden Zhumabekov wrote: > This is a rework of my previous patches improving performance of > rte_hash_crc. In addition, this revision brings a fallback mechanism to > ensure that CRC32 hash is calculated regardless of hardware support from CPU > (i.e. S

[dpdk-dev] [PATCH v2] librte_pmd_packet: add PMD for AF_PACKET-based virtual devices

2014-11-17 Thread Neil Horman
On Thu, Nov 13, 2014 at 12:57:25PM +0100, Thomas Monjalon wrote: > 2014-11-13 06:14, Neil Horman: > > On Thu, Nov 13, 2014 at 02:03:18AM -0800, Thomas Monjalon wrote: > > > 2014-10-08 15:14, Neil Horman: > > > > On Wed, Oct 08, 2014 at 05:57:46PM +0200, Thomas Monjalon wrote: > > > > > 2014-09-29 1

[dpdk-dev] Load-balancing position field in DPDK load_balancer sample app vs. Hash table

2014-11-17 Thread Zhang, Helin
Hi Andrey Yes, Fortville supports hardware symmetric hashing offload. I am waiting for the comments of its patch set submitted recently, and hopefully it can be accepted soon. To use it, we need to enable the hash function of symmetric, but not the default one of Toeplitz hash function. Then we n