[dpdk-dev] [PATCH v3 2/2] virtio/vdev: add a new vdev named eth_cvio

2016-04-22 Thread Tan, Jianfeng
Hi, On 4/22/2016 3:36 PM, David Marchand wrote: > Hello, > > On Fri, Apr 22, 2016 at 7:15 AM, Tan, Jianfeng > wrote: >> On 4/21/2016 4:51 PM, David Marchand wrote: >>> virtio code relies on drv_flags (even updating it while this should be >>> per-device). >>> So first, virtio should rely on

[dpdk-dev] [PATCH v3 2/2] virtio/vdev: add a new vdev named eth_cvio

2016-04-22 Thread Tan, Jianfeng
Hi Yuanhan, On 4/22/2016 6:14 AM, Yuanhan Liu wrote: > On Thu, Apr 21, 2016 at 02:56:36AM +, Jianfeng Tan wrote: >> Add a new virtual device named eth_cvio, it can be used just like >> eth_ring, eth_null, etc. >> >> Configured parameters include: >>- rx (optional, 1 by default), number of

[dpdk-dev] [PATCH v3 1/2] virtio/vdev: add embeded device emulation

2016-04-22 Thread Tan, Jianfeng
Hi Yuanhan, On 4/22/2016 6:01 AM, Yuanhan Liu wrote: > On Thu, Apr 21, 2016 at 02:56:35AM +, Jianfeng Tan wrote: >> Background: Previously, we usually use a virtio device in QEMU/VM's >> context as below pic shows. Virtio nic is emulated in QEMU, and usually >> presented in VM as a PCI

[dpdk-dev] [PATCH] app/test_acl: fix division by float zero

2016-04-22 Thread Daniel Mrzyglod
Fix issue reported by Coverity. Coverity ID 13240 This could cause an immediate crash or incorrect computation. In search_ip5tuples: An expression which may be zero is used as a divisor in floating-point arithmetic. divide_by_zero: In expression (long double)tm / pkt, division by expression pkt

[dpdk-dev] [PATCH v2] ixgbe: fix bad shift operation in ixgbe_set_pool_rx/tx

2016-04-22 Thread Tomasz Kulasek
Fix issue reported by Coverity. Coverity ID 13193: Bad bit shift operation (BAD_SHIFT) large_shift: In expression 1 << pool, left shifting by more than 31 bits has undefined behavior. The shift amount, pool, is at least 32. This patch is a rework of register addr selection logic and mask

[dpdk-dev] [PATCH v2 3/3] app/testpmd: support setting up txq_flags value in command line

2016-04-22 Thread maciej.cze...@caviumnetworks.com
From: Maciej Czekaj "port config all txqflags " allows for specifying txq_flags value in command line. Signed-off-by: Maciej Czekaj --- app/test-pmd/cmdline.c | 69 +

[dpdk-dev] [PATCH v2 2/3] app/testpmd: extend port config with scatter parameter

2016-04-22 Thread maciej.cze...@caviumnetworks.com
From: Maciej Czekaj "port config all scatter on|off" allows for controlling rxmode.enable_scatter in command line. Signed-off-by: Maciej Czekaj --- app/test-pmd/cmdline.c | 17 +

[dpdk-dev] [PATCH v2 1/3] app/testpmd: add "enable-scatter" parameter

2016-04-22 Thread maciej.cze...@caviumnetworks.com
From: Maciej Czekaj This parameter allows for controlling rxmode.enable_scatter which in turn allow for multi-segment packet receive tests. Signed-off-by: Maciej Czekaj --- app/test-pmd/parameters.c | 3 +++

[dpdk-dev] [PATCH v2 0/3] testpmd: extend commands for better scatter-gather tests

2016-04-22 Thread maciej.cze...@caviumnetworks.com
From: Maciej Czekaj v2: - included documentation changes - added value check for "port config all txqflags" as in --tqxflags paramater Maciej Czekaj (3): app/testpmd: add "enable-scatter" parameter app/testpmd: extend port config with scatter parameter

[dpdk-dev] [PATCH] version: 16.07-rc0

2016-04-22 Thread Thomas Monjalon
After having removed the deprecated stuff, we can start pushing new fixes and features in the version 16.07. Signed-off-by: Thomas Monjalon --- lib/librte_eal/common/include/rte_version.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git

