dpdk.git
6 years agodoc: remove jumbo frames entry from tap features
Matan Azrad [Tue, 13 Feb 2018 13:22:06 +0000 (13:22 +0000)]
doc: remove jumbo frames entry from tap features

The current implementation of the Tap PMD doesn't support packet size
grater than 1522B, so no need to set jumbo frames in the Tap features.

Fixes: 04b2c3c94346 ("doc: add tap features")
Cc: stable@dpdk.org
Signed-off-by: Matan Azrad <matan@mellanox.com>
Acked-by: Pascal Mazon <pascal.mazon@6wind.com>
6 years agodoc: add missing SFN8xxx adapters in sfc guide
Andrew Rybchenko [Fri, 9 Feb 2018 17:57:36 +0000 (17:57 +0000)]
doc: add missing SFN8xxx adapters in sfc guide

Signed-off-by: Andrew Rybchenko <arybchenko@solarflare.com>
Acked-by: John McNamara <john.mcnamara@intel.com>
6 years agodoc: update Intel VF guide
Jingjing Wu [Mon, 12 Feb 2018 10:25:55 +0000 (18:25 +0800)]
doc: update Intel VF guide

update for intel VF usage:
 - Add note for AVF device ID generation.
 - Add steps for vfio-pci pass through.

Signed-off-by: Jingjing Wu <jingjing.wu@intel.com>
Acked-by: John McNamara <john.mcnamara@intel.com>
6 years agodoc: rename nfp PF features file
Alejandro Lucero [Mon, 12 Feb 2018 09:57:38 +0000 (09:57 +0000)]
doc: rename nfp PF features file

Other NICs with VF support do not use the _pf suffix for these files
but just for the VF description.

This change makes NFP files consistent in this regard and helps with
how the DPDK web nics features matrix shows NFP devices.

Signed-off-by: Alejandro Lucero <alejandro.lucero@netronome.com>
6 years agonet/mlx5: add log on flow creation error
Xueming Li [Tue, 6 Feb 2018 12:52:53 +0000 (20:52 +0800)]
net/mlx5: add log on flow creation error

Add error message dump when flow create error happened.

Signed-off-by: Xueming Li <xuemingl@mellanox.com>
Acked-by: Nelio Laranjeiro <nelio.laranjeiro@6wind.com>
6 years agonet/mlx5: revert multicast rule verbs flow type
Shahaf Shuler [Tue, 13 Feb 2018 12:31:55 +0000 (14:31 +0200)]
net/mlx5: revert multicast rule verbs flow type

This is to revert the following commits:
commit da646bd93888 ("net/mlx5: fix all multi verification code position")
commit 0a40a1363a4d ("net/mlx5: fix flow type for allmulti rules")

The last one introduced a bug in the following diff:
@ -1262,6 +1274,7 @@ struct ibv_spec_header {
                eth.val.ether_type &= eth.mask.ether_type;
        }
        mlx5_flow_create_copy(parser, &eth, eth_size);
+       parser->allmulti = eth.val.dst_mac[0] & 1;
        return 0;
 }

As broadcast rules will be considered of type allmulti as well.

The patch was originally intended to enable VF to receive all multicast
traffic by using the IBV_FLOW_ATTR_MC_DEFAULT flow type.
Since the support was removed from the kernel there is no point with
fixing this issue, hence the revert.

Fixes: da646bd93888 ("net/mlx5: fix all multi verification code position")
Fixes: 0a40a1363a4d ("net/mlx5: fix flow type for allmulti rules")
Cc: stable@dpdk.org
Signed-off-by: Shahaf Shuler <shahafs@mellanox.com>
Acked-by: Nelio Laranjeiro <nelio.laranjeiro@6wind.com>
6 years agonet/mlx5: fix close after start failure
Xueming Li [Tue, 6 Feb 2018 12:54:22 +0000 (20:54 +0800)]
net/mlx5: fix close after start failure

This patch fixed primary socket assertion error during close on a device
that failed to start.

Fixes: f8b9a3bad467 ("net/mlx5: install a socket to exchange a file descriptor")
Cc: stable@dpdk.org
Signed-off-by: Xueming Li <xuemingl@mellanox.com>
Acked-by: Nelio Laranjeiro <nelio.laranjeiro@6wind.com>
6 years agonet/dpaa2: remove unused global variable
Jerin Jacob [Sat, 10 Feb 2018 11:57:36 +0000 (17:27 +0530)]
net/dpaa2: remove unused global variable

Due to the missing __packed definition, compiler treating
it as as global variable.

grep "Allocating common symbols" -A 100 build/app/testacl.map
__packed   0x8  /build/lib/librte_pmd_dpaa2.a(dpaa2_rxtx.o)

Fixes: a5fc38d422a7 ("net/dpaa2: support Rx packet parsing")
Cc: stable@dpdk.org
Signed-off-by: Jerin Jacob <jerin.jacob@caviumnetworks.com>
Acked-by: Hemant Agrawal <hemant.agrawal@nxp.com>
6 years agoethdev: increase log level of port allocation failure
Martin Klozik [Tue, 13 Feb 2018 15:03:47 +0000 (15:03 +0000)]
ethdev: increase log level of port allocation failure

DPDK API does not propagate the reason of device allocation failure
from rte_eth_dev_allocate() up to the DPDK application (e.g. Open
vSwitch).
Log level of associated log entries was changed to warning. So user
can find additional details in log files also in production systems,
where debug messages cannot be turned on.

Signed-off-by: Martin Klozik <martinx.klozik@intel.com>
Acked-by: Thomas Monjalon <thomas@monjalon.net>
6 years agoethdev: fix data alignment
Jerin Jacob [Mon, 12 Feb 2018 13:13:43 +0000 (18:43 +0530)]
ethdev: fix data alignment

The struct rte_eth_dev_data is used in ethdev fastpath routines
and it not aligned to cache line size. This patch fixes the ethdev
data alignment.

The alignment was broken from the "first public release" changeset
where ethdev data address was aligned only to the first port.
Remaining ports alignment was defined by the size of the struct
(rte_eth_dev_data). This scheme is not guaranteed to be cache line
aligned all the time.

"ethdev: add port ownership" change set introduced a
rte_eth_dev_shared_data container for port ownership change,
This resulted in rte_eth_dev->data memory for the first port also
as cache unaligned.

Added a compiler alignment attribute to make sure
rte_eth_dev->data always cache aligned so that CPU/compiler
1) Avoid sharing the element with another cache line
2) Can load/store the elements in struct rte_eth_dev_data as
naturally aligned.

Some platform like thunderX could see performance regression of 1%
at "ethdev: add port ownership" change set with
1 port/1 queue l3fwd application and this patch fixes that regression.

example command:
sudo ./examples/l3fwd/build/l3fwd -c 0xff00 -- -p 0x1 --config="(0,0,9)"

Fixes: af75078fece3 ("first public release")
Fixes: 5b7ba31148a8 ("ethdev: add port ownership")
Cc: stable@dpdk.org
Signed-off-by: Jerin Jacob <jerin.jacob@caviumnetworks.com>
Signed-off-by: Pavan Nikhilesh <pbhagavatula@caviumnetworks.com>
Acked-by: Thomas Monjalon <thomas@monjalon.net>
6 years agoeal: fix errno in IPC API
Anatoly Burakov [Sat, 10 Feb 2018 13:15:15 +0000 (13:15 +0000)]
eal: fix errno in IPC API

