Re: [dpdk-dev] [PATCH 3/3] net/i40e: implement device reset on port

2017-03-13 Thread Zhao1, Wei
Hi,Ferruh > -Original Message- > From: Yigit, Ferruh > Sent: Wednesday, March 8, 2017 7:21 PM > To: Zhao1, Wei ; dev@dpdk.org > Cc: Lu, Wenzhuo > Subject: Re: [dpdk-dev] [PATCH 3/3] net/i40e: implement device reset on > port > > On 3/3/2017 4:56 AM, Wei Zhao wrote: > > Implement the devi

Re: [dpdk-dev] [PATCH] net/virtio-user: fix overflow

2017-03-13 Thread Yuanhan Liu
On Mon, Mar 13, 2017 at 03:09:15PM +, Wenfeng Liu wrote: > This commit fixes an array overflow when number of queues is higher than 8. Firstly, this commit log could be a bit more informative, to something like: virtio-user limits the qeueue number to 8 but provides no limit check aga

Re: [dpdk-dev] [PATCHv8 19/46] pool/dpaa2: add DPAA2 hardware offloaded mempool

2017-03-13 Thread Hemant Agrawal
On 3/9/2017 11:27 AM, Hemant Agrawal wrote: On 3/8/2017 9:09 PM, Thomas Monjalon wrote: 2017-03-08 18:22, Hemant Agrawal: On Fri, 3 Mar 2017 18:16:36 +0530, Hemant Agrawal wrote: I think the current mempool handlers should be moved first in a separate patch. Yes it should have been done earl

Re: [dpdk-dev] [PATCH] net/virtio-user: fix tapfds close issue

2017-03-13 Thread Yuanhan Liu
On Mon, Mar 13, 2017 at 09:33:21AM +, Wenfeng Liu wrote: > The valid tap file descriptor range should be equal or greater > than zero instead of non-zero > > Fixes: e3b434818bbb ("net/virtio-user: support kernel vhost") > Cc: sta...@dpdk.org Applied to dpdk-next-virtio. Thanks. --y

Re: [dpdk-dev] [PATCH] vhost: add back support for concurrent enqueue

2017-03-13 Thread Yuanhan Liu
On Thu, Mar 09, 2017 at 02:46:27PM +0100, Thomas Monjalon wrote: > 2016-08-23 15:42, Rich Lane: > > On Mon, Aug 22, 2016 at 7:16 AM, Yuanhan Liu > > wrote: > > > On Thu, Aug 18, 2016 at 11:27:06AM -0700, Rich Lane wrote: > > > > I could also add back concurrent enqueue support for mergeable RX, bu

Re: [dpdk-dev] i40e SR-IOV with Linux PF

2017-03-13 Thread Wu, Jingjing
> -Original Message- > From: Thomas Monjalon [mailto:thomas.monja...@6wind.com] > Sent: Monday, March 13, 2017 7:35 AM > To: Zhang, Helin ; Wu, Jingjing > ; Chen, Jing D > Cc: Yigit, Ferruh ; vincent.jar...@6wind.com; > dev@dpdk.org > Subject: Re: i40e SR-IOV with Linux PF > > +Cc dev@d

Re: [dpdk-dev] [PATCH] examples/ip_fragmentation: fix check of packet type

2017-03-13 Thread Dai, Wei
> -Original Message- > From: Ananyev, Konstantin > Sent: Tuesday, March 14, 2017 6:13 AM > To: Dai, Wei ; dev@dpdk.org > Cc: sta...@dpdk.org > Subject: RE: [PATCH] examples/ip_fragmentation: fix check of packet type > > > > > > Hi Wei, > > > > > > > > > > > Hi, Konstantin > > > > I see y

[dpdk-dev] [PATCH] net/vmxnet3: fix queue size changes

2017-03-13 Thread Charles (Chas) Williams
If the user reconfigures the queues size, then the previosly allocated memzone may potentially be too small. Instead, always free the old memzone and allocate a new one. Fixes: dfaff37fc46d ("vmxnet3: import new vmxnet3 poll mode driver implementation") Signed-off-by: Chas Williams --- driver

Re: [dpdk-dev] [PATCH] examples/ip_fragmentation: fix check of packet type