[dpdk-dev] [PATCH] remove poisoned flags

2016-04-22 Thread Thomas Monjalon
2016-04-20 10:59, David Marchand: > On Tue, Apr 19, 2016 at 10:19 PM, Thomas Monjalon > wrote: > > Some flags were poisoned after having been removed from EAL and mbuf > > in releases 1.8 (b10eef348d, 62814bc2e9) and 2.0 (4769bc5a27cc). > > After several releases, they have probably disappeared

[dpdk-dev] [PATCH] hash: remove aliases for bulk lookup

2016-04-22 Thread Thomas Monjalon
2016-04-19 21:41, Thomas Monjalon: > The function rte_hash_lookup_multi() was renamed rte_hash_lookup_bulk() > in DPDK 1.4 and was kept as an undocumented alias. > > Signed-off-by: Thomas Monjalon Applied

[dpdk-dev] [PATCH v2] ethdev: remove deprecated statistics

2016-04-22 Thread Thomas Monjalon
2016-04-20 11:54, Remy Horton: > On 20/04/2016 10:47, Thomas Monjalon wrote: > > Some statistics were deprecated since release 2.1 (49f386542af4). > > The last deprecated counter to be used was imcasts. > > > > The VF loopback statistics are also removed as they are used only > > in igb and

[dpdk-dev] [PATCH 4/4] eal: add assert macro for debug

2016-04-22 Thread Thomas Monjalon
The macro RTE_VERIFY always checks a condition. It is optimized with "unlikely" hint. While this macro is well suited for test applications, it is preferred in libraries and examples to enable such check in debug mode. That's why the macro RTE_ASSERT is introduced to call RTE_VERIFY only if built

[dpdk-dev] [PATCH 1/4] eal: increase log level of some messages

2016-04-22 Thread Thomas Monjalon
Some relevant EAL logs are raised from debug to info level in order to show the available number of cores and the detected devices. The driver ids and name are logged only if the device is not blacklisted. Signed-off-by: Thomas Monjalon --- lib/librte_eal/common/eal_common_lcore.c | 2 +-

[dpdk-dev] [PATCH 0/4] cleanup debug and dead code

2016-04-22 Thread Thomas Monjalon
With this series, the default log level is not debug anymore. And more code depends on debug level instead of having some almost dead code. Thomas Monjalon (4): eal: increase log level of some messages log: increase default level to info examples: remove useless debug flags eal: add

[dpdk-dev] [PATCH 4/4] eal: add assert macro for debug

2016-04-22 Thread Stephen Hemminger
On Fri, 22 Apr 2016 15:08:50 -0700 Yuanhan Liu wrote: > On Fri, Apr 22, 2016 at 11:14:35PM +0200, Thomas Monjalon wrote: > > 2016-04-22 12:51, Yuanhan Liu: > > > On Fri, Apr 22, 2016 at 03:44:00PM +0200, Thomas Monjalon wrote: > > > > --- a/drivers/net/vmxnet3/vmxnet3_rxtx.c > > > > +++

[dpdk-dev] [PATCH v3 2/2] virtio/vdev: add a new vdev named eth_cvio

2016-04-22 Thread Tan, Jianfeng
Hi Thomas, On 4/21/2016 6:05 PM, Thomas Monjalon wrote: > 2016-04-21 02:56, Jianfeng Tan: >> Add a new virtual device named eth_cvio, it can be used just like >> eth_ring, eth_null, etc. > Why this name eth_cvio? > Why the prefix eth_? > The virtio-net driver uses a kernel device. Here it is a

[dpdk-dev] [PATCH v3 01/13] e1000: move pci device ids to driver

2016-04-22 Thread Thomas Monjalon
2016-04-22 08:13, Neil Horman: > On Thu, Apr 21, 2016 at 02:41:38PM +0200, Thomas Monjalon wrote: > > 2016-04-21 08:08, Neil Horman: > > > On Thu, Apr 21, 2016 at 09:27:18AM +0200, David Marchand wrote: > > > > I don't mind doing trivial changes, but I don't have time for more on > > > > this

[dpdk-dev] [PATCH v2] eal: fix unchecked return value from library

2016-04-22 Thread Daniel Mrzyglod
Fix issue reported by Coverity. Coverity ID 13194 The function returns a value that indicates an error condition. If this is not checked, the error condition may not be handled correctly. In pci_vfio_mp_sync_thread: Value returned from a library function is not checked for errors before being