rte_errno values should not be negative.

Fixes: bacaa2754017 ("eal: add channel for multi-process communication")
Fixes: 783b6e54971d ("eal: add synchronous multi-process communication")

Signed-off-by: Anatoly Burakov <anatoly.burakov@intel.com>
Reviewed-by: Jianfeng Tan <jianfeng.tan@intel.com>
6 years agolib: remove unused map symbols
Pavan Nikhilesh [Mon, 12 Feb 2018 14:23:05 +0000 (19:53 +0530)]
lib: remove unused map symbols

Signed-off-by: Pavan Nikhilesh <pbhagavatula@caviumnetworks.com>
Acked-by: Neil Horman <nhorman@tuxdriver.com>
Acked-by: Hemant Agrawal <hemant.agrawal@nxp.com>
Acked-by: Ferruh Yigit <ferruh.yigit@intel.com>
6 years agomaintainers: update for driver information tool
Neil Horman [Fri, 9 Feb 2018 15:23:36 +0000 (10:23 -0500)]
maintainers: update for driver information tool

I wrote pmdinfogen initially, and since there isn't a maintainer for it,
I'll volunteer to take care of it

Signed-off-by: Neil Horman <nhorman@tuxdriver.com>
Acked-by: Thomas Monjalon <thomas@monjalon.net>
6 years agoversion: 18.02-rc4
Thomas Monjalon [Thu, 8 Feb 2018 22:20:35 +0000 (23:20 +0100)]
version: 18.02-rc4

Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
6 years agodoc: add tested Intel platforms with Intel NICs
Yulong Pei [Wed, 7 Feb 2018 10:17:48 +0000 (18:17 +0800)]
doc: add tested Intel platforms with Intel NICs

Add tested Intel platforms with Intel NICs to the release note.

Signed-off-by: Yulong Pei <yulong.pei@intel.com>
6 years agodoc: add increased default ring size to release notes
John McNamara [Thu, 8 Feb 2018 12:47:58 +0000 (12:47 +0000)]
doc: add increased default ring size to release notes

Added note on the increased ring size in testpmd and the sample
applications to the release note.

Fixes: bd8f10f6d69f ("app/testpmd: increase default ring sizes to 1024")
Fixes: 867a6c66ecf3 ("examples: increase default ring sizes to 1024")

Signed-off-by: John McNamara <john.mcnamara@intel.com>
Acked-by: Ferruh Yigit <ferruh.yigit@intel.com>
6 years agodoc: update limitations in procinfo guide
Marko Kovacevic [Wed, 7 Feb 2018 21:46:26 +0000 (21:46 +0000)]
doc: update limitations in procinfo guide

Updated the limitations for dpdk-procinfo
to inform user of the proper use. To avoid
failures and build issues

Signed-off-by: Marko Kovacevic <marko.kovacevic@intel.com>
Acked-by: John McNamara <john.mcnamara@intel.com>
6 years agodoc: update callback functions in pipeline sample guide
Bao-Long Tran [Thu, 8 Feb 2018 01:19:03 +0000 (08:19 +0700)]
doc: update callback functions in pipeline sample guide

Update f_post_init for pipeline frontend.
Move f_track from pipeline backend to pipeline frontend.

Signed-off-by: Bao-Long Tran <longtb5@viettel.com.vn>
Acked-by: Marko Kovacevic <marko.kovacevic@intel.com>
6 years agoexamples/l3fwd: convert Altivec files to SPDX license tag
Lee Daly [Tue, 23 Jan 2018 16:45:29 +0000 (16:45 +0000)]
examples/l3fwd: convert Altivec files to SPDX license tag

This updates the Intel and IBM license on files in examples to be
the standard BSD-3-Clause license used for the rest of DPDK,
bringing the files in compliance with the DPDK licensing policy.

Signed-off-by: Lee Daly <lee.daly@intel.com>
Acked-by: Ferruh Yigit <ferruh.yigit@intel.com>
Acked-by: Chao Zhu <chaozhu@linux.vnet.ibm.com>
6 years agoexamples/performance-thread: convert license to SPDX tag
Lee Daly [Thu, 8 Feb 2018 14:03:54 +0000 (14:03 +0000)]
examples/performance-thread: convert license to SPDX tag

This updates the Intel and Cavium license on files in examples to be
the standard BSD-3-Clause license used for the rest of DPDK, bringing
the files in compliance with the DPDK licensing policy.

Signed-off-by: Lee Daly <lee.daly@intel.com>
Acked-by: Ferruh Yigit <ferruh.yigit@intel.com>
Acked-by: Jerin Jacob <jerin.jacob@caviumnetworks.com>
6 years agoexamples/ipsec-secgw: support AES 256
Anoob Joseph [Thu, 8 Feb 2018 06:12:42 +0000 (06:12 +0000)]
examples/ipsec-secgw: support AES 256

Adding support for AES 256 algorithm in ipsec-secgw application

Signed-off-by: Anoob Joseph <anoob.joseph@caviumnetworks.com>
Acked-by: Radu Nicolau <radu.nicolau@intel.com>
6 years agoapp/testpmd: fix flow director filter
Beilei Xing [Thu, 8 Feb 2018 12:35:40 +0000 (20:35 +0800)]
app/testpmd: fix flow director filter

This patch fixes issue during dealing with flow
director filter.

Fixes: 6c684f579df5 ("app/testpmd: add or delete flow director filter")
Cc: stable@dpdk.org
Signed-off-by: Beilei Xing <beilei.xing@intel.com>
Acked-by: Jingjing Wu <jingjing.wu@intel.com>
6 years agobuildtools: output build failure reason to stderr
Andrew Rybchenko [Wed, 7 Feb 2018 08:24:00 +0000 (08:24 +0000)]
buildtools: output build failure reason to stderr

If build fails because of failed experimental check and stdout is
redirected to /dev/null, it is absolutely unclear why build fails.

Fixes: a4bcd61de82d ("buildtools: add script to check experimental API exports")

Signed-off-by: Andrew Rybchenko <arybchenko@solarflare.com>
Acked-by: Neil Horman <nhorman@tuxdriver.com>
6 years agoeal/ppc64: revert arch-specific TSC freq query
Gowrishankar Muthukrishnan [Tue, 30 Jan 2018 08:59:13 +0000 (14:29 +0530)]
eal/ppc64: revert arch-specific TSC freq query

This reverts commit 15692396fd68932b6a81f00f12d4b0da12baa7d3
(eal/ppc64: implement arch-specific TSC freq query).
We intended to derive pkt/sec estimation with cpu clock frequency.
As timebase register serves the timer purpose, we need to stick with it
for calculating pkt/sec, hence reverting the change.

Fixes: 15692396fd ("eal/ppc64: implement arch-specific TSC freq query")
Cc: stable@dpdk.org
Signed-off-by: Gowrishankar Muthukrishnan <gowrishankar.m@linux.vnet.ibm.com>
Acked-by: Chao Zhu <chaozhu@linux.vnet.ibm.com>
6 years agodoc: add systems support in tap guide
Ophir Munk [Thu, 8 Feb 2018 14:54:14 +0000 (14:54 +0000)]
doc: add systems support in tap guide