2017-03-13 Thread Ananyev, Konstantin
> > Hi Wei, > > > > > > > > Hi, Konstantin > > > I see your point. > > > I think your method can work. > > > But I think your method is a bit complex. As you method need to add more > > codes. > > > Anyway this is a simple bug. > > > How do you think now ? > > > > I still think it is better for a

Re: [dpdk-dev] [PATCH v3 16/16] doc: adds information related to the AVP PMD

2017-03-13 Thread Legacy, Allain
Vincent, Perhaps you can help me understand why the performance or functionality of AVP vs. Virtio is relevant to the decision of accepting this driver. There are many drivers in the DPDK; most of which provide the same functionality at comparable performance rates. AVP is just another such d

[dpdk-dev] [PATCH v4 17/17] doc: adds information related to the AVP PMD

2017-03-13 Thread Allain Legacy
Updates the documentation and feature lists for the AVP PMD device. Signed-off-by: Allain Legacy Signed-off-by: Matt Peters Acked-by: John McNamara --- MAINTAINERS| 1 + doc/guides/nics/avp.rst| 112 + doc/guides/nic

[dpdk-dev] [PATCH v4 16/17] net/avp: migration interrupt handling

2017-03-13 Thread Allain Legacy
This commit introduces changes required to support VM live-migration. This is done by registering and responding to interrupts coming from the host to signal that the memory is about to be made invalid and replaced with a new memory zone on the destination compute node. Enabling and disabling of

[dpdk-dev] [PATCH v4 14/17] net/avp: device promiscuous functions

2017-03-13 Thread Allain Legacy
Adds support for setting and clearing promiscuous mode on an AVP device. When enabled the _mac_filter function will allow packets destined to any MAC address to be processed by the receive functions. Signed-off-by: Allain Legacy Signed-off-by: Matt Peters --- drivers/net/avp/avp_ethdev.c | 28 +

[dpdk-dev] [PATCH v4 15/17] net/avp: device start and stop operations

2017-03-13 Thread Allain Legacy
Adds support for device start and stop functions. This allows an application to control the administrative state of an AVP device. Stopping the device will notify the host application to stop sending packets on that device's receive queues. Signed-off-by: Allain Legacy Signed-off-by: Matt Peter

[dpdk-dev] [PATCH v4 13/17] net/avp: device statistics operations

2017-03-13 Thread Allain Legacy
Adds device functions to query and reset statistics. Signed-off-by: Allain Legacy --- drivers/net/avp/avp_ethdev.c | 67 1 file changed, 67 insertions(+) diff --git a/drivers/net/avp/avp_ethdev.c b/drivers/net/avp/avp_ethdev.c index 78018f5..c5039e2

[dpdk-dev] [PATCH v4 10/17] net/avp: queue setup and release

2017-03-13 Thread Allain Legacy
Adds queue management operations so that an appliation can setup and release the transmit and receive queues. Signed-off-by: Allain Legacy Signed-off-by: Matt Peters --- drivers/net/avp/avp_ethdev.c | 180 ++- 1 file changed, 179 insertions(+), 1 deletion

[dpdk-dev] [PATCH v4 11/17] net/avp: packet receive functions

2017-03-13 Thread Allain Legacy
Adds function required for receiving packets from the host application via AVP device queues. Both the simple and scattered functions are supported. Signed-off-by: Allain Legacy Signed-off-by: Matt Peters --- drivers/net/avp/Makefile | 1 + drivers/net/avp/avp_ethdev.c | 451

[dpdk-dev] [PATCH v4 12/17] net/avp: packet transmit functions

2017-03-13 Thread Allain Legacy
Adds support for packet transmit functions so that an application can send packets to the host application via an AVP device queue. Both the simple and scattered functions are supported. Signed-off-by: Allain Legacy Signed-off-by: Matt Peters --- drivers/net/avp/avp_ethdev.c | 335

[dpdk-dev] [PATCH v4 09/17] net/avp: device configuration

2017-03-13 Thread Allain Legacy
Adds support for "dev_configure" operations to allow an application to configure the device. Signed-off-by: Allain Legacy Signed-off-by: Matt Peters --- drivers/net/avp/avp_ethdev.c | 240 +++ 1 file changed, 240 insertions(+) diff --git a/drivers/net/av

[dpdk-dev] [PATCH v4 08/17] net/avp: device initialization

2017-03-13 Thread Allain Legacy
Adds support for initialization newly probed AVP PCI devices. Initial queue translations are setup in preparation for device configuration. Signed-off-by: Allain Legacy Signed-off-by: Matt Peters --- drivers/net/avp/avp_ethdev.c | 299 +++ 1 file changed

[dpdk-dev] [PATCH v4 07/17] net/avp: driver registration

2017-03-13 Thread Allain Legacy
Adds the initial framework for registering the driver against the support PCI device identifiers. Signed-off-by: Allain Legacy Signed-off-by: Matt Peters --- config/common_linuxapp | 1 + config/defconfig_i686-native-linuxapp-gcc| 5 + config/defconfig_i686-native-

[dpdk-dev] [PATCH v4 06/17] drivers/net: adds driver makefiles for AVP PMD

2017-03-13 Thread Allain Legacy
Adds a default Makefile to the driver directory but does not include any source files. Signed-off-by: Allain Legacy Signed-off-by: Matt Peters --- drivers/net/Makefile | 1 + drivers/net/avp/Makefile | 52 2 files changed, 53 insertions(+)

[dpdk-dev] [PATCH v4 04/17] net/avp: add PMD version map file

2017-03-13 Thread Allain Legacy
Adds a default ABI version file for the AVP PMD. Signed-off-by: Allain Legacy Signed-off-by: Matt Peters --- drivers/net/avp/rte_pmd_avp_version.map | 4 1 file changed, 4 insertions(+) create mode 100644 drivers/net/avp/rte_pmd_avp_version.map diff --git a/drivers/net/avp/rte_pmd_avp_ve

[dpdk-dev] [PATCH v4 05/17] net/avp: debug log macros

2017-03-13 Thread Allain Legacy
Adds a header file with log macros for the AVP PMD Signed-off-by: Allain Legacy Signed-off-by: Matt Peters --- config/common_base | 4 drivers/net/avp/avp_logs.h | 59 ++ 2 files changed, 63 insertions(+) create mode 100644 drivers/net/

[dpdk-dev] [PATCH v4 01/17] config: adds attributes for the AVP PMD

2017-03-13 Thread Allain Legacy
Updates the common base configuration file to include a top level config attribute for the AVP PMD. Signed-off-by: Allain Legacy Signed-off-by: Matt Peters --- config/common_base | 5 + 1 file changed, 5 insertions(+) diff --git a/config/common_base b/config/common_base index aeee13e..912b

[dpdk-dev] [PATCH v4 03/17] maintainers: claim responsibility for AVP PMD

2017-03-13 Thread Allain Legacy
Updating the maintainers file to claim the AVP PMD driver on behalf of Wind River Systems, Inc. Signed-off-by: Allain Legacy Signed-off-by: Matt Peters --- MAINTAINERS | 5 + 1 file changed, 5 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 5030c1c..fef23a0 100644 --- a/MAINTAIN

[dpdk-dev] [PATCH v4 02/17] net/avp: public header files

2017-03-13 Thread Allain Legacy
Adds public/exported header files for the AVP PMD. The AVP device is a shared memory based device. The structures and constants that define the method of operation of the device must be visible by both the PMD and the host DPDK application. They must not change without proper version controls an

[dpdk-dev] [PATCH v4 00/17] Wind River Systems AVP PMD

2017-03-13 Thread Allain Legacy
This patch series submits an initial version of the AVP PMD from Wind River Systems. The series includes shared header files, driver implementation, and changes to documentation files in support of this new driver. The AVP driver is a shared memory based device. It is intended to be used as a PM

Re: [dpdk-dev] [PATCH v4 00/10] Rework vdev probing to use rte_bus infrastructure

2017-03-13 Thread Thomas Monjalon
2017-03-06 11:56, Jan Blunck: > Jan Blunck (10): > eal: probe legacy PCI devices before other bus devices > eal: probe new virtual bus after other bus devices > eal: move virtual device probing into a bus > eal: remove unused rte_eal_dev_init() > eal: Refactor vdev driver probe/remove >

Re: [dpdk-dev] [PATCH v4 09/10] eal: make virtual bus use rte_vdev_device

2017-03-13 Thread Thomas Monjalon
2017-03-06 11:56, Jan Blunck: > This allows the virtual bus to be rescanned and probed by tracking the > creation of rte_vdev_device. > > Signed-off-by: Jan Blunck > Tested-by: Ferruh Yigit > Acked-by: Shreyansh Jain There are a lot of strcmp in this patch. Maybe it should be abstracted in a "

Re: [dpdk-dev] [PATCH v4 03/10] eal: move virtual device probing into a bus

2017-03-13 Thread Thomas Monjalon
2017-03-06 11:56, Jan Blunck: > +static int > +vdev_scan(void) > +{ > + /* for virtual devices we don't need to scan anything */ > + return 0; > +} The vdev_scan could save the devargs. The vdev args are the equivalent of PCI BDF.

Re: [dpdk-dev] [PATCH v4 02/10] eal: probe new virtual bus after other bus devices

2017-03-13 Thread Thomas Monjalon
2017-03-06 11:56, Jan Blunck: > @@ -86,9 +86,14 @@ int > rte_bus_probe(void) > { > int ret; > - struct rte_bus *bus; > + struct rte_bus *bus, *vbus = NULL; > > TAILQ_FOREACH(bus, &rte_bus_list, next) { > + if (!strcmp(bus->name, "virtual")) { > +

Re: [dpdk-dev] [PATCH] net/vmxnet3: fix broken debug build

2017-03-13 Thread Shrikrishna Khare
On Mon, 13 Mar 2017, Andrew Rybchenko wrote: > Fixes: 4dd452523000 ("net/vmxnet3: add receive data ring support") > > Signed-off-by: Andrew Rybchenko Thank you for catching and fixing this. Acked-by: Shrikrishna Khare > --- > drivers/net/vmxnet3/vmxnet3_ethdev.c | 4 ++-- > 1 file changed

Re: [dpdk-dev] i40e SR-IOV with Linux PF

2017-03-13 Thread Thomas Monjalon
+Cc dev@dpdk.org 2017-03-13 15:29, Thomas Monjalon: > Hi i40e developers, > > Referring to the VFD discussion, I thought basic behaviours were the > same regardless of the PF driver: > http://dpdk.org/ml/archives/dev/2016-December/053056.html > " > > In the meanwhile, we have some test models ong

[dpdk-dev] [PATCH v3] devtools: add tags and cscope index file generation support

2017-03-13 Thread Jerin Jacob
This script generates cscope, gtags, and tags index files based on EAL environment(architecture and OS(linux/bsd)). If EAL environment(config) is not specified, the script generates tag files based on available source code. Usage: build-tags.sh [-h] [-v] tags|cscope|gtags|etags [config] example

Re: [dpdk-dev] [PATCH 04/13] net/sfc: factor out libefx-based Rx datapath

2017-03-13 Thread Andrew Rybchenko
On 03/05/2017 12:05 AM, Ferruh Yigit wrote: On 3/2/2017 7:07 AM, Andrew Rybchenko wrote: Split control and datapath to make datapath substitutable and possibly reusable with alternative control path. Does it make sense to document how alternative control path can be used? Possibility to use a

[dpdk-dev] [PATCH] net/vmxnet3: fix broken debug build

2017-03-13 Thread Andrew Rybchenko
Fixes: 4dd452523000 ("net/vmxnet3: add receive data ring support") Signed-off-by: Andrew Rybchenko --- drivers/net/vmxnet3/vmxnet3_ethdev.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/vmxnet3/vmxnet3_ethdev.c b/drivers/net/vmxnet3/vmxnet3_ethdev.c index 0

Re: [dpdk-dev] [PATCH v9 09/18] lib: add symbol versioning to distributor

2017-03-13 Thread Hunt, David
On 13/3/2017 11:01 AM, Van Haaren, Harry wrote: From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Hunt, David Subject: Re: [dpdk-dev] [PATCH v9 09/18] lib: add symbol versioning to distributor On 10/3/2017 4:22 PM, Bruce Richardson wrote: On Mon, Mar 06, 2017 at 09:10:24AM +, David Hun

Re: [dpdk-dev] [PATCH v9 09/18] lib: add symbol versioning to distributor

2017-03-13 Thread Van Haaren, Harry
> From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Hunt, David > Subject: Re: [dpdk-dev] [PATCH v9 09/18] lib: add symbol versioning to > distributor > > On 10/3/2017 4:22 PM, Bruce Richardson wrote: > > On Mon, Mar 06, 2017 at 09:10:24AM +, David Hunt wrote: > >> Also bumped up the ABI v

Re: [dpdk-dev] [PATCH v9 09/18] lib: add symbol versioning to distributor

2017-03-13 Thread Hunt, David
On 10/3/2017 4:22 PM, Bruce Richardson wrote: On Mon, Mar 06, 2017 at 09:10:24AM +, David Hunt wrote: Also bumped up the ABI version number in the Makefile Signed-off-by: David Hunt --- lib/librte_distributor/Makefile| 2 +- lib/librte_distributor/rte_distributor.c

[dpdk-dev] [PATCH v2] app/testpmd: add default MAC set cmd

2017-03-13 Thread Pascal Mazon
Signed-off-by: Pascal Mazon --- app/test-pmd/cmdline.c | 12 +--- doc/guides/testpmd_app_ug/testpmd_funcs.rst | 7 +++ 2 files changed, 16 insertions(+), 3 deletions(-) diff --git a/app/test-pmd/cmdline.c b/app/test-pmd/cmdline.c index 43fc6366f279..4bbefa43966e

Re: [dpdk-dev] [PATCH v9 09/18] lib: add symbol versioning to distributor

2017-03-13 Thread Hunt, David
-Original Message- From: Richardson, Bruce Sent: Friday, 10 March, 2017 4:22 PM To: Hunt, David Cc: dev@dpdk.org Subject: Re: [PATCH v9 09/18] lib: add symbol versioning to distributor On Mon, Mar 06, 2017 at 09:10:24AM +, David Hunt wrote: > Also bumped up the ABI version number i

[dpdk-dev] [PATCH] net/virtio-user: fix tapfds close issue

2017-03-13 Thread Wenfeng Liu
The valid tap file descriptor range should be equal or greater than zero instead of non-zero Fixes: e3b434818bbb ("net/virtio-user: support kernel vhost") Signed-off-by: Wenfeng Liu --- drivers/net/virtio/virtio_user/vhost_kernel.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --g

[dpdk-dev] [PATCH] net/pcap: fix use-after-free

2017-03-13 Thread Johan Samuelsson
Signed-off-by: Johan Samuelsson --- drivers/net/pcap/rte_eth_pcap.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/pcap/rte_eth_pcap.c b/drivers/net/pcap/rte_eth_pcap.c index 7f0b650..075e3be 100644 --- a/drivers/net/pcap/rte_eth_pcap.c +++ b/drivers/net/pcap/rte_

[dpdk-dev] [PATCH v2 3/3] eal: remove references to execinfo.h for musl

2017-03-13 Thread Wei Dai
execinfo.h is not supported by musl now. need to remove references to execinfo.h to build DPDK with musl. musl is an implementation of the userspace portion of the standard library functionality described in the ISO C and POSIX standards, plus common extensions. Get more details about musl from h

[dpdk-dev] [PATCH v2 0/3] add a macro to enable support of backtrace

2017-03-13 Thread Wei Dai
Some LIBC implementation like musl doesn't support backtrace( ) and backtrace_symbols declared in execinfo.h. Currently some DPDK customers fail to build PDDK with musl. In order to build DPDK with musl, there is a need to reomve references toexecinfo.h. Add a configuration parameter in config/com

[dpdk-dev] [PATCH v2 2/3] config: add a marco to enable backtrace or not

2017-03-13 Thread Wei Dai
When building DPDK with musl, there is need not to generate the MACRO named RTE_EAL_ENABLE_BACKTRACE in rte_config.h to remove some references to execinfo.h which is not supported by musl now. This also applies to some other LIBC implementation which doesn't support backtrace( ) and backtrace_symbo

[dpdk-dev] [PATCH v2 1/3] examples/performance-thread: remove reference to execinfo.h

2017-03-13 Thread Wei Dai
There is no function to refer any part of execinfo.h, so remove the reference to it. And there is no this file in musl. So need to remove it to support musl. The musl is an alternative LIBC to GLIBC and provides the standard C/POSIX library and extensions. The musl can be got from http://www.musl-

Re: [dpdk-dev] [PATCH v3 1/2] librte_net: add crc compute APIs

2017-03-13 Thread Singh, Jasvinder
Hi Konstantin, > -Original Message- > From: Ananyev, Konstantin > Sent: Monday, March 13, 2017 3:06 AM > To: Singh, Jasvinder ; dev@dpdk.org > Cc: Doherty, Declan ; De Lara Guarch, Pablo > > Subject: RE: [dpdk-dev] [PATCH v3 1/2] librte_net: add crc compute APIs > > Hi Jasvinder, > > >

Re: [dpdk-dev] [PATCH v3] eventdev: fix links map initialization for SW PMD

2017-03-13 Thread Jerin Jacob
On Tue, Mar 07, 2017 at 10:25:00AM -0600, Gage Eads wrote: > This patch initializes the links_map array entries to > EVENT_QUEUE_SERVICE_PRIORITY_INVALID, as expected by > rte_event_port_links_get(). This is necessary for the sw eventdev PMD, > which does not initialize links_map when rte_event_por

Re: [dpdk-dev] [PATCH v2] eventdev: event device to contain rte device holder

2017-03-13 Thread Jerin Jacob
On Fri, Mar 03, 2017 at 09:03:02PM +0530, Nipun Gupta wrote: > rte_device is a generic device which is available to the applications > and EAL. This patch replaces rte_pci_device in 'struct rte_eventdev' > and in 'struct rte_event_dev_info' with common rte_device. > > Signed-off-by: Nipun Gupta >

Re: [dpdk-dev] [PATCH] eventdev: improve API docs for timeout ticks

2017-03-13 Thread Jerin Jacob
On Wed, Mar 08, 2017 at 10:35:34AM +, Harry van Haaren wrote: > Improve the documentation of the return values of the > rte_event_dequeue_timeout_ticks() function, adding a > -ENOTSUP value for eventdevs that do not support waiting. > > Signed-off-by: Harry van Haaren > Acked-by: Jerin Jacob

Re: [dpdk-dev] [PATCH v4 01/17] eventdev: increase size of enq deq conf variables

2017-03-13 Thread Jerin Jacob
On Fri, Mar 10, 2017 at 07:43:16PM +, Harry van Haaren wrote: > Large port enqueue sizes were not supported as the value > it was stored in was a uint8_t. Using uint8_ts to save > space in config apis makes no sense - increasing the 3 > instances of uint8_t enqueue / dequeue depths to more > ap

Re: [dpdk-dev] [PATCH 3/3] eal: remove references to execinfo.h for musl

2017-03-13 Thread Dai, Wei
> -Original Message- > From: jblu...@gmail.com [mailto:jblu...@gmail.com] On Behalf Of Jan Blunck > Sent: Friday, March 10, 2017 10:50 PM > To: Thomas Monjalon > Cc: Dai, Wei ; dev ; Mcnamara, John > ; david.march...@intel.com > Subject: Re: [dpdk-dev] [PATCH 3/3] eal: remove references to

[dpdk-dev] [PATCH v2] net/thunderx: check data offset alignment requirement

2017-03-13 Thread Jerin Jacob
nicvf HW expects the DMA address of the packet data to be aligned with cache line size. Packet data offset is a function of struct mbuf size, mbuf private size and headroom. mbuf private size can be changed from the application in pool creation, this check detects HW alignment requirement constrai

Re: [dpdk-dev] [PATCH] examples/ip_fragmentation: fix check of packet type

2017-03-13 Thread Dai, Wei
> -Original Message- > From: Ananyev, Konstantin > Sent: Monday, March 13, 2017 3:02 PM > To: Dai, Wei ; dev@dpdk.org > Cc: sta...@dpdk.org > Subject: RE: [PATCH] examples/ip_fragmentation: fix check of packet type > > Hi Wei, > > > > > Hi, Konstantin > > I see your point. > > I think you

[dpdk-dev] [PATCH] net/virtio-user: fix overflow

2017-03-13 Thread Wenfeng Liu
This commit fixes an array overflow when number of queues is higher than 8. Fixes: 37a7eb2ae816 ("net/virtio-user: add device emulation layer") Signed-off-by: Wenfeng Liu --- drivers/net/virtio/virtio_pci.h | 3 ++- drivers/net/virtio/virtio_user/virtio_user_dev.c | 2 +- drive

Re: [dpdk-dev] [PATCH] examples/ip_fragmentation: fix check of packet type

2017-03-13 Thread Ananyev, Konstantin
Hi Wei, > > Hi, Konstantin > I see your point. > I think your method can work. > But I think your method is a bit complex. As you method need to add more > codes. > Anyway this is a simple bug. > How do you think now ? I still think it is better for all apps to handle this issue in a uniform wa