[dpdk-dev] [PATCH] app/test_acl: fix division by float zero

2016-04-22 Thread Ananyev, Konstantin
> -Original Message- > From: Mrzyglod, DanielX T > Sent: Friday, April 22, 2016 5:01 PM > To: Ananyev, Konstantin > Cc: dev at dpdk.org; Mrzyglod, DanielX T > Subject: [PATCH] app/test_acl: fix division by float zero > > Fix issue reported by Coverity. > Coverity ID 13240 > > This could

[dpdk-dev] [PATCH 4/4] eal: add assert macro for debug

2016-04-22 Thread Yuanhan Liu
On Fri, Apr 22, 2016 at 11:14:35PM +0200, Thomas Monjalon wrote: > 2016-04-22 12:51, Yuanhan Liu: > > On Fri, Apr 22, 2016 at 03:44:00PM +0200, Thomas Monjalon wrote: > > > --- a/drivers/net/vmxnet3/vmxnet3_rxtx.c > > > +++ b/drivers/net/vmxnet3/vmxnet3_rxtx.c > > > @@ -662,8 +662,8 @@

[dpdk-dev] [RFC PATCH v1 0/3] Remove string operations from xstats

2016-04-22 Thread David Harton (dharton)
> -Original Message- > From: Mcnamara, John [mailto:john.mcnamara at intel.com] > Sent: Wednesday, April 20, 2016 12:50 PM > To: David Harton (dharton) ; Horton, Remy > ; dev at dpdk.org > Cc: Van Haaren, Harry > Subject: RE: [dpdk-dev] [RFC PATCH v1 0/3] Remove string operations from >

[dpdk-dev] [PATCH] virtio: fix segfault when transmit pkts

2016-04-22 Thread Xie, Huawei
On 4/22/2016 6:43 AM, Yuanhan Liu wrote: > On Thu, Apr 21, 2016 at 12:36:10PM +, Jianfeng Tan wrote: >> Issue: when using virtio nic to transmit pkts, it causes segment fault. > Jianfeng, > > It's good to describe the issues, steps to reproduce it and how to fix > it. But you don't have to

[dpdk-dev] Segmentation fault in ip_frag_find

2016-04-22 Thread Padam Jeet Singh
Hello, We are using the DPDK framework and testing the IP reassembly code. While reassembly is working fine, the application segfaults with the following (partially shown) stack trace. The segfault is not entirely reproducible - it happens quite randomly. Program terminated with signal 11,

[dpdk-dev] Question about memzone failure problem.

2016-04-22 Thread Wiles, Keith
I was creating a rte_ring and got a message ?RING: Cannot reserve memory? I track it down to me trying create a ring using the same name as another ring. The question is I tracked down the problem and in the eal_common_memzone.c file I noticed the message for the problem was using DEBUG in the

[dpdk-dev] [PATCH 1/1] lib/librte_eal: fix resource leak

2016-04-22 Thread Panu Matilainen
On 04/21/2016 02:19 PM, Sergio Gonzalez Monroy wrote: > On 20/04/2016 10:15, David Marchand wrote: >> On Tue, Apr 19, 2016 at 6:27 PM, Marcin Kerlin >> wrote: >>> Fix issue reported by Coverity. >>> >>> Coverity ID 13295, 13296, 13303: >>> Resource leak: The system resource will not be reclaimed

[dpdk-dev] [PATCH v3 2/2] virtio/vdev: add a new vdev named eth_cvio

2016-04-22 Thread Tan, Jianfeng
Hi, On 4/21/2016 4:51 PM, David Marchand wrote: > Hello, > > On Thu, Apr 21, 2016 at 4:56 AM, Jianfeng Tan > wrote: >> Add a new virtual device named eth_cvio, it can be used just like >> eth_ring, eth_null, etc. >> >> Configured parameters include: >>- rx (optional, 1 by default), number

[dpdk-dev] [dpdk-users] DPDK 16.04 link changes cause PMD drivers to not be loaded

2016-04-22 Thread Panu Matilainen
On 04/21/2016 06:24 PM, Aurojit Panda wrote: > > > Thomas Monjalon wrote: >> 2016-04-21 08:01, Aurojit Panda: >>> Panu Matilainen wrote: >> [...] Again, PMDs are *plugins* that are *meant* to be loaded at runtime. That allows for all sorts of flexibility especially for packaging and