Signed-off-by: Ophir Munk <ophirmu@mellanox.com>
Acked-by: Pascal Mazon <pascal.mazon@6wind.com>
Acked-by: John McNamara <john.mcnamara@intel.com>
6 years agonet/qede: fix tunnel parameters init
Harish Patil [Wed, 7 Feb 2018 21:51:01 +0000 (13:51 -0800)]
net/qede: fix tunnel parameters init

Add the missing VF check to initialize tunnel params correctly.

Fixes: dd28bc8c6ef4 ("net/qede: fix VF port creation sequence")
Cc: stable@dpdk.org
Signed-off-by: Harish Patil <harish.patil@cavium.com>
6 years agonet/mlx5: revert support of IPv4 time-to-live filter
Shahaf Shuler [Wed, 7 Feb 2018 14:34:32 +0000 (16:34 +0200)]
net/mlx5: revert support of IPv4 time-to-live filter

Neither upstream kernel nor MLNX_OFED support such filter.
There is no point announcing this feature.

Reverts commit 0fb2c9842b20 ("net/mlx5: support IPv4 time-to-live filter")

Fixes: 0fb2c9842b20 ("net/mlx5: support IPv4 time-to-live filter")

Signed-off-by: Shahaf Shuler <shahafs@mellanox.com>
Acked-by: Nelio Laranjeiro <nelio.laranjeiro@6wind.com>
6 years agoapp/testpmd: enable CRC strip without capability check
Ferruh Yigit [Thu, 8 Feb 2018 12:08:03 +0000 (12:08 +0000)]
app/testpmd: enable CRC strip without capability check

Some hardware doesn't support disabling CRC strip. In techboard it has
been decided to enable CRC strip always.

The testpmd update in commit 8b9bd0efe0b6, enables CRC strip only if
PMD reports CRC strip capability. Not all PMDs updated to report CRC
strip.

For the PMDs not reporting CRC strip testpmd behavior changed and
disabling CRC strip for them. And this may generate error for PMDs that
doesn't support disabling CRC strip.

Removing capability check for this release. In long term there can be
option to remove CRC strip flag completely or adding a new flag to let
PMD say disabling is not supported.

Fixes: 8b9bd0efe0b6 ("app/testpmd: disable Rx VLAN offloads by default")

Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
Acked-by: Konstantin Ananyev <konstantin.ananyev@intel.com>
6 years agonet/mlx4: add CRC stripping capability
Moti Haimovsky [Thu, 8 Feb 2018 11:14:27 +0000 (13:14 +0200)]
net/mlx4: add CRC stripping capability

This patch updates mlx4 Rx offload capabilities to also indicate that
Rx CRC stripping is (always) supported.

Since the device does not support disabling CRC stripping the PMD
silently ignores such requests.

Signed-off-by: Moti Haimovsky <motih@mellanox.com>
Acked-by: Adrien Mazarguil <adrien.mazarguil@6wind.com>
6 years agonet/i40e: check multi-driver option parsing
Beilei Xing [Thu, 8 Feb 2018 13:18:12 +0000 (21:18 +0800)]
net/i40e: check multi-driver option parsing

This patch fixes the coverity CHECKED_RETURN issue.

Coverity issue: 261779
Fixes: cfdfca493cae ("net/i40e: fix multiple driver support")
Cc: stable@dpdk.org
Reported-by: John McNamara <john.mcnamara@intel.com>
Signed-off-by: Beilei Xing <beilei.xing@intel.com>
Acked-by: Jingjing Wu <jingjing.wu@intel.com>
Acked-by: Helin Zhang <helin.zhang@intel.com>
6 years agodoc: add igb guide
Wei Zhao [Fri, 2 Feb 2018 03:29:32 +0000 (11:29 +0800)]
doc: add igb guide

This patch adds user guide specifically for igb NIC, like ixgbe,
to list all the supported features and known issues.

Signed-off-by: Wei Zhao <wei.zhao1@intel.com>
Acked-by: John McNamara <john.mcnamara@intel.com>
6 years agonet/bnxt: fix packet type
Olivier Matz [Thu, 8 Feb 2018 08:24:17 +0000 (09:24 +0100)]
net/bnxt: fix packet type

The hw flags are not read correctly: the defines
RX_PKT_CMPL_FLAGS_ITYPE_* are not bits but values, so the should not be
tested with if (value & X) but with if ((value & MASK) == X).
This was resulting in a wrong packet type.

For instance, an IPv4/ICMP packet was returning a value of 7 for
the layer 4, which is undefined.

This patch rework the way packet types are processed, to ensure
that only valid packet types will be advertised.

Fixes: 3d2a6644eb05 ("net/bnxt: support getting ptypes")
Cc: stable@dpdk.org
Signed-off-by: Olivier Matz <olivier.matz@6wind.com>
Acked-by: Ajit Khaparde <ajit.khaparde@broadcom.com>
6 years agonet/bnxt: fix return code in MAC address set
Ajit Khaparde [Wed, 7 Feb 2018 01:16:16 +0000 (17:16 -0800)]
net/bnxt: fix return code in MAC address set

There is no need to return an error if an existing MAC is added.

Fixes: 778b759ba10e ("net/bnxt: add MAC address")
Cc: stable@dpdk.org
Signed-off-by: Ajit Khaparde <ajit.khaparde@broadcom.com>
6 years agonet/bnxt: add 100G speed config capability
Ajit Khaparde [Wed, 7 Feb 2018 01:16:15 +0000 (17:16 -0800)]
net/bnxt: add 100G speed config capability

We are not parsing for 100G speed correctly.
With this patch we should be able to configure 100G link.

Fixes: 90ed2b72291a ("net/bnxt: add 100G speed detection")
Cc: stable@dpdk.org
Signed-off-by: Ajit Khaparde <ajit.khaparde@broadcom.com>
6 years agonet/bnxt: fix link speed setting with autoneg off
Ajit Khaparde [Wed, 7 Feb 2018 01:16:14 +0000 (17:16 -0800)]
net/bnxt: fix link speed setting with autoneg off

When Autoneg is turned off especially on remote side,
link does not come up. This patch fixes that.

Fixes: 7bc8e9a227cc ("net/bnxt: support async link notification")
Cc: stable@dpdk.org
Signed-off-by: Ajit Khaparde <ajit.khaparde@broadcom.com>
6 years agodoc: update bnxt feature list
Ajit Khaparde [Wed, 7 Feb 2018 01:16:13 +0000 (17:16 -0800)]
doc: update bnxt feature list

Updating bnxt.ini file.

Signed-off-by: Ajit Khaparde <ajit.khaparde@broadcom.com>
6 years agonet/bnxt: fix Rx checksum flags
Olivier Matz [Tue, 6 Feb 2018 13:39:31 +0000 (14:39 +0100)]
net/bnxt: fix Rx checksum flags

Fix the Rx offload flags when the IP or L4 checksum is seen as incorrect
by the hardware. In this case, the proper value is PKT_RX_IP_CKSUM_BAD.

PKT_RX_IP_CKSUM_NONE means that the checksum may be incorrect in the
packet headers, but the integrity of the IP header is verified. This is
mostly useful for virtual drivers.

Fixes: 7ec39d8c524b ("net/bnxt: update status of Rx IP/L4 CKSUM")
Cc: stable@dpdk.org
Signed-off-by: Olivier Matz <olivier.matz@6wind.com>
Acked-by: Ajit Khaparde <ajit.khaparde@broadcom.com>
6 years agonet/tap: fix cleanup on allocation failure
Moti Haimovsky [Mon, 5 Feb 2018 16:17:20 +0000 (18:17 +0200)]
net/tap: fix cleanup on allocation failure

This patch complements the partial cleanup done inside
eth_dev_tap_create when the routine failed.
Such a failure left a non-functional device attached to the system.

Fixes: 050fe6e9ff97 ("drivers/net: use ethdev allocation helper for vdev")
Cc: stable@dpdk.org
Signed-off-by: Moti Haimovsky <motih@mellanox.com>
Acked-by: Pascal Mazon <pascal.mazon@6wind.com>
6 years agonet/ixgbe: fix CRC stripping capability
Xiaoyun Li [Thu, 8 Feb 2018 05:45:15 +0000 (13:45 +0800)]
net/ixgbe: fix CRC stripping capability

testpmd needs to get CRC strip offload from rx_offload_capa with a
recent change. This patch adds CRC strip flag to rx_offload_capa
for ixgbe and ixgbevf, as they have the capability.

Fixes: 8b9bd0efe0b6 ("app/testpmd: disable Rx VLAN offloads by default")

Signed-off-by: Xiaoyun Li <xiaoyun.li@intel.com>
Acked-by: Wenzhuo Lu <wenzhuo.lu@intel.com>
6 years agonet/i40e: revert enhancing loopback AQ command
Yanglong Wu [Thu, 8 Feb 2018 04:59:53 +0000 (12:59 +0800)]
net/i40e: revert enhancing loopback AQ command

This reverts commit 7b7711bea5f4 ("net/i40e/base: enhance loopback
AQ command").

It was to support multiple type of loopbacks, which results to
configuration error about adminq. So revert it as agreed.

Fixes: 7b7711bea5f4 ("net/i40e/base: enhance loopback AQ command")

Signed-off-by: Yanglong Wu <yanglong.wu@intel.com>
Acked-by: Qi Zhang <qi.z.zhang@intel.com>
6 years agodoc: add multi-driver option in i40e guide
Beilei Xing [Sun, 4 Feb 2018 03:40:57 +0000 (11:40 +0800)]
doc: add multi-driver option in i40e guide

This patch updates the i40e guide with new device parameter of
'support-multi-driver'.

Signed-off-by: Beilei Xing <beilei.xing@intel.com>
Acked-by: Marko Kovacevic <marko.kovacevic@intel.com>
Acked-by: John McNamara <john.mcnamara@intel.com>
6 years agoversion: 18.02-rc3
Thomas Monjalon [Tue, 6 Feb 2018 22:41:40 +0000 (23:41 +0100)]
version: 18.02-rc3

Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
6 years agodoc: convert Intel sharing license header to SPDX tag
Ferruh Yigit [Thu, 1 Feb 2018 17:19:39 +0000 (17:19 +0000)]
doc: convert Intel sharing license header to SPDX tag

Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
Acked-by: Bruce Richardson <bruce.richardson@intel.com>
Acked-by: Marko Kovacevic <marko.kovacevic@intel.com>
Acked-by: Thomas Monjalon <thomas@monjalon.net>
6 years agodoc: convert Intel license headers to SPDX tags
Ferruh Yigit [Thu, 1 Feb 2018 17:18:17 +0000 (17:18 +0000)]
doc: convert Intel license headers to SPDX tags

Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
Acked-by: Bruce Richardson <bruce.richardson@intel.com>
6 years agocompat: relicense some files
Neil Horman [Thu, 1 Feb 2018 12:19:11 +0000 (07:19 -0500)]
compat: relicense some files

Received a note the other day from the Linux Foundation governance board
for DPDK indicating that several files I have copyright on need to be
relicensed to be compliant with the DPDK licensing guidelines.  I have
some concerns with some parts of the request, but am not opposed to
other parts.  So, for those pieces that we are in consensus on, I'm
proposing that we change their license from BSD 2 clause to 3 clause.
I'm also updating the files to use the SPDX licensing scheme

Signed-off-by: Neil Horman <nhorman@tuxdriver.com>
Acked-by: Hemant Agrawal <hemant.agrawal@nxp.com>
6 years agodoc: remove UTF-8 BOM from membership guide
Anatoly Burakov [Thu, 21 Dec 2017 15:28:53 +0000 (15:28 +0000)]
doc: remove UTF-8 BOM from membership guide

Fixes: 55694b2a9f64 ("doc: add membership documentation")
Cc: stable@dpdk.org
Signed-off-by: Anatoly Burakov <anatoly.burakov@intel.com>
Acked-by: Marko Kovacevic <marko.kovacevic@intel.com>
Acked-by: John McNamara <john.mcnamara@intel.com>
6 years agodoc: add hyperthreading note to power library guide
David Hunt [Wed, 31 Jan 2018 12:29:46 +0000 (12:29 +0000)]
doc: add hyperthreading note to power library guide

Signed-off-by: David Hunt <david.hunt@intel.com>
Acked-by: Marko Kovacevic <marko.kovacevic@intel.com>
6 years agodoc: update definition of lcore id and index
Marko Kovacevic [Fri, 2 Feb 2018 15:02:32 +0000 (15:02 +0000)]
doc: update definition of lcore id and index

Added examples in lcore index for better explanation on
various examples, Sited examples for lcore id.

Signed-off-by: Marko Kovacevic <marko.kovacevic@intel.com>
Acked-by: John McNamara <john.mcnamara@intel.com>
6 years agodoc: add note on multiple crypto vdevs
Vipin Varghese [Tue, 6 Feb 2018 16:11:57 +0000 (21:41 +0530)]
doc: add note on multiple crypto vdevs

Add information to explain applications using multiple instances of sw
crypto with example.

Signed-off-by: Vipin Varghese <vipin.varghese@intel.com>
Acked-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>
6 years agodoc: update minimum requirement of l2fwd-crypto
Vipin Varghese [Mon, 5 Feb 2018 17:32:07 +0000 (23:02 +0530)]
doc: update minimum requirement of l2fwd-crypto

Added note section to update information for use cases working with
multiple crypto devices.

Signed-off-by: Vipin Varghese <vipin.varghese@intel.com>
Acked-by: John McNamara <john.mcnamara@intel.com>
Acked-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>
6 years agoexamples/ptpclient: fix Tx configuration
Pablo de Lara [Thu, 1 Feb 2018 16:42:27 +0000 (16:42 +0000)]
examples/ptpclient: fix Tx configuration

The PTP Client application requires IEEE1588 to be supported
by the network driver used, which needs full Tx data path
to be used.

Fixes: b960219b0d83 ("examples/ptpclient: convert to new ethdev offloads API")

Signed-off-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>
6 years agoeventdev: check error in default Rx conf callback
Nikhil Rao [Sun, 4 Feb 2018 18:18:31 +0000 (23:48 +0530)]
eventdev: check error in default Rx conf callback

The default adapter configuration callback is invoked when a Rx
queue is added to the adapter and the adapter detects that a SW
service is needed. The adapter needs to re-configure the device
with an additional port and to do do, it needs to stop the
device and restart it after it is done reconfiguring it. This
patch adds code to check the return code of
rte_event_dev_start() for both when the reconfiguration fails
and when it succeeds and introduces a new error code (-EIO)
for the first case.

Coverity issue: 257000
Fixes: 9c38b704d280 ("eventdev: add eth Rx adapter implementation")
Cc: stable@dpdk.org
Signed-off-by: Nikhil Rao <nikhil.rao@intel.com>
Acked-by: Jerin Jacob <jerin.jacob@caviumnetworks.com>
6 years agonet/virtio-user: check error on clearing non block flag
Sebastian Basierski [Tue, 14 Nov 2017 10:39:56 +0000 (11:39 +0100)]
net/virtio-user: check error on clearing non block flag

Report error message if clearing O_NONBLOCK flag will fail,
then return from function.

Coverity issue: 143439
Fixes: ef53b6030039 ("net/virtio-user: support LSC")
Cc: stable@dpdk.org
Signed-off-by: Sebastian Basierski <sebastianx.basierski@intel.com>
Acked-by: Jianfeng Tan <jianfeng.tan@intel.com>
Acked-by: Maxime Coquelin <maxime.coquelin@redhat.com>
6 years agonet/i40e: fix Rx interrupt
Beilei Xing [Tue, 6 Feb 2018 05:33:31 +0000 (13:33 +0800)]
net/i40e: fix Rx interrupt

This patch fixes interval error and corrects macros when
enabling Rx interrupt mode. The patch also fixes a logical
error during supporting multiple drivers.

Fixes: cfdfca493cae ("net/i40e: fix multiple driver support")
Cc: stable@dpdk.org
Signed-off-by: Beilei Xing <beilei.xing@intel.com>
Acked-by: Helin Zhang <helin.zhang@intel.com>
6 years agonet/i40e: fix multiple DDP packages conflict
Kirill Rybalchenko [Thu, 1 Feb 2018 12:43:05 +0000 (12:43 +0000)]
net/i40e: fix multiple DDP packages conflict

Should be not possible to load conflicting DDP profiles. Only DDP
profiles of the same group (not 0) can be loaded together. If DDP
profile group is 0, it is exclusive, i.e. it cannot be loaded with
any other DDP profile. If DDP profile groups are different, these
profiles cannot be loaded together.

Fixes: b319712f53c8 ("net/i40e: extended list of operations for DDP processing")
Cc: stable@dpdk.org
Signed-off-by: Kirill Rybalchenko <kirill.rybalchenko@intel.com>
Acked-by: Andrey Chilikin <andrey.chilikin@intel.com>
Acked-by: Beilei Xing <beilei.xing@intel.com>
6 years agoapp/testpmd: fix port DCB configuration
Yanglong Wu [Mon, 5 Feb 2018 09:56:19 +0000 (17:56 +0800)]
app/testpmd: fix port DCB configuration

The port_conf for the DCB configuration should
inherit the same configuration of the port.

Fixes: 0074d02fca21 ("app/testpmd: convert to new Rx offloads API")

Signed-off-by: Yanglong Wu <yanglong.wu@intel.com>
Acked-by: Shahaf Shuler <shahafs@mellanox.com>
6 years agotest/timer_perf: fix memory leak
Anatoly Burakov [Tue, 6 Feb 2018 13:35:42 +0000 (13:35 +0000)]
test/timer_perf: fix memory leak

Fixes: 277afaf3dbcb ("app/test: add timer_perf")
Cc: stable@dpdk.org
Signed-off-by: Anatoly Burakov <anatoly.burakov@intel.com>
6 years agotest/table: fix memory leak
Anatoly Burakov [Tue, 6 Feb 2018 13:35:41 +0000 (13:35 +0000)]
test/table: fix memory leak

Always deallocate allocated resources after the test is done.

Fixes: 5205954791cb ("app/test: packet framework unit tests")
Cc: stable@dpdk.org
Signed-off-by: Anatoly Burakov <anatoly.burakov@intel.com>
Acked-by: Cristian Dumitrescu <cristian.dumitrescu@intel.com>
6 years agotest/ring_perf: fix memory leak
Anatoly Burakov [Tue, 6 Feb 2018 13:35:40 +0000 (13:35 +0000)]
test/ring_perf: fix memory leak

Fixes: ac3fb3019c52 ("app: rework ring tests")
Cc: stable@dpdk.org
Signed-off-by: Anatoly Burakov <anatoly.burakov@intel.com>
Acked-by: Olivier Matz <olivier.matz@6wind.com>
6 years agotest/ring: fix memory leak
Anatoly Burakov [Tue, 6 Feb 2018 13:35:39 +0000 (13:35 +0000)]
test/ring: fix memory leak

Get rid of global static ring variable and don't reuse rings
between test runs.

Acked-by: Olivier Matz <olivier.matz@6wind.com>
Fixes: 4e32101f9b01 ("ring: support freeing")
Cc: stable@dpdk.org
Signed-off-by: Anatoly Burakov <anatoly.burakov@intel.com>
6 years agotest/reorder: fix memory leak
Anatoly Burakov [Tue, 6 Feb 2018 13:35:38 +0000 (13:35 +0000)]
test/reorder: fix memory leak

Add a teardown function that frees allocated resources.

Fixes: d0c9b58d7156 ("app/test: new reorder unit test")
Cc: stable@dpdk.org
Signed-off-by: Anatoly Burakov <anatoly.burakov@intel.com>
6 years agotest/bitmap: fix memory leak
Anatoly Burakov [Tue, 6 Feb 2018 13:35:37 +0000 (13:35 +0000)]
test/bitmap: fix memory leak

Fixes: c7e4a134e769 ("test: verify bitmap operations")
Cc: stable@dpdk.org
Signed-off-by: Anatoly Burakov <anatoly.burakov@intel.com>
Acked-by: Cristian Dumitrescu <cristian.dumitrescu@intel.com>
6 years agoexamples/ip_pipeline: fix timer period unit
Bao-Long Tran [Wed, 31 Jan 2018 08:48:16 +0000 (15:48 +0700)]
examples/ip_pipeline: fix timer period unit

The timer_period option specified by users via config file
should have unit of 1 millisecond. However timer_period is
internally converted to unit of 10 millisecond.

Fixes: 4e14069328fc ("examples/ip_pipeline: measure CPU utilization")
Cc: stable@dpdk.org
Signed-off-by: Bao-Long Tran <longtb5@viettel.com.vn>
Reviewed-by: Jasvinder Singh <jasvinder.singh@intel.com>
6 years agoexamples/l2fwd: check if user portmask is valid
Vipin Varghese [Thu, 1 Feb 2018 18:58:42 +0000 (00:28 +0530)]
examples/l2fwd: check if user portmask is valid

User can pass portmask with any value, even invalid mask. The code
checks against actual portmask.

Signed-off-by: Vipin Varghese <vipin.varghese@intel.com>
Acked-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>
6 years agobbdev: fix exported dynamic log type
Amr Mokhtar [Tue, 6 Feb 2018 13:29:30 +0000 (13:29 +0000)]
bbdev: fix exported dynamic log type

This patch fixes shared library compilation due to undefined
reference to an exported variable 'bbdev_logtype'.

Fixes: 4935e1e9f76e ("bbdev: introduce wireless base band device lib")
Fixes: b8cfe2c9aed2 ("bb/turbo_sw: add software turbo driver")
Fixes: 7dc2b1589440 ("bb/null: add null base band device driver")

Signed-off-by: Amr Mokhtar <amr.mokhtar@intel.com>
6 years agoethdev: fix stats query for lowest xstat id
Bruce Richardson [Tue, 6 Feb 2018 16:06:59 +0000 (16:06 +0000)]
ethdev: fix stats query for lowest xstat id

When querying either the name or the value of a stat using the xstats
APIs, a check is done to see if the regular stats API or the xstats APIs
for the driver need to be used. However, the id of the stat requested is
checked to see if it is greater than the number of basic stats, rather
than checking for greater-or-equal, meaning that the xstat with the lowest
id gets incorrectly treated as a basic stat.

This problem manifests itself when you call proc_info using "--xstats-id"
for the first xstat, you get no name of the stat printed, and a random(ish)
stat value.

Fixes: 4773152f850b ("ethdev: optimize xstats by ids APIs")

Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
Reviewed-by: Ferruh Yigit <ferruh.yigit@intel.com>
6 years agobus/dpaa: fix default IOVA mode
Pavan Nikhilesh [Tue, 6 Feb 2018 17:22:48 +0000 (22:52 +0530)]
bus/dpaa: fix default IOVA mode

Fix dpaa bus returning IOVA as PA even when it is not running on dpaa
platform.

Fixes: 1ee9569576f6 ("config: enable dpaaX drivers for generic ARMv8")
Fixes: d5a4e3a00c4a ("bus/dpaa: set IOVA mode as physical")
Cc: stable@dpdk.org
Signed-off-by: Pavan Nikhilesh <pbhagavatula@caviumnetworks.com>
Acked-by: Hemant Agrawal <hemant.agrawal@nxp.com>
6 years agobus/fslmc: fix build with latest glibc
Jerin Jacob [Tue, 6 Feb 2018 17:14:55 +0000 (22:44 +0530)]
bus/fslmc: fix build with latest glibc

Fix the following build error with latest glibc-headers(
Part of aarch64-linux-gnu-gcc-7.3.0 toolchain)

In file included from drivers/bus/fslmc/mc/fsl_mc_sys.h:30:0,
                 from drivers/bus/fslmc/mc/dpmng.c:7:
/usr/aarch64-linux-gnu/usr/include/libio.h:21:2: error:
  #warning "<libio.h> is deprecated; use <stdio.h> instead." [-Werror=cpp]

Fixes: 3af733ba8da8 ("bus/fslmc: introduce MC object functions")
Cc: stable@dpdk.org
Signed-off-by: Jerin Jacob <jerin.jacob@caviumnetworks.com>
6 years agoapp/testpmd: rework simple forward config
Matan Azrad [Mon, 5 Feb 2018 14:09:23 +0000 (14:09 +0000)]
app/testpmd: rework simple forward config

A new function was added to get a Tx port index as a function of the
topology mode and the Rx port index.

Use this function to get the Tx port index of simple stream.

Signed-off-by: Matan Azrad <matan@mellanox.com>
6 years agoapp/testpmd: fix port topology in RSS forward config
Matan Azrad [Mon, 5 Feb 2018 14:09:22 +0000 (14:09 +0000)]
app/testpmd: fix port topology in RSS forward config

The testpmd user can configure port topology mode to define the port
topology between the testpmd forward ports(paired, chained and loop).

When multi-queue ports are configured by the user, the testpmd
streams are created by rss_fwd_config_setup() function, this function
doesn't take into account the chained topology mode and configures the
forward streams with paired topology mode in this case.

Configure the stream Tx port by dedicated function which calculates
a valid Tx port index as a function of the topology mode and the Rx
port index.

Fixes: af75078 ("first public release")
Cc: stable@dpdk.org
Signed-off-by: Matan Azrad <matan@mellanox.com>
6 years agoapp/testpmd: fix port index in RSS forward config
Matan Azrad [Mon, 5 Feb 2018 14:09:21 +0000 (14:09 +0000)]
app/testpmd: fix port index in RSS forward config

When multi-queue ports are configured by the user, the testpmd streams
are created by rss_fwd_config_setup() function.

This function may configure to the streams either invalid Rx ports or
invalid Tx ports.

An invalid Tx port is configured when the number of ports is odd.
In this case, the last Tx port will be always invalid.

An invalid Rx port is configured when NUMA support is configured by the
user and the number of forward ports is much smaller than the number of
all ports. In this case, also the Tx port is invalid.

Change calculations to get valid ports.

Fixes: af75078 ("first public release")
Cc: stable@dpdk.org
Signed-off-by: Matan Azrad <matan@mellanox.com>
6 years agodoc: fix release note for rawdev library
Shreyansh Jain [Thu, 1 Feb 2018 10:19:22 +0000 (15:49 +0530)]
doc: fix release note for rawdev library

'+' sign was missing from librawdev library which is added
in this release.

Fixes: a9bb0c44c775 ("doc: add rawdev library guide and doxygen page")

Signed-off-by: Shreyansh Jain <shreyansh.jain@nxp.com>
Acked-by: Marko Kovacevic <marko.kovacevic@intel.com>
6 years agoraw/skeleton: fix device start test
Shreyansh Jain [Tue, 6 Feb 2018 06:43:38 +0000 (12:13 +0530)]
raw/skeleton: fix device start test

Device can only be started if firmware is loaded, as per Skeleton
rawdev driver semantics. This patch fixes original implementation
which attempted to start the device without loading firmware.

Fixes: 55ca1b0f2151 ("raw/skeleton: add test cases")
Cc: shreyansh.jain@nxp.com
Signed-off-by: Shreyansh Jain <shreyansh.jain@nxp.com>
6 years agobuild: add more implementers IDs and PNs for ARM
Herbert Guan [Wed, 31 Jan 2018 07:39:19 +0000 (15:39 +0800)]
build: add more implementers IDs and PNs for ARM

1) Add native PN option '-march=native' to allow automatic detection.
   Set 'arm_force_native_march' to 'true' in config/arm/meson.build
   to use native PN option.
2) Add implementer_pn option for part num selection in cross compile
3) Add known Arm cortex PN support
4) Add known implementers' IDs (use generic flags/archs by default)
5) Sync build options with config/common_armv8a_linuxapp

Signed-off-by: Herbert Guan <herbert.guan@arm.com>
Acked-by: Pavan Nikhilesh <pbhagavatula@caviumnetworks.com>
6 years agodoc: update flow filtering example guide
Wei Zhao [Wed, 31 Jan 2018 02:12:27 +0000 (10:12 +0800)]
doc: update flow filtering example guide

This patch updates the user guidance of example application of
flow_filtering, with more details of Tx queues configuration.

Signed-off-by: Wei Zhao <wei.zhao1@intel.com>
Reviewed-by: Ori Kam <orika@mellanox.com>
6 years agonet/mlx5: fix UAR remapping on non configured queues
Nélio Laranjeiro [Tue, 6 Feb 2018 09:26:21 +0000 (10:26 +0100)]
net/mlx5: fix UAR remapping on non configured queues

priv_tx_uar_remap() is wrongly considering the queue is already configured
and thus present in the queue array of the device.

Fixes: f8b9a3bad467 ("net/mlx5: install a socket to exchange a file descriptor")
Cc: stable@dpdk.org
Signed-off-by: Nelio Laranjeiro <nelio.laranjeiro@6wind.com>
6 years agonet/mlx5: fix flow RSS configuration
Nélio Laranjeiro [Tue, 6 Feb 2018 09:22:15 +0000 (10:22 +0100)]
net/mlx5: fix flow RSS configuration

An RSS configuration without a key is valid according to the
rte_eth_rss_conf API definition.

Fixes: 8086cf08b2f0 ("net/mlx5: handle RSS hash configuration in RSS flow")
Cc: stable@dpdk.org
Reported-by: Yuanhan Liu <yliu@fridaylinux.org>
Signed-off-by: Nelio Laranjeiro <nelio.laranjeiro@6wind.com>
6 years agonet/mlx: make rdma-core glue path configurable
Adrien Mazarguil [Fri, 2 Feb 2018 16:46:18 +0000 (17:46 +0100)]
net/mlx: make rdma-core glue path configurable

Since rdma-core glue libraries are intrinsically tied to their respective
PMDs and used as internal plug-ins, their presence in the default search
path among other system libraries for the dynamic linker is not necessarily
desired.

This commit enables their installation and subsequent look-up at run time
in RTE_EAL_PMD_PATH if configured to a nonempty string. This path can also
be overridden by environment variables MLX[45]_GLUE_PATH.

Signed-off-by: Adrien Mazarguil <adrien.mazarguil@6wind.com>
6 years agonet/mlx: version rdma-core glue libraries
Adrien Mazarguil [Fri, 2 Feb 2018 16:46:16 +0000 (17:46 +0100)]
net/mlx: version rdma-core glue libraries

When built as separate objects, these libraries do not have unique names.
Since they do not maintain a stable ABI, loading an incompatible library
may result in a crash (e.g. in case multiple versions are installed).

This patch addresses the above by versioning glue libraries, both on the
file system (version suffix) and by comparing a dedicated version field
member in glue structures.

Signed-off-by: Adrien Mazarguil <adrien.mazarguil@6wind.com>
6 years agonet/mlx: fix missing includes for rdma-core glue
Adrien Mazarguil [Fri, 2 Feb 2018 16:46:14 +0000 (17:46 +0100)]
net/mlx: fix missing includes for rdma-core glue

For consistency since these includes are already pulled by others.

Fixes: 4eba244b78ec ("net/mlx4: move rdma-core calls to separate file")
Fixes: 0e83b8e536c1 ("net/mlx5: move rdma-core calls to separate file")

Signed-off-by: Adrien Mazarguil <adrien.mazarguil@6wind.com>
6 years agonet/mlx: add debug checks to glue structure
Adrien Mazarguil [Fri, 2 Feb 2018 16:46:12 +0000 (17:46 +0100)]
net/mlx: add debug checks to glue structure

This code should catch mistakes early if a glue structure member is added
without a corresponding implementation in the library.

Signed-off-by: Adrien Mazarguil <adrien.mazarguil@6wind.com>
6 years agonet/bnxt: fix missing timestamp flag in mbuf
Somnath Kotur [Tue, 6 Feb 2018 02:52:03 +0000 (08:22 +0530)]
net/bnxt: fix missing timestamp flag in mbuf

The timestamp flag needs to be set in the offload flags
for the received pkt in case of PTP offload.

Fixes: b11cceb83a34 ("net/bnxt: support timesync")

Signed-off-by: Somnath Kotur <somnath.kotur@broadcom.com>
6 years agonet/mlx4: fix Rx offload non-fragmented indication
Moti Haimovsky [Tue, 30 Jan 2018 17:02:51 +0000 (19:02 +0200)]
net/mlx4: fix Rx offload non-fragmented indication

This patch fixes the missing RTE_PTYPE_L4_NONFRAG on non-fragmented
IP packets with unrecognized payload type.

Fixes: aee4a03fee4f ("net/mlx4: enhance Rx packet type offloads")
Cc: stable@dpdk.org
Signed-off-by: Moti Haimovsky <motih@mellanox.com>
Acked-by: Adrien Mazarguil <adrien.mazarguil@6wind.com>
6 years agonet/virtio: fix crash while freeing mbufs
David Harton [Sat, 3 Feb 2018 14:55:23 +0000 (09:55 -0500)]
net/virtio: fix crash while freeing mbufs

virtio_dev_free_mbufs was recently modified to free the
virtqueues but failed to check whether the array was
allocated.  Added a check to ensure vqs was non-null.

Fixes: bdb32afbb610 ("net/virtio: rationalize queue flushing")

Signed-off-by: David Harton <dharton@cisco.com>
Reviewed-by: Olivier Matz <olivier.matz@6wind.com>
Reviewed-by: Maxime Coquelin <maxime.coquelin@redhat.com>
6 years agodoc: remove TUN from TAP PMD guide
Vipin Varghese [Tue, 23 Jan 2018 14:29:26 +0000 (14:29 +0000)]
doc: remove TUN from TAP PMD guide

TUN PMD is not supported, removing the references
from Network Interface Controller Driver Tun/Tap
Poll Mode Driver section

Signed-off-by: Vipin Varghese <vipin.varghese@intel.com>
Reviewed-by: Ferruh Yigit <ferruh.yigit@intel.com>
6 years agonet/tap: fix eBPF handling of non-RSS flows
Ophir Munk [Mon, 5 Feb 2018 14:40:42 +0000 (14:40 +0000)]
net/tap: fix eBPF handling of non-RSS flows

The eBPF classifier (section "cls_q" in tap_bpf_program.c) is tracing
marked packets in which skb->cb[1] contains an RSS queue number, and
redirects those packets to the matched queue.
It is expected that skb->cb[1] has been previously set with a valid RSS
queue number during an eBPF action (section "l3_l4" in tap_bpf_program.c).
However, for non-RSS flows, skb->cb[1] may contain a random unset value,
which could falsely be interpreted as a valid RSS queue.
To avoid this potential error, tap_bpf_program.c has been updated as
follows:
1. After calculating the RSS queue number, it is added a unique offset in
order to uniquely identify it as a valid RSS queue number.
2. After matching an RSS queue to a packet, skb->cb[1] is set to 0.

Fixes: cdc07e83bb24 ("net/tap: add eBPF program file")
Fixes: aabe70df73a3 ("net/tap: add eBPF bytes code")

Signed-off-by: Ophir Munk <ophirmu@mellanox.com>
Acked-by: Pascal Mazon <pascal.mazon@6wind.com>
6 years agonet/tap: fix multi segments capability
Ophir Munk [Mon, 5 Feb 2018 10:59:08 +0000 (10:59 +0000)]
net/tap: fix multi segments capability

TAP device is supporting multi segments Tx, however this capability is
not reported when querying the TAP device.
This commit adds this capability report.

Fixes: 818fe14a9891 ("net/tap: use new Tx offloads API")
Cc: stable@dpdk.org
Signed-off-by: Ophir Munk <ophirmu@mellanox.com>
Acked-by: Pascal Mazon <pascal.mazon@6wind.com>
6 years agovhost: remove pending IOTLB entry if miss request failed
Maxime Coquelin [Mon, 5 Feb 2018 15:04:57 +0000 (16:04 +0100)]
vhost: remove pending IOTLB entry if miss request failed

In case vhost_user_iotlb_miss returns an error, the pending IOTLB
entry has to be removed from the list as no IOTLB update will be
received.

Fixes: fed67a20ac94 ("vhost: introduce guest IOVA to backend VA helper")
Cc: stable@dpdk.org
Suggested-by: Tiwei Bie <tiwei.bie@intel.com>
Signed-off-by: Maxime Coquelin <maxime.coquelin@redhat.com>
6 years agovhost: fix IOTLB pool out-of-memory handling
Maxime Coquelin [Mon, 5 Feb 2018 15:04:56 +0000 (16:04 +0100)]
vhost: fix IOTLB pool out-of-memory handling

In the unlikely case the IOTLB memory pool runs out of memory,
an issue may happen if all entries are used by the IOTLB cache,
and an IOTLB miss happen. If the iotlb pending list is empty,
then no memory is freed and allocation fails a second time.

This patch fixes this by doing an IOTLB cache random evict if
the IOTLB pending list is empty, ensuring the second allocation
try will succeed.

In the same spirit, the opposite is done when inserting an
IOTLB entry in the IOTLB cache fails due to out of memory. In
this case, the IOTLB pending is flushed if the IOTLB cache is
empty to ensure the new entry can be inserted.

Fixes: d012d1f293f4 ("vhost: add IOTLB helper functions")
Fixes: f72c2ad63aeb ("vhost: add pending IOTLB miss request list and helpers")
Cc: stable@dpdk.org
Signed-off-by: Maxime Coquelin <maxime.coquelin@redhat.com>
6 years agodoc: update mlx required OFED version
Shahaf Shuler [Mon, 5 Feb 2018 12:31:12 +0000 (14:31 +0200)]
doc: update mlx required OFED version

Signed-off-by: Shahaf Shuler <shahafs@mellanox.com>
Acked-by: Nelio Laranjeiro <nelio.laranjeiro@6wind.com>
Acked-by: Adrien Mazarguil <adrien.mazarguil@6wind.com>
6 years agodoc: update mlx PMD release notes
Shahaf Shuler [Mon, 5 Feb 2018 12:31:11 +0000 (14:31 +0200)]
doc: update mlx PMD release notes

Signed-off-by: Shahaf Shuler <shahafs@mellanox.com>
Acked-by: Nelio Laranjeiro <nelio.laranjeiro@6wind.com>
Acked-by: Adrien Mazarguil <adrien.mazarguil@6wind.com>
6 years agonet/failsafe: fix default Tx offloads capabilities
Ophir Munk [Mon, 5 Feb 2018 16:02:06 +0000 (16:02 +0000)]
net/failsafe: fix default Tx offloads capabilities

Failsafe reported Tx offloads capabilities are the AND result of its
default capabilities and those of its sub-devices.
In the corrupted code failsafe default Tx capabilities were set to 0.
As a result when running testpmd with "--tx-offloads=0x8000" parameter
(request for multi segments offload) - an error was returned:

PMD: net_failsafe:
Some Tx offloads are not supported, requested 0x8000 supported 0x0

To fix this, failsafe default Tx offload capabilities are set to
DEV_TX_OFFLOAD_MULTI_SEGS |
DEV_TX_OFFLOAD_IPV4_CKSUM |
DEV_TX_OFFLOAD_UDP_CKSUM |
DEV_TX_OFFLOAD_TCP_CKSUM,

Fixes: a46f8d584eb8 ("net/failsafe: add fail-safe PMD")
Cc: stable@dpdk.org
Signed-off-by: Ophir Munk <ophirmu@mellanox.com>
Acked-by: Gaetan Rivet <gaetan.rivet@6wind.com>
6 years agonet/vdev_netvsc: fix device detection error check
Matan Azrad [Mon, 5 Feb 2018 12:48:22 +0000 (12:48 +0000)]
net/vdev_netvsc: fix device detection error check

The vdev_netvsc driver does periodic detection of PCI devices matched
to the netvsc existed interfaces.

When it finds a match, the PCI address is written to the pipe of the
associated fail-safe PMD instance and a positive value is returned to
the periodic check which is wrongly considered as error.

Change the check to consider only a negative value as error.

Fixes: e7dc5d7becc5 ("net/vdev_netvsc: implement core functionality")

Signed-off-by: Matan Azrad <matan@mellanox.com>
6 years agodoc: align qede dynamic log names with standard
Rasesh Mody [Sat, 3 Feb 2018 06:03:19 +0000 (22:03 -0800)]
doc: align qede dynamic log names with standard

Signed-off-by: Rasesh Mody <rasesh.mody@cavium.com>
6 years agonet/qede: fix few log messages
Rasesh Mody [Sat, 3 Feb 2018 06:03:18 +0000 (22:03 -0800)]
net/qede: fix few log messages

Fixes: 9e334305178f ("net/qede: fix MTU set and max Rx length")
Fixes: 22d07d939c3c ("net/qede/base: update")
Cc: stable@dpdk.org
Signed-off-by: Rasesh Mody <rasesh.mody@cavium.com>
6 years agonet/qede: fix VF port creation sequence
Harish Patil [Sat, 3 Feb 2018 06:03:17 +0000 (22:03 -0800)]
net/qede: fix VF port creation sequence

Few adjustments are required to effectively handle VF vport create/delete
sequence. The problem is exposed by recent ethdev TX offload changes
which requires port to be in down state before applying TX offloads.

 - Move vport creation from dev_init() to dev_configure()
 - Force to stop vport if it was already started due to previous run
   (restart case)
 - Move link state enable/disable to dev_init() and dev_close()
   respectively.
 - For MTU change, recreate vport with new MTU value and restore old
   config. This is necessary since VF MTU value can be changed only upon
   vport creation.

Fixes: ec94dbc57362 ("qede: add base driver")
Cc: stable@dpdk.org
Signed-off-by: Harish Patil <harish.patil@cavium.com>
6 years agonet/octeontx: register fpa as platform HW mempool
Pavan Nikhilesh [Mon, 5 Feb 2018 10:55:57 +0000 (16:25 +0530)]
net/octeontx: register fpa as platform HW mempool

Register octeontx-fpavf as platform HW mempool when net/octeontx pmd is
used.

Signed-off-by: Pavan Nikhilesh <pbhagavatula@caviumnetworks.com>
Acked-by: Santosh Shukla <santosh.shukla@caviumnetworks.com>
6 years agonet/bonding: check error of MAC address setting
Radu Nicolau [Thu, 1 Feb 2018 17:21:26 +0000 (17:21 +0000)]
net/bonding: check error of MAC address setting

Coverity issue: 260405
Fixes: 2efb58cbab6e ("bond: new link bonding library")
Cc: stable@dpdk.org
Signed-off-by: Radu Nicolau <radu.nicolau@intel.com>
6 years agonet/ena: fix jumbo support in Rx offloads flags
Rafal Kozik [Thu, 1 Feb 2018 13:06:36 +0000 (14:06 +0100)]
net/ena: fix jumbo support in Rx offloads flags

ENA device supports Rx jumbo frames and such information needs to
be provided in the offloads flags.

Fixes: 7369f88f88c0 ("net/ena: convert to new Rx offloads API")

Signed-off-by: Rafal Kozik <rk@semihalf.com>
Acked-by: Michal Krawczyk <mk@semihalf.com>
6 years agonet/failsafe: fix error string after ownership call
Gaetan Rivet [Thu, 1 Feb 2018 10:59:30 +0000 (11:59 +0100)]
net/failsafe: fix error string after ownership call

Ownership API returns a negative value, strerror expects a valid errno
value, thus positive.

Coverity issue: 260401
Fixes: dcd0c9c32b8d ("net/failsafe: use ownership mechanism for slaves")

Signed-off-by: Gaetan Rivet <gaetan.rivet@6wind.com>