[dpdk-dev] Running 2 process on the same machine

2016-11-07 Thread Wiles, Keith
> On Nov 7, 2016, at 7:28 AM, Keren Hochman > wrote: > > Hi, > I need to run 2 process that uses dpdk on the same machine. One uses dpdk > drivers, and the other just read from a pcap file. If I disable hugepages > in the second process rte_mempool_create fails. What is the correct way to >

[dpdk-dev] [dpdk-announce] release candidate 16.11-rc3

2016-11-07 Thread Thomas Monjalon
A new DPDK release candidate is ready for testing: http://dpdk.org/browse/dpdk/tag/?id=v16.11-rc3 It will be hopefully the last release candidate for 16.11. As it is not so small, please test it very well in order to secure the release planned to be out at the end of this week. Thank you

[dpdk-dev] [PATCH] doc: add limitation for L3fwd-power app

2016-11-07 Thread Thomas Monjalon
> > L3fwd-power app needs vector mode to be disabled in order to work > > properly. The app used to work previously, because it was using Rx scalar > > function, but now it uses vector function. > > > > Vector mode needs to be disabled to make the app works, which has been > > documented in

[dpdk-dev] [PATCH v2] doc/guides: add more info about VT-d/iommu settings

2016-11-07 Thread Thomas Monjalon
> > Add more information about VT-d/iommu settings for QAT PMD. > > Remove limitation indicating QAT driver is not performance tuned. > > > > Signed-off-by: Fiona Trahe > Acked-by: Arek Kusztal Wrapped lines and applied.

[dpdk-dev] [PATCH] doc: remove Intel reference from multi-process support guide

2016-11-07 Thread Thomas Monjalon
> > multi-process support has been verified on non IA such as ARMv8. > > > > Signed-off-by: Jerin Jacob > > Acked-by: John McNamara Applied, thanks

[dpdk-dev] [PATCH] doc: fix typos

2016-11-07 Thread Thomas Monjalon
> > Fixes typos present in the documentation and code comments. > > > > Signed-off-by: Alain Leon > > Acked-by: John McNamara Applied, thanks

[dpdk-dev] [PATCH v3] examples/ipsec-secgw: fix pointer to local outside scope

2016-11-07 Thread Thomas Monjalon
2016-11-07 17:31, Ferruh Yigit: > On 11/7/2016 5:25 PM, Fan Zhang wrote: > > Coverity issue: 137871 > > Fixes: 0d547ed03717 ("examples/ipsec-secgw: support configuration file") > > > > Signed-off-by: Fan Zhang > > Although checkpatch will complain about long lines, I believe it is > better not

[dpdk-dev] [PATCH v2] examples/ipsec-secgw: fix copy into fixed size buffer issue

2016-11-07 Thread Thomas Monjalon
2016-11-07 15:59, Ferruh Yigit: > On 11/7/2016 2:21 PM, Fan Zhang wrote: > > Fixes: 0d547ed0 ("examples/ipsec-secgw: support configuration > > file") > > Coverity issue: 137875 > > > > Signed-off-by: Fan Zhang > > Acked-by: Ferruh Yigit Applied

[dpdk-dev] [PATCH] ethdev: fix statistics description

2016-11-07 Thread Thomas Monjalon
2016-11-02 10:21, Morten Br?rup: > > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > > > 2016-08-26 18:08, Wei Dai: > > > > /** > > > > * A structure used to retrieve statistics for an Ethernet port. > > > > + * Not all statistics fields in struct rte_eth_stats are supported > > >

[dpdk-dev] [PATCH] lib/ip_frag: fix IP reassembly not working issue

2016-11-07 Thread Thomas Monjalon
2016-11-06 12:16, Wenzhuo Lu: > After changing pkt[0] to pkt[], the example IP reassembly is not > working. > It's weird because this change is fine. There should be no > difference between them. > As a workaround, revert this change. > > Fixes: 347a1e037fd3 (lib: use C99 syntax for zero-size

[dpdk-dev] [PATCH] net/vmxnet3: fix mbuf release on reset/stop

2016-11-07 Thread Thomas Monjalon
2016-10-26 10:45, Yong Wang: > During device reset/stop, vmxnet3 releases all mbufs in tx and > rx cmd ring. For rx, we should go over all ring descriptors and > free using rte_pktmbuf_free_seg() instead of rte_pktmbuf_free() > as the metadata of the mbuf might not be properly initialized >

[dpdk-dev] [PATCH] net/qede: fix gcc compiler option checks

2016-11-07 Thread Thomas Monjalon
2016-10-27 23:37, Rasesh Mody: > From: Rasesh Mody > > Using GCC_VERSION to check gcc version and decide whether to include > that compiler option. > > Fixes: ec94dbc57362 ("qede: add base driver") No need for the above line. It is fixing the commit below (which fixes above one). > Fixes:

[dpdk-dev] [PATCH] net/qede: fix gcc compiler option checks

2016-11-07 Thread Thomas Monjalon
2016-10-28 22:49, Mody, Rasesh: > > From: Stephen Hemminger > > > ifeq ($(CONFIG_RTE_TOOLCHAIN_GCC),y) > > > -ifeq ($(shell gcc -Wno-unused-but-set-variable -Werror -E - < /dev/null > > > > /dev/null 2>&1; echo $$?),0) > > > +ifeq ($(shell test $(GCC_VERSION) -ge 44 && echo 1), 1) > > >

[dpdk-dev] [PATCH v3] net/qede: fix advertising link speed capability

2016-11-07 Thread Thomas Monjalon
2016-10-31 11:35, Rasesh Mody: > From: Harish Patil > > Fix to advertise device's link speed capability based on NVM > port configuration instead of returning driver supported speeds. > > Fixes: 95e67b479506 ("net/qede: add 100G link speed capability") > > Signed-off-by: Harish Patil [...] >

[dpdk-dev] [PATCH 1/3 v3] bnxt: use appropriate data type in bnxt_alloc_vnic_attributes

2016-11-07 Thread Thomas Monjalon
2016-11-07 09:12, Ajit Khaparde: > Prevent the arithmetic in bnxt_alloc_vnic_attributes from causing > any unintentional havoc because of the usage of a signed variable. > > Coverity: 137874 > > Signed-off-by: Ajit Khaparde Series applied, thanks As a regular contributor, please follow the

[dpdk-dev] [PATCH 0/2] update mlx5 release note and guide

2016-11-07 Thread Thomas Monjalon
> > Nelio Laranjeiro (2): > > doc: update mlx5 dependencies > > doc: add mlx5 release notes > > Acked-by: Adrien Mazarguil Applied with 2/2 v2, thanks

[dpdk-dev] [PATCH] doc: fix mlx5 features overview

2016-11-07 Thread Thomas Monjalon
> > Fixes: 75ef62a94301 ("net/mlx5: fix link speed capability information") > > Fixes: 188408719888 ("net/mlx5: fix support for newer link speeds") > > > > Signed-off-by: Nelio Laranjeiro > > Acked-by: John McNamara Applied, thanks

[dpdk-dev] [PATCH 0/3] fix Rx checksum offloads

2016-11-07 Thread Thomas Monjalon
2016-11-02 16:57, Adrien Mazarguil: > On Wed, Nov 02, 2016 at 11:39:36AM +0100, Nelio Laranjeiro wrote: > > Fill correctly the Mbuf Rx offloads. > > > > Nelio Laranjeiro (3): > > net/mlx5: fix Rx checksum macros > > net/mlx5: define explicit fields for Rx offloads > > net/mlx: fix support

[dpdk-dev] how to search this mailing list? is gmane link archive broken?

2016-11-07 Thread Thomas Monjalon
2016-11-07 17:12, Montorsi, Francesco: > Hi all, > if this was already raised, sorry for that. > I noticed that the gmane archive for this mailing list is not working anymore: > > http://news.gmane.org/gmane.comp.networking.dpdk.devel > > reports "Page not found". Also I noticed that the

[dpdk-dev] how to search this mailing list? is gmane link archive broken?

2016-11-07 Thread Andriy Berestovskyy
Hey, You can try "site:" Google search operator, i.e. try to google: site:dpdk.org/ml/archives/dev/ Regards, Andriy On Mon, Nov 7, 2016 at 6:12 PM, Montorsi, Francesco wrote: > Hi all, > if this was already raised, sorry for that. > I noticed that the gmane archive for this mailing list is

[dpdk-dev] [PATCH] net/mlx5: fix handling of small mbuf sizes

2016-11-07 Thread Thomas Monjalon
2016-10-28 14:19, Adrien Mazarguil: > On Mon, Oct 24, 2016 at 11:10:59AM +0300, Raslan Darawsheh wrote: > > When mbufs are smaller than MRU, multi-segment support must be enabled to > > default set when not in promiscuous or allmulticast modes. > > > > Fixes: 9964b965ad69 ("net/mlx5: re-add Rx

[dpdk-dev] [PATCH] net/mlx5: fix wrong use of vector instruction

2016-11-07 Thread Thomas Monjalon
2016-11-02 16:56, Adrien Mazarguil: > On Tue, Nov 01, 2016 at 08:13:27AM +, Elad Persiko wrote: > > Constraint alignment was not respected in Tx. > > > > Fixes: 1d88ba171942 ("net/mlx5: refactor Tx data path") > > > > Signed-off-by: Elad Persiko > > Acked-by: Adrien Mazarguil "wrong use

[dpdk-dev] [PATCH] doc: update release notes with enic PMD changes

2016-11-07 Thread Thomas Monjalon
> > Signed-off-by: John Daley > > Acked-by: John McNamara Applied with few fixes

[dpdk-dev] [PATCH] net/enic: fix max packet length check

2016-11-07 Thread Thomas Monjalon
2016-11-01 11:08, John Daley: > When the device was configured with an explicit maximum packet length, > it would fail if the value was greater than MTU configured in CIMC/UCSM > (plus L2 header length). It should have been compared against maximum > allowed by the device. > > Fixes: bb34ffb848a0

[dpdk-dev] [PATCH 0/2] net/ena: check for free descriptors

2016-11-07 Thread Thomas Monjalon
2016-10-29 03:06, Jakub Palider: > In some configurations there is mismatch between declared and actual > descriptor count which under heavy load may lead to resource shortage. > > The first patch unifies the way head and tail indexes are handled and > is crucial for compactness and succeeding

[dpdk-dev] [PATCH v4 2/2] net/i40e: fix VF bonded device link down

2016-11-07 Thread Thomas Monjalon
2016-11-04 17:10, Qiming Yang: > If VF device is used as slave of a bond device, it will be polled > periodically through alarm. Interrupt is involved here. And then > VF will send I40E_VIRTCHNL_OP_GET_LINK_STAT message to > PF to query the status. The response is handled by interrupt > callback.

[dpdk-dev] [PATCH RESEND v2] net/i40e: fix floating VEB issue

2016-11-07 Thread Thomas Monjalon
2016-11-04 19:08, Yuanhan Liu: > From: Beilei Xing > > Turning off S-TAG identification will impact floating VEB, > VFs can't communicate with each other. > This patch fixes this issue by judging whether floating > VEB is enabled, S-TAG identification will be turned off > only when floating VEB

[dpdk-dev] [PATCH] net/i40e: fix dcb configure failure

2016-11-07 Thread Thomas Monjalon
> > Removing stopping LLDP in firmware is a workaround for a known errata which > > can cause Rx hang. But the changing will cause DCB configuration fails. > > That is > > because when LLDP is enabled, the return value of i40e_init_dcb is success. > > But > > following check just considered the

[dpdk-dev] [PATCH v3] examples/ipsec-secgw: fix pointer to local outside scope

2016-11-07 Thread Ferruh Yigit
On 11/7/2016 5:25 PM, Fan Zhang wrote: > Coverity issue: 137871 > Fixes: 0d547ed03717 ("examples/ipsec-secgw: support configuration file") > > Signed-off-by: Fan Zhang > --- Although checkpatch will complain about long lines, I believe it is better not to wrap log messages. Acked-by: Ferruh

[dpdk-dev] Running 2 process on the same machine

2016-11-07 Thread Keren Hochman
Hi, I need to run 2 process that uses dpdk on the same machine. One uses dpdk drivers, and the other just read from a pcap file. If I disable hugepages in the second process rte_mempool_create fails. What is the correct way to handle this? Thanks, Keren

[dpdk-dev] [PATCH v3] examples/ipsec-secgw: fix pointer to local outside scope

2016-11-07 Thread Fan Zhang
Coverity issue: 137871 Fixes: 0d547ed03717 ("examples/ipsec-secgw: support configuration file") Signed-off-by: Fan Zhang --- examples/ipsec-secgw/parser.c | 116 +++--- 1 file changed, 52 insertions(+), 64 deletions(-) diff --git

[dpdk-dev] [PATCH v2 10/10] net/virtio: fix multiple queue enabling

2016-11-07 Thread Yuanhan Liu
On Sat, Nov 05, 2016 at 05:41:05PM +0800, Yuanhan Liu wrote: > When queue number shrinks to 1 from X, the following code stops us > sending the multiple queue ctrl message: > > if (nb_queues > 1) { > if (virtio_set_multiple_queues(dev, nb_queues) != 0) >

[dpdk-dev] rte_ixgbevf_pmd not reporting dropped packets

2016-11-07 Thread Montorsi, Francesco
Hi all, I'm using DPDK inside the OS of a VM that is SR-IOV-accelerated. I noticed however that the "rte_ixgbevf_pmd" PMD does not report drops... I am sending packets at TX side at 14Mpps; at the RX side I'm using "testpmd" and it's reporting 'just' 12Mpps, but zero drops (also through

[dpdk-dev] how to search this mailing list? is gmane link archive broken?

2016-11-07 Thread Montorsi, Francesco
Hi all, if this was already raised, sorry for that. I noticed that the gmane archive for this mailing list is not working anymore: http://news.gmane.org/gmane.comp.networking.dpdk.devel reports "Page not found". Also I noticed that the gmane link on the dpdk.org website has been removed.

[dpdk-dev] [PATCH v9] app/testpmd: fix DCB configuration

2016-11-07 Thread Thomas Monjalon
> > Data Centre Bridge (DCB) configuration fails when SRIOV is enabled if nb_rxq > > or nb_txq are greater than nb_q_per_pool. > > > > The failure occurs during configuration of the ixgbe PMD when it is > > started, in > > the ixgbe_check_mq_mode function. > > > > Fixes: 2a977b891f99

[dpdk-dev] [PATCH v1 1/2] doc: update ixgbe guide

2016-11-07 Thread Thomas Monjalon
> > add information about new ixgbe PMD API. > > > > Signed-off-by: Bernard Iremonger > > Acked-by: John McNamara Applied without patch 2 and squashed with release notes

[dpdk-dev] [PATCH v1 2/2] doc: update poll mode driver guide

2016-11-07 Thread Thomas Monjalon
2016-10-14 17:35, Bernard Iremonger: > --- a/doc/guides/prog_guide/poll_mode_drv.rst > +++ b/doc/guides/prog_guide/poll_mode_drv.rst > @@ -356,3 +356,9 @@ Some additions in the metadata scheme are as follows: > An example where queue numbers are used is as follows: ``tx_q7_bytes`` which >

[dpdk-dev] [PATCH] ixgbe: fix wrong VF TX registers

2016-11-07 Thread Thomas Monjalon
2016-11-06 11:55, Wenzhuo Lu: > The VF TX registers are using PF's addresses by mistake, > correct them. > > Fixes: 0198848a47f5 ("ixgbe: add access to specific device info") > > Reported-by: Xuekun Hu > Signed-off-by: Wenzhuo Lu Applied with other similar patches squashed, thanks

[dpdk-dev] [PATCH v2] E1000: fix for forced speed/duplex config

2016-11-07 Thread Thomas Monjalon
> > From the code, it looks like, hw->mac.autoneg, variable is used to switch > > between calling either autoneg function or forcing speed/duplex function. > > But > > this variable is not modified in eth_em_start/eth_igb_start routines (it is > > always > > set to 1) even while forcing the link

[dpdk-dev] [PATCH 4/4] net/bonding: fix configuration of LACP slaves

2016-11-07 Thread Kulasek, TomaszX
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Robert Sanford > Sent: Monday, August 1, 2016 22:43 > To: dev at dpdk.org > Cc: Doherty, Declan ; De Lara Guarch, Pablo > ; olivier.matz at 6wind.com > Subject: [dpdk-dev] [PATCH 4/4] net/bonding: fix

[dpdk-dev] [PATCH 3/4] net/bonding: another fix to LACP mempool size

2016-11-07 Thread Kulasek, TomaszX
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Robert Sanford > Sent: Monday, August 1, 2016 22:43 > To: dev at dpdk.org > Cc: Doherty, Declan ; De Lara Guarch, Pablo > ; olivier.matz at 6wind.com > Subject: [dpdk-dev] [PATCH 3/4] net/bonding: another fix

[dpdk-dev] [PATCH 0/3] vhost: comments and doc update due to vhost-cuse removal

2016-11-07 Thread Thomas Monjalon
2016-11-02 11:14, Yuanhan Liu: > Here is a small patchset of updating vhost programming and sample guide > and comments, due to the removal of vhost-cuse. Applied, thanks

[dpdk-dev] [PATCH] net/bonding: only handle lacp slow packets

2016-11-07 Thread Kulasek, TomaszX
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of linhaifeng > Sent: Friday, November 4, 2016 04:30 > To: dev at dpdk.org > Cc: Yigit, Ferruh ; Doherty, Declan > > Subject: [dpdk-dev] [PATCH] net/bonding: only handle lacp slow packets > > From: Linhaifeng

[dpdk-dev] [PATCH v2] examples/ipsec-secgw: fix copy into fixed size buffer issue

2016-11-07 Thread Ferruh Yigit
On 11/7/2016 2:21 PM, Fan Zhang wrote: > Fixes: 0d547ed0 ("examples/ipsec-secgw: support configuration > file") > Coverity issue: 137875 > > Signed-off-by: Fan Zhang Acked-by: Ferruh Yigit

[dpdk-dev] [PATCH v2 00/10] net/virtio: fix queue reconfigure issue

2016-11-07 Thread Thomas Monjalon
2016-11-05 17:40, Yuanhan Liu: > This patchset fixes few issues related to virtio queue reconfigure: > increase or shrink the queue number. The major issue and the reason > behind is described with length details in patch 4 "net/virtio: allocate > queue at init stage". > > Those bugs can not be

[dpdk-dev] [PATCH v3 1/3] lib: add information metrics library

2016-11-07 Thread Pattan, Reshma
Hi, > -Original Message- > From: Pattan, Reshma > Sent: Friday, November 4, 2016 4:43 PM > To: Remy Horton > Cc: dev at dpdk.org > Subject: RE: [dpdk-dev] [PATCH v3 1/3] lib: add information metrics library > > Hi, > > Few comments below. > > > -Original Message- > > From: dev

[dpdk-dev] [PATCH v2 04/10] net/virtio: allocate queue at init stage

2016-11-07 Thread Thomas Monjalon
2016-11-05 17:40, Yuanhan Liu: > -int virtio_dev_queue_setup(struct rte_eth_dev *dev, > - int queue_type, > - uint16_t queue_idx, > - uint16_t vtpci_queue_idx, > - uint16_t nb_desc, > - unsigned int

[dpdk-dev] [PATCH v2 00/10] net/virtio: fix queue reconfigure issue

2016-11-07 Thread Yao, Lei A
Tested-by: Lei Yao - Apply patch to v16.11-rc2 - Compile: Pass - OS: Ubuntu16.04 4.4.0-45-generic - GCC: 5.4.0 Most of the basic Virtio related test cases are tested with this patch. No function issue found and no obvious performance drop. The following is the pass case list: TC1:?

[dpdk-dev] [PATCH] pci: fix one device probing

2016-11-07 Thread Thomas Monjalon
2016-11-07 14:26, David Marchand: > On Thu, Aug 4, 2016 at 1:50 PM, Igor Ryzhov wrote: > > The rte_eal_pci_probe_one function could return false positive result if > > no driver is found for the device. > > > > Signed-off-by: Igor Ryzhov > > Acked-by: David Marchand Applied, thanks

[dpdk-dev] [PATCH] pci: do not mark device as taken when probe fails

2016-11-07 Thread Thomas Monjalon
2016-11-07 10:03, David Marchand: > dev->driver should be set only if a driver did take the device. > > Signed-off-by: David Marchand Applied, thanks

[dpdk-dev] [PATCH] pci: fix one device probing

2016-11-07 Thread David Marchand
On Thu, Aug 4, 2016 at 1:50 PM, Igor Ryzhov wrote: > The rte_eal_pci_probe_one function could return false positive result if > no driver is found for the device. > > Signed-off-by: Igor Ryzhov > --- > lib/librte_eal/common/eal_common_pci.c | 2 +- > 1 file changed, 1 insertion(+), 1

[dpdk-dev] [PATCH v2] examples/ipsec-secgw: fix pointer to local outside scope

2016-11-07 Thread Fan Zhang
Coverity issue: 137871 Fixes: 0d547ed03717 ("examples/ipsec-secgw: support configuration file") Signed-off-by: Fan Zhang --- examples/ipsec-secgw/parser.c | 120 -- 1 file changed, 58 insertions(+), 62 deletions(-) diff --git

[dpdk-dev] [PATCH v2] examples/ipsec-secgw: fix copy into fixed size buffer issue

2016-11-07 Thread Fan Zhang
Fixes: 0d547ed0 ("examples/ipsec-secgw: support configuration file") Coverity issue: 137875 Signed-off-by: Fan Zhang --- examples/ipsec-secgw/sa.c | 11 +++ 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/examples/ipsec-secgw/sa.c b/examples/ipsec-secgw/sa.c index

[dpdk-dev] [PATCH v3] net/ring: remove unnecessary NULL check

2016-11-07 Thread Thomas Monjalon
2016-11-02 14:05, Ferruh Yigit: > On 11/2/2016 1:46 PM, Mauricio Vasquez B wrote: > > Coverity detected this as an issue because internals->data will never be > > NULL, > > then the check is not necessary. > > > > Fixes: d082c0395bf6 ("ring: fix memory leak when detaching") > > Coverity issue:

[dpdk-dev] [PATCH] doc: fix typos

2016-11-07 Thread Mcnamara, John
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Alain Leon > Sent: Monday, November 7, 2016 3:20 AM > To: dev at dpdk.org > Cc: Alain Leon > Subject: [dpdk-dev] [PATCH] doc: fix typos > > Fixes typos present in the documentation and code comments. > >

[dpdk-dev] [PATCH v4] latencystats: added new library for latency stats

2016-11-07 Thread Reshma Pattan
Library is designed to calculate latency stats and report them to the application when queried. Library measures minimum, average, maximum latencies and jitter in nano seconds. Current implementation supports global latency stats, i.e. per application stats. Added new field to mbuf struct to mark

[dpdk-dev] [PATCH v3 2/2] mempool: pktmbuf pool default fallback for mempool ops error

2016-11-07 Thread Hemant Agrawal
Hi Olivier, > -Original Message- > From: Olivier Matz [mailto:olivier.matz at 6wind.com] > Sent: Friday, October 14, 2016 5:41 PM > > On 9/22/2016 6:42 PM, Hemant Agrawal wrote: > >> Hi Olivier > >> > >> On 9/19/2016 7:27 PM, Olivier Matz wrote: > >>> Hi Hemant, > >>> > >>> On 09/16/2016

[dpdk-dev] [PATCH] pci: do not mark device as taken when probe fails

2016-11-07 Thread David Marchand
dev->driver should be set only if a driver did take the device. Signed-off-by: David Marchand --- lib/librte_eal/common/eal_common_pci.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/lib/librte_eal/common/eal_common_pci.c b/lib/librte_eal/common/eal_common_pci.c

[dpdk-dev] [PATCH] pci: Don't call probe callback if driver already loaded.

2016-11-07 Thread David Marchand
Hello Thomas, On Sun, Nov 6, 2016 at 10:59 PM, Thomas Monjalon wrote: > 2016-10-25 14:50, Ben Walker: >> If the user asks to probe multiple times, the probe >> callback should only be called on devices that don't have >> a driver already loaded. >> >> This is useful if a driver is registered

[dpdk-dev] [PATCH 1/3 v3] bnxt: use appropriate data type in bnxt_alloc_vnic_attributes

2016-11-07 Thread Ajit Khaparde
Prevent the arithmetic in bnxt_alloc_vnic_attributes from causing any unintentional havoc because of the usage of a signed variable. Coverity: 137874 Signed-off-by: Ajit Khaparde -- v1: attempt did not seem complete. v2: simplify the fix by redoing the fix for Coverity 127557 ---

[dpdk-dev] [PATCH 2/3] bnxt: add a FALLTHROUGH comment in the cascading switch statement

2016-11-07 Thread Ajit Khaparde
The cascading switch statement in bnxt_hwrm.c is missing the FALLTHROUGH comment. Adding that. Coverity: 127552 Signed-off-by: Ajit Khaparde Acked-by: Ferruh Yigit --- drivers/net/bnxt/bnxt_hwrm.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/net/bnxt/bnxt_hwrm.c

[dpdk-dev] [PATCH 3/3 v2] bnxt: remove support for few PCI IDs

2016-11-07 Thread Ajit Khaparde
Some of the production parts will arrive after the 16.11 release. Back off support for those devices. We will add these IDs again at an appropriate time. Signed-off-by: Ajit Khaparde -- v2: Adding Signed-off-by which seems to have fallen off in the previous attempt. ---

[dpdk-dev] [PATCH] maintainers: claim responsability for xen

2016-11-07 Thread Jianfeng Tan
As some users are still using xen as the hypervisor, I suggest to continue support for xen in DPDK. And from 16.11, I will be the maintainer of all xen-related files. Signed-off-by: Jianfeng Tan --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index

[dpdk-dev] [PATCH] doc: fix typos

2016-11-07 Thread Alain Leon
Fixes typos present in the documentation and code comments. Signed-off-by: Alain Leon --- doc/guides/prog_guide/overview.rst | 2 +- lib/librte_kni/rte_kni.h | 2 +- lib/librte_mbuf/rte_mbuf.h | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git

[dpdk-dev] [PATCH] examples/ipsec-secgw: fix buffer not null terminated

2016-11-07 Thread Thomas Monjalon
2016-11-04 14:18, Ferruh Yigit: > On 11/3/2016 12:12 PM, Fan Zhang wrote: > > Fixes: 0d547ed03717 ("examples/ipsec-secgw: support configuration file") > > Coverity issue: 137854 > > > > Signed-off-by: Fan Zhang > > Acked-by: Ferruh Yigit > > > Minor nit, for all coverity fixes, defined

[dpdk-dev] [PATCH] examples/ipsec-secgw: fix buffer not terminated issue

2016-11-07 Thread Thomas Monjalon
2016-11-04 14:16, Ferruh Yigit: > On 11/3/2016 12:12 PM, Fan Zhang wrote: > > Fixes: 0d547ed03717 ("examples/ipsec-secgw: support configuration file") > > Coverity issue: 137855 > > > > Signed-off-by: Fan Zhang > > Acked-by: Ferruh Yigit Merged with previous patch and applied

[dpdk-dev] [PATCH] app/test: fix wrong pointer values in crypto perftest

2016-11-07 Thread Thomas Monjalon
> > This commit fixes problem with device hanging because of > > wrong pointer values in snow3g performance test > > > > Fixes: 97fe6461c7cb ("app/test: add SNOW 3G performance test") > > > > Signed-off-by: Arek Kusztal > > --- > Acked-by: Fiona Trahe Applied, thanks

[dpdk-dev] [PATCH] crypto: clarify how crypto operations affect buffers

2016-11-07 Thread Thomas Monjalon
> > Updated comments on API to clarify which parts of mbufs are > > copied or changed by crypto operations. > > > > Signed-off-by: Fiona Trahe > > Acked-by: Pablo de Lara Applied, thanks