[dpdk-dev] [PATCH 4/4] eal: add assert macro for debug

2016-04-22 Thread Yuanhan Liu
On Fri, Apr 22, 2016 at 03:44:00PM +0200, Thomas Monjalon wrote: > diff --git a/drivers/net/vmxnet3/vmxnet3_rxtx.c > b/drivers/net/vmxnet3/vmxnet3_rxtx.c > index 4ac0456..b7486cd 100644 > --- a/drivers/net/vmxnet3/vmxnet3_rxtx.c > +++ b/drivers/net/vmxnet3/vmxnet3_rxtx.c > @@ -296,7 +296,7 @@

[dpdk-dev] [PATCH] examples/netmap_compat: fix infinite loop

2016-04-22 Thread Michal Kobylinski
Fix issue reported by Coverity. Coverity ID 30701: Infinite loop: The loop does not have a normal termination condition, so will continue until an abnormal condition arises. In rte_netmap_poll: Infinite loop with unsatisfiable exit condition. Fixes: 06371afe394d ("examples/netmap_compat: import

[dpdk-dev] [PATCH] mem: fix overflowed return value

2016-04-22 Thread Michal Kobylinski
Fix issue reported by Coverity. Coverity ID 13255: Overflowed return value: The return value will be too small or even negative, likely resulting in unexpected behavior in a caller that uses the return value. In rte_mem_virt2phy: An integer overflow occurs, with the overflowed value used as the

[dpdk-dev] [PATCH] cfgfile: fix integer overflow

2016-04-22 Thread Michal Kobylinski
Fix issue reported by Coverity. Coverity ID 13289: Integer overflowed argument: The argument will be too small or even negative, likely resulting in unexpected behavior (for example, under-allocation in a memory allocation function). In rte_cfgfile_load: An integer overflow occurs, with the

[dpdk-dev] [PATCH v2 00/17] prepare for rte_device / rte_driver

2016-04-22 Thread Jan Viktorin
[...] > > > > > > Changes since v1: > > > - rebased on HEAD, new drivers should be okay > > > - patches have been split into smaller pieces > > > - RTE_INIT macro has been added, but in the end, I am not sure it is > > > useful > > > - device type has been removed from ethdev, as it was used

[dpdk-dev] [PATCH v3 1/2] virtio/vdev: add embeded device emulation

2016-04-22 Thread Thomas Monjalon
2016-04-22 18:12, Tan, Jianfeng: > On 4/22/2016 6:01 AM, Yuanhan Liu wrote: > > Further more, I'd suggest to divide the code into following files: > > > > - virtio-user/virtio.c > > > >All virtio device emulation goes here. > > > > - virtio-user/vhost-user.c > > > >The vhost-user frontend

[dpdk-dev] [PATCH 3/3] app/testpmd: support setting up txq_flags value in command line

2016-04-22 Thread maciej.cze...@caviumnetworks.com
From: Maciej Czekaj "port config all txqflags " allows for specifying txq_flags value in command line. Signed-off-by: Maciej Czekaj --- app/test-pmd/cmdline.c | 64 ++ 1 file changed, 64

[dpdk-dev] [PATCH 1/3] app/testpmd: add "enable-scatter" parameter

2016-04-22 Thread maciej.cze...@caviumnetworks.com
From: Maciej Czekaj This parameter allows for controlling rxmode.enable_scatter which in turn allow for multi-segment packet receive tests. Signed-off-by: Maciej Czekaj --- app/test-pmd/parameters.c | 3 +++ 1 file changed, 3 insertions(+)

[dpdk-dev] [PATCH 0/3] testpmd: extend commands for better scatter-gather tests

2016-04-22 Thread maciej.cze...@caviumnetworks.com
From: Maciej Czekaj This patch adds 1 parameter and 2 command line items: * --enable-scatter * port config all scatter on|off * port config all txqflags value With these, testpmd can be used for testing scatter-gather in both TX and RX. Maciej Czekaj (3):

[dpdk-dev] ivshmem is secure or not ? why ?

2016-04-22 Thread Mauricio Vásquez
Hello Yangyongqiang, On Fri, Apr 22, 2016 at 9:55 AM, Yangyongqiang (Tony, Shannon) < yangyongqiang at huawei.com> wrote: > From http://dpdk.org/doc/guides/prog_guide/ivshmem_lib.html, I get this > : different vms can use different metadatas, so different vms can have > different memory shared

[dpdk-dev] [PATCH] examples/netmap_compat: fix infinite loop

2016-04-22 Thread Mcnamara, John
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Michal Kobylinski > Sent: Friday, April 22, 2016 11:47 AM > To: Richardson, Bruce ; dev at dpdk.org > Cc: Kobylinski, MichalX > Subject: [dpdk-dev] [PATCH] examples/netmap_compat: fix infinite loop > > > +

[dpdk-dev] Compiler minimum versions. (Was: [PATCH v5 05/10] qede: Add core driver)

2016-04-22 Thread Bruce Richardson
On Wed, Apr 20, 2016 at 10:54:51AM +0200, Thomas Monjalon wrote: > 2016-04-20 01:09, Rasesh Mody: > > Hi Thomas, > > > > > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > > > Sent: Tuesday, April 19, 2016 5:59 AM > > > > > > 2016-03-31 19:15, Rasesh Mody: > > > > +ifeq

[dpdk-dev] Odp.: [PATCH 0/3] testpmd: extend commands for better scatter-gather tests

2016-04-22 Thread Czekaj, Maciej
Od: dev w imieniu u?ytkownika Mcnamara, John Wys?ane: 22 kwietnia 2016 12:19 Do: Czekaj, Maciej; De Lara Guarch, Pablo DW: dev at dpdk.org Temat: Re: [dpdk-dev] [PATCH 0/3] testpmd: extend commands for better scatter-gather tests > -Original

[dpdk-dev] [PATCH v3 2/2] virtio/vdev: add a new vdev named eth_cvio

2016-04-22 Thread Thomas Monjalon
2016-04-22 15:26, Tan, Jianfeng: > Hi Thomas, > > On 4/21/2016 6:05 PM, Thomas Monjalon wrote: > > 2016-04-21 02:56, Jianfeng Tan: > >> Add a new virtual device named eth_cvio, it can be used just like > >> eth_ring, eth_null, etc. > > Why this name eth_cvio? > > Why the prefix eth_? > > The

[dpdk-dev] [PATCH v3 1/2] virtio/vdev: add embeded device emulation

2016-04-22 Thread Yuanhan Liu
On Fri, Apr 22, 2016 at 06:12:21PM +0800, Tan, Jianfeng wrote: > >>How: we implement another instance of struct virtio_pci_ops to intercept > >>the communications between VM and QEMU. Instead of rd/wr ioport or PCI > >>configuration space, here we directly talk with backend through the vhost >

[dpdk-dev] [PATCH 0/3] testpmd: extend commands for better scatter-gather tests

2016-04-22 Thread Mcnamara, John
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of > maciej.czekaj at caviumnetworks.com > Sent: Friday, April 22, 2016 10:58 AM > To: De Lara Guarch, Pablo > Cc: dev at dpdk.org; Maciej Czekaj > Subject: [dpdk-dev] [PATCH 0/3] testpmd: extend commands for

[dpdk-dev] [PATCH v2][PKTGEN] Rate as a decimal number

2016-04-22 Thread inaki.murillo
Currently Pktgen does not accept a decimal number for the rate. This patch makes possible to set a decimal number as a rate. Signed-off-by: I?aki Murillo Arroyo --- app/cmd-functions.c | 71 --- app/cmd-functions.h | 38

[dpdk-dev] [PATCH] ip_frag : Fix double-free of chained mbufs

2016-04-22 Thread Ananyev, Konstantin
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of cychong > Sent: Thursday, April 21, 2016 6:28 PM > To: dev at dpdk.org > Cc: cychong at gmail.com > Subject: [dpdk-dev] [PATCH] ip_frag : Fix double-free of chained mbufs > > If any fragment hole is found in

[dpdk-dev] [PATCH v3 2/2] virtio/vdev: add a new vdev named eth_cvio

2016-04-22 Thread David Marchand
Hello, On Fri, Apr 22, 2016 at 7:15 AM, Tan, Jianfeng wrote: > On 4/21/2016 4:51 PM, David Marchand wrote: >> virtio code relies on drv_flags (even updating it while this should be >> per-device). >> So first, virtio should rely on dev_flags. > > > Mainly drv_flags's RTE_PCI_DRV_INTR_LSC, and

[dpdk-dev] [PATCH] mem: fix overflowed return value

2016-04-22 Thread Stephen Hemminger
On Fri, 22 Apr 2016 12:44:18 +0200 Michal Kobylinski wrote: > Fix issue reported by Coverity. > > Coverity ID 13255: Overflowed return value: The return value will be too > small or even negative, likely resulting in unexpected behavior in a > caller that uses the return value. In

[dpdk-dev] [PATCH] cfgfile: fix integer overflow

2016-04-22 Thread Stephen Hemminger
On Fri, 22 Apr 2016 12:41:01 +0200 Michal Kobylinski wrote: > Fix issue reported by Coverity. > > Coverity ID 13289: Integer overflowed argument: The argument will be too > small or even negative, likely resulting in unexpected behavior (for > example, under-allocation in a memory allocation

[dpdk-dev] ixgbe : query regarding your code changes for VF mac add

2016-04-22 Thread Ivan Boule
Hi Santosh, My job at 6WIND does not consist in answering to DPDK questions. In general, I have other priorities, including vacations... In the meantime, nobody prevents you to add traces in the code to really understand what happens, as suggested in my last answer. Regards, Ivan On

[dpdk-dev] ixgbe : query regarding your code changes for VF mac add

2016-04-22 Thread Ivan Boule
Hi Santosh, To explain what's really going on in your use case, it would be better to add traces in the following functions of the ixgbevf PMD: Ivan At the end of eth_ixgbevf_dev_init() printf("%s portid=%d mac=%02X:%02X:%02X:%02X:%02X:%02X\n", __func__, dev->data->port_id,

[dpdk-dev] [PATCH] kni: set kni mac on ioctl_create

2016-04-22 Thread Igor Ryzhov
v2 is here, sorry: http://www.dpdk.org/dev/patchwork/patch/12190/. > 22 ???. 2016 ?., ? 4:57, Zhang, Helin ???(?): > > > > From: Igor Ryzhov [mailto:iryzhov at nfware.com] > Sent: Thursday, April 21, 2016 11:16 PM > To: Zhang, Helin > Cc: Sergey Balabanov; dev at dpdk.org > Subject: Re:

[dpdk-dev] ivshmem is secure or not ? why ?

2016-04-22 Thread Yangyongqiang (Tony, Shannon)
>From http://dpdk.org/doc/guides/prog_guide/ivshmem_lib.html, I get this : >different vms can use different metadatas, so different vms can have different >memory shared with host. For example: If vm1 shares MZ1 with host, and vm2 shares MZ2 with host, then vm1 can not look MZ2. If this is

[dpdk-dev] [PATCH v3] i40evf: Report error if HW CRC strip is disabled for non-DPDK PF hosts

2016-04-22 Thread Björn Töpel
On hosts running a non-DPDK PF driver, the VF has no means of changing the HW CRC strip setting for a RX queue. It's implicitly enabled. This patch checks if the host is running a non-DPDK PF kernel driver, and returns an error, if HW CRC stripping was disabled. Signed-off-by: Bj?rn T?pel ---

[dpdk-dev] [PATCH v2] kni: don't reassign ethernet address every time an interface goes up

2016-04-22 Thread Zhang, Helin
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Igor Ryzhov > Sent: Friday, April 22, 2016 12:28 AM > To: dev at dpdk.org > Subject: [dpdk-dev] [PATCH v2] kni: don't reassign ethernet address every > time an interface goes up > > Currently every time a KNI

[dpdk-dev] [RFC PATCH 0/2] performance utility in testpmd

2016-04-22 Thread Wang, Zhihong
> -Original Message- > From: Richardson, Bruce > Sent: Thursday, April 21, 2016 7:00 PM > To: Thomas Monjalon > Cc: Wang, Zhihong ; dev at dpdk.org; De Lara > Guarch, > Pablo > Subject: Re: [dpdk-dev] [RFC PATCH 0/2] performance utility in testpmd > > On Thu, Apr 21, 2016 at

[dpdk-dev] [RFC PATCH 0/2] performance utility in testpmd

2016-04-22 Thread Wang, Zhihong
> -Original Message- > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Thursday, April 21, 2016 5:54 PM > To: Wang, Zhihong > Cc: dev at dpdk.org; De Lara Guarch, Pablo > Subject: Re: [dpdk-dev] [RFC PATCH 0/2] performance utility in testpmd > > 2016-04-20 18:43,

[dpdk-dev] [PATCH v2] i40evf: Report error if HW CRC strip is disabled for Linux PF hosts

2016-04-22 Thread Zhang, Helin
> -Original Message- > From: Topel, Bjorn > Sent: Friday, April 22, 2016 1:17 PM > To: Zhang, Helin ; dev at dpdk.org > Cc: david.marchand at 6wind.com; Wu, Jingjing > Subject: RE: [PATCH v2] i40evf: Report error if HW CRC strip is disabled for > Linux > PF hosts > > >> >> + /*

[dpdk-dev] [PATCH v2] i40evf: Report error if HW CRC strip is disabled for Linux PF hosts

2016-04-22 Thread Topel, Bjorn
>> >> + /* For Linux PF hosts, VF has no ability to disable HW CRC strip, >> >> + * and is implicitly enabled by the PF. >> >> + */ >> >> + if (!conf->rxmode.hw_strip_crc) { >> >> + vf = I40EVF_DEV_PRIVATE_TO_VF(dev->data->dev_private); >> >> + if

[dpdk-dev] [PATCH v2] i40evf: Report error if HW CRC strip is disabled for Linux PF hosts

2016-04-22 Thread Topel, Bjorn
>> + /* For Linux PF hosts, VF has no ability to disable HW CRC strip, >> + * and is implicitly enabled by the PF. >> + */ >> + if (!conf->rxmode.hw_strip_crc) { >> + vf = I40EVF_DEV_PRIVATE_TO_VF(dev->data->dev_private); >> + if ((vf->version_major ==

[dpdk-dev] [PATCH] ip_frag : Fix double-free of chained mbufs

2016-04-22 Thread cychong
If any fragment hole is found in ipv4_frag_reassemble() and ipv6_frag_reassemble(), whole ip_frag_pkt mbufs are moved to death-row. Any mbufs already chained to another mbuf are freed multiple times as there are still in ip_frag_pkt array. Signed-off-by: cychong ---

[dpdk-dev] [PATCH] kni: set kni mac on ioctl_create

2016-04-22 Thread Zhang, Helin
From: Igor Ryzhov [mailto:iryz...@nfware.com] Sent: Thursday, April 21, 2016 11:16 PM To: Zhang, Helin Cc: Sergey Balabanov; dev at dpdk.org Subject: Re: [dpdk-dev] [PATCH] kni: set kni mac on ioctl_create Hello. I rebased a patch and added Suggested-by string. Check it,

[dpdk-dev] [PATCH v2] i40evf: Report error if HW CRC strip is disabled for Linux PF hosts

2016-04-22 Thread Zhang, Helin
> -Original Message- > From: Topel, Bjorn > Sent: Thursday, April 21, 2016 11:03 PM > To: dev at dpdk.org > Cc: david.marchand at 6wind.com; Zhang, Helin ; Wu, > Jingjing ; Topel, Bjorn > Subject: [PATCH v2] i40evf: Report error if HW CRC strip is disabled for > Linux PF > hosts > >

[dpdk-dev] [RFC PATCH] avail idx update optimizations

2016-04-22 Thread Huawei Xie
eliminate unnecessary cache to cache transfer between virtio and vhost core --- drivers/net/virtio/virtqueue.h | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/net/virtio/virtqueue.h b/drivers/net/virtio/virtqueue.h index 4e9239e..8c46a83 100644 ---

[dpdk-dev] [PATCH 1/6] i40e: fix problematic dereference

2016-04-22 Thread Zhang, Helin
> -Original Message- > From: Stephen Hemminger [mailto:stephen at networkplumber.org] > Sent: Friday, April 22, 2016 12:10 AM > To: Zhang, Helin > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH 1/6] i40e: fix problematic dereference > > On Thu, 21 Apr 2016 11:42:52 +0800 > Helin

[dpdk-dev] PKT_RX_VLAN_PKT when VLAN stripping is disabled

2016-04-22 Thread John Daley (johndale)
Hi, When VLAN stripping is disabled, X710 and 82599ES act differently for me in this case when receiving VLAN tagged packets. On 82599ES the flag is set but on X710 the flag not set. Do I maybe have old X710 firmware? Or is it not set for X710 on purpose in this case and instead the flag is