[dpdk-dev] [PATCHv3 1/2] config/armv8a: disable igb_uio

2016-05-13 Thread Santosh Shukla
On Fri, May 13, 2016 at 02:50:48PM +0200, Thomas Monjalon wrote: > 2016-05-11 19:17, Hemant Agrawal: > > IGB_UIO not supported for arm64 arch in kernel so disable. > > If I understand well, a patch is needed in the kernel to make > igb_uio works? Please confirm. > Yes. User need this [1]

[dpdk-dev] [PATCH 5/7] eal/linux: mmap ioports on ppc64

2016-05-13 Thread Olivier Matz
On 05/13/2016 04:33 PM, David Marchand wrote: >> --- a/lib/librte_eal/common/include/rte_pci.h >> +++ b/lib/librte_eal/common/include/rte_pci.h >> @@ -105,7 +105,8 @@ extern struct pci_device_list pci_device_list; /**< >> Global list of PCI devices. >> /** Nb. of values in PCI resource format.

[dpdk-dev] [PATCH 6/7] virtio: fix pci accesses for ppc64 in legacy mode

2016-05-13 Thread Olivier Matz
Hi David, On 05/13/2016 04:28 PM, David Marchand wrote: >> +#ifdef RTE_ARCH_PPC_64 >> + switch (length) { >> + case 4: >> + *(uint32_t *)dst = rte_be_to_cpu_32(*(uint32_t *)dst); >> + break; >> + case 2: >> + *(uint16_t *)dst =

[dpdk-dev] [PATCH] librte_ether: use RTE_ETH_VALID_PORTID_OR_ERR_RET to check port_id

2016-05-13 Thread Thomas Monjalon
2016-04-29 17:23, Mauricio Vasquez B: > The RTE_ETH_VALID_PORTID_OR_ERR_RET macro is used in some places > to check if a port id is valid or not. This commit makes use of it in > some new parts of the code. There are other occurences: rte_eth_dev_socket_id rte_eth_add_rx_callback

[dpdk-dev] [PATCHv3 1/2] config/armv8a: disable igb_uio

2016-05-13 Thread Jianbo Liu
On 13 May 2016 at 15:47, Jerin Jacob wrote: > On Fri, May 13, 2016 at 03:37:01AM +, Hemant Agrawal wrote: >> >> >> > -Original Message- >> > From: Jianbo Liu [mailto:jianbo.liu at linaro.org] >> > Sent: Friday, May 13, 2016 7:13 AM >> > To: Santosh Shukla >> > Cc: Stephen Hemminger ;

[dpdk-dev] [PATCH] arm64: change rte_memcpy to inline function

2016-05-13 Thread Thomas Monjalon
2016-05-10 14:01, Jianbo Liu: > Other APP may call rte_memcpy by function pointer, > so change it to an inline function. Any example in mind? > --- a/lib/librte_eal/common/include/arch/arm/rte_memcpy_64.h > +++ b/lib/librte_eal/common/include/arch/arm/rte_memcpy_64.h > -#define rte_memcpy(d, s,

[dpdk-dev] [PATCH] qede: fix build with gcc >= 6.0

2016-05-13 Thread Thomas Monjalon
2016-05-10 13:01, Panu Matilainen: > With gcc >= 6.0, qede base driver fails to build with: > drivers/net/qede/base/ecore_cxt.c: In function 'ecore_cdu_init_common': > cc1: error: left shift of negative value [-Werror=shift-negative-value] > > Since the base drivers are untouchable, work around

[dpdk-dev] [PATCH v2 08/11] app/test: convert current pci_test into a single test case

2016-05-13 Thread Thomas Monjalon
2016-05-13 17:19, Jan Viktorin: > On Thu, 12 May 2016 17:34:13 +0200 > Thomas Monjalon wrote: > > > 2016-05-10 20:13, Jan Viktorin: > > > The current test_pci is just a single test case that tests the > > > blacklisting > > > of devices. Rename it to test_pci_blacklist and call it from the > >

[dpdk-dev] [PATCH 11/15] vfio: move global vfio_cfg to eal_vfio.c

2016-05-13 Thread Jan Viktorin
Self review: * missing #ifdef VFIO_PRESENT ... #endif in eal_vfio.c leads to fail when VFIO build is disabled Jan On Fri, 29 Apr 2016 15:44:12 +0200 Jan Viktorin wrote: > The vfio_cfg is a module-global variable and so together with this > variable, it is necessary to move functions: > > *

[dpdk-dev] [PATCH] qede: fix icc build error

2016-05-13 Thread Thomas Monjalon
> >fix errors: > >icc: command line warning #10006: ignoring unknown option > >'-Wno-unused-value' > >icc: command line warning #10006: ignoring unknown option > >'-Wno-format-nonliteral' > >icc: command line warning #10006: ignoring unknown option > >'-Wno-shift-negative-value' >

[dpdk-dev] [PATCH v2 09/11] eal/pci: replace SYSFS_PCI_DEVICES with pci_get_sysfs_path()

2016-05-13 Thread Jan Viktorin
On Thu, 12 May 2016 18:10:07 +0200 Thomas Monjalon wrote: > 2016-05-12 17:46, Jan Viktorin: > > On Thu, 12 May 2016 17:41:22 +0200 > > Thomas Monjalon wrote: > > > 2016-05-10 20:13, Jan Viktorin: > > > > + orig = pci_get_sysfs_path(); > > > > + ret = setenv("SYSFS_PCI_DEVICES",

[dpdk-dev] [PATCH v2 08/11] app/test: convert current pci_test into a single test case

2016-05-13 Thread Jan Viktorin
On Thu, 12 May 2016 17:34:13 +0200 Thomas Monjalon wrote: > 2016-05-10 20:13, Jan Viktorin: > > The current test_pci is just a single test case that tests the blacklisting > > of devices. Rename it to test_pci_blacklist and call it from the test_pci. > > The functions are also moved. It is

[dpdk-dev] [PATCH 1/5] bonding: replace spinlock with read/write lock

2016-05-13 Thread Ananyev, Konstantin
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Ananyev, Konstantin > Sent: Friday, May 13, 2016 6:11 PM > To: Stephen Hemminger; Doherty, Declan > Cc: Iremonger, Bernard; dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH 1/5] bonding: replace spinlock with

[dpdk-dev] [PATCH 1/5] bonding: replace spinlock with read/write lock

2016-05-13 Thread Ananyev, Konstantin
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Stephen Hemminger > Sent: Friday, May 06, 2016 4:56 PM > To: Doherty, Declan > Cc: Iremonger, Bernard; dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH 1/5] bonding: replace spinlock with read/write > lock >

[dpdk-dev] [PATCH] armv7a: disable qede pmd

2016-05-13 Thread Thomas Monjalon
2016-05-09 23:34, Jan Viktorin: > The qede PMD driver is failing when building for ARMv7: > > drivers/net/qede/base/ecore_dev.c: In function ?ecore_hw_init_pf?: > drivers/net/qede/base/ecore_dev.c:1150:6: > error: variable ?prs_reg? set but not used > [-Werror=unused-but-set-variable] >

[dpdk-dev] [PATCH 5/7] eal/linux: mmap ioports on ppc64

2016-05-13 Thread David Marchand
On Fri, May 13, 2016 at 2:50 PM, Olivier Matz wrote: > On PPC64, the ioports are mapped in memory. Implement the missing part > of ioport API for PPC64 when using uio. This may also work on other > architectures but it has not been tested. > > Signed-off-by: David Marchand > Signed-off-by:

[dpdk-dev] [PATCH 6/7] virtio: fix pci accesses for ppc64 in legacy mode

2016-05-13 Thread David Marchand
Hello Olivier, On Fri, May 13, 2016 at 2:50 PM, Olivier Matz wrote: > From: David Marchand > > Although ppc supports both endianesses, qemu supposes that the cpu is > big endian and enforces this for the virtio-net stuff. > > Fix PCI accesses in legacy mode. Only ppc64le is supported at the

[dpdk-dev] [PATCH v3] i40e: configure MTU

2016-05-13 Thread Beilei Xing
This patch enables configuring MTU for i40e. Since changing MTU needs to reconfigure queue, stop port first before configuring MTU. Signed-off-by: Beilei Xing --- v3 changes: Add frame size with extra I40E_VLAN_TAG_SIZE. Delete i40e_dev_rx_init(pf) cause it will be called when port starts. v2

[dpdk-dev] [dpdk-dev, 2/3] eth_dev: add support for device dma mask

2016-05-13 Thread Thomas Monjalon
2016-05-13 09:38, Alejandro Lucero: > On Thu, May 12, 2016 at 4:41 PM, Jan Viktorin > > Alejandro Lucero wrote: > > > On Thu, May 12, 2016 at 3:52 PM, Jan Viktorin > > > > "Alejandro.Lucero" wrote: > > > > > - New dma_mask field in rte_eth_dev_data. > > > > > - If PMD sets device dma_mask,

[dpdk-dev] [RFC PATCH] tools:new tool for system info CPU, memory and huge pages

2016-05-13 Thread Wiles, Keith
Sorry this one should have been an RFC instead of a patch. I create this new tool to combine some information and use /sys/devices instead. What I was hoping was some of you could try this script and see if it works correctly. Also I was hope to find out if this script is useful and what other

[dpdk-dev] [PATCH v2 40/40] bnxt: cleanup null pointer checks

2016-05-13 Thread Stephen Hurd
Prefer !ptr to ptr == NULL Signed-off-by: Stephen Hurd Reviewed-by: Ajit Kumar Khaparde --- drivers/net/bnxt/bnxt_ethdev.c | 10 +- drivers/net/bnxt/bnxt_filter.c | 2 +- drivers/net/bnxt/bnxt_hwrm.c | 6 +++--- drivers/net/bnxt/bnxt_ring.c | 2 +- drivers/net/bnxt/bnxt_vnic.c

[dpdk-dev] [PATCH v2 39/40] bnxt: add flow control operations

2016-05-13 Thread Stephen Hurd
Add flow_ctrl_get and flow_ctrl_set device operations. Signed-off-by: Stephen Hurd Reviewed-by: Ajit Kumar Khaparde --- drivers/net/bnxt/bnxt_ethdev.c | 83 ++ 1 file changed, 83 insertions(+) diff --git a/drivers/net/bnxt/bnxt_ethdev.c

[dpdk-dev] [PATCH v2 38/40] bnxt: add RSS device operations

2016-05-13 Thread Stephen Hurd
Add rss_hash_update and rss_hash_conf_get Signed-off-by: Stephen Hurd Reviewed-by: Ajit Kumar Khaparde --- drivers/net/bnxt/bnxt_ethdev.c | 117 + 1 file changed, 117 insertions(+) diff --git a/drivers/net/bnxt/bnxt_ethdev.c

[dpdk-dev] [PATCH v2 37/40] bnxt: add reta update/query operations

2016-05-13 Thread Stephen Hurd
Update/query reta operations Signed-off-by: Stephen Hurd Reviewed-by: Ajit Kumar Khaparde --- drivers/net/bnxt/bnxt_ethdev.c | 56 ++ 1 file changed, 56 insertions(+) diff --git a/drivers/net/bnxt/bnxt_ethdev.c b/drivers/net/bnxt/bnxt_ethdev.c index

[dpdk-dev] [PATCH v2 36/40] bnxt: add dev set link up/down operations

2016-05-13 Thread Stephen Hurd
Sets link to up or down as appropriate. Signed-off-by: Stephen Hurd Reviewed-by: Ajit Kumar Khaparde --- drivers/net/bnxt/bnxt_ethdev.c | 20 1 file changed, 20 insertions(+) diff --git a/drivers/net/bnxt/bnxt_ethdev.c b/drivers/net/bnxt/bnxt_ethdev.c index

[dpdk-dev] [PATCH v2 35/40] bnxt: add MAC address add/remove operations

2016-05-13 Thread Stephen Hurd
Add/remove MAC addresses Signed-off-by: Stephen Hurd Reviewed-by: Ajit Kumar Khaparde --- drivers/net/bnxt/bnxt_ethdev.c | 69 ++ 1 file changed, 69 insertions(+) diff --git a/drivers/net/bnxt/bnxt_ethdev.c b/drivers/net/bnxt/bnxt_ethdev.c index

[dpdk-dev] [PATCH v2 34/40] bnxt: add device close operation

2016-05-13 Thread Stephen Hurd
Frees all resources except the hwrm ones, which are required to notify the HWRM that the driver is unloaded (these are freed in uninit()). Signed-off-by: Stephen Hurd Reviewed-by: Ajit Kumar Khaparde --- drivers/net/bnxt/bnxt_ethdev.c | 11 +++ 1 file changed, 11 insertions(+) diff

[dpdk-dev] [PATCH v2 33/40] bnxt: add all multicast enable/disable operations

2016-05-13 Thread Stephen Hurd
Enables/disables all multicast traffic. Signed-off-by: Stephen Hurd Reviewed-by: Ajit Kumar Khaparde --- drivers/net/bnxt/bnxt_ethdev.c | 30 ++ 1 file changed, 30 insertions(+) diff --git a/drivers/net/bnxt/bnxt_ethdev.c b/drivers/net/bnxt/bnxt_ethdev.c index

[dpdk-dev] [PATCH v2 32/40] bnxt: add promiscuous enable/disable operations

2016-05-13 Thread Stephen Hurd
Enables and diables promiscuous mode. Signed-off-by: Stephen Hurd Reviewed-by: Ajit Kumar Khaparde --- drivers/net/bnxt/bnxt_ethdev.c | 30 ++ 1 file changed, 30 insertions(+) diff --git a/drivers/net/bnxt/bnxt_ethdev.c b/drivers/net/bnxt/bnxt_ethdev.c index

[dpdk-dev] [PATCH v2 31/40] bnxt: add start/stop/link update operations

2016-05-13 Thread Stephen Hurd
BNXT driver will now minimally pass traffic with testpmd. Signed-off-by: Stephen Hurd Reviewed-by: Ajit Kumar Khaparde --- drivers/net/bnxt/bnxt_ethdev.c | 267 + 1 file changed, 267 insertions(+) diff --git a/drivers/net/bnxt/bnxt_ethdev.c

[dpdk-dev] [PATCH v2 30/40] bnxt: add HWRM port phy qcfg call and wrapper

2016-05-13 Thread Stephen Hurd
Add HWRM port pgy qcfg HWRM command and bnxt_get_hwrm_link_config() wrapper which parses the link state. Signed-off-by: Stephen Hurd Reviewed-by: Ajit Kumar Khaparde --- drivers/net/bnxt/bnxt_hwrm.c | 120 + drivers/net/bnxt/bnxt_hwrm.h | 1 +

[dpdk-dev] [PATCH v2 29/40] bnxt: work around HWRM error when creating rings

2016-05-13 Thread Stephen Hurd
Some HWRM versions will stop responding if we request poll mode interrupt. As a workaround, request an MSI interrupt even though we never enable it. Signed-off-by: Stephen Hurd Reviewed-by: Ajit Kumar Khaparde --- drivers/net/bnxt/bnxt_hwrm.c | 6 +- 1 file changed, 5 insertions(+), 1

[dpdk-dev] [PATCH v2 28/40] bnxt: add ring allocation and group init

2016-05-13 Thread Stephen Hurd
Add a function to initialize ring groups, and a function to allocate the rings via HWRM. This should be the last functionality needed to add start/stop device operations. Signed-off-by: Stephen Hurd Reviewed-by: Ajit Kumar Khaparde --- drivers/net/bnxt/bnxt_ring.c | 119

[dpdk-dev] [PATCH v2 27/40] bnxt: add struct forward decl

2016-05-13 Thread Stephen Hurd
Add missing forward declaration of struct bnxt_ring_struct to avoid requiring additional headers for the function declaation. Signed-off-by: Stephen Hurd Reviewed-by: Ajit Kumar Khaparde --- drivers/net/bnxt/bnxt_hwrm.h | 1 + 1 file changed, 1 insertion(+) diff --git

[dpdk-dev] [PATCH v2 26/40] bnxt: add HWRM stat context free function

2016-05-13 Thread Stephen Hurd
Add function and associated structures and definitions as well as some convenienct functions for manipulating the state of the entire function. Signed-off-by: Stephen Hurd Reviewed-by: Ajit Kumar Khaparde --- drivers/net/bnxt/bnxt_hwrm.c | 219 +

[dpdk-dev] [PATCH v2 25/40] bnxt: add ring group alloc/free functions

2016-05-13 Thread Stephen Hurd
Add HWRM ring group add/free functions and associated structs and definitions. Signed-off-by: Stephen Hurd Reviewed-by: Ajit Kumar Khaparde --- drivers/net/bnxt/bnxt_hwrm.c | 84 +++ drivers/net/bnxt/bnxt_hwrm.h | 4 + drivers/net/bnxt/hsi_struct_def_dpdk.h |

[dpdk-dev] [PATCH v2 24/40] bnxt: add HWRM ring alloc/free functions

2016-05-13 Thread Stephen Hurd
Add HWRM calls to allocate and free TX/RX/CMPL rings along with the associated structs and definitions. Signed-off-by: Stephen Hurd Reviewed-by: Ajit Kumar Khaparde --- drivers/net/bnxt/bnxt_hwrm.c | 108 drivers/net/bnxt/bnxt_hwrm.h | 7 +

[dpdk-dev] [PATCH v2 23/40] bnxt: add HWRM stats context allocation

2016-05-13 Thread Stephen Hurd
Add HWRM code to allocate a statistics context and a helper function to allocate one for evert completion ring. Signed-off-by: Stephen Hurd Reviewed-by: Ajit Kumar Khaparde --- drivers/net/bnxt/bnxt_hwrm.c | 52 drivers/net/bnxt/bnxt_hwrm.h | 3 ++

[dpdk-dev] [PATCH v2 22/40] bnxt: add L2 Rx mask set/clear functions

2016-05-13 Thread Stephen Hurd
Allows setting and clearing L2 context RX masks per vnic Signed-off-by: Stephen Hurd Reviewed-by: Ajit Kumar Khaparde --- drivers/net/bnxt/bnxt_hwrm.c | 45 +++ drivers/net/bnxt/bnxt_hwrm.h | 3 + drivers/net/bnxt/hsi_struct_def_dpdk.h | 135

[dpdk-dev] [PATCH v2 21/40] bnxt: add HWRM vnic RSS config function

2016-05-13 Thread Stephen Hurd
Used to enable RSS configuration Signed-off-by: Stephen Hurd Reviewed-by: Ajit Kumar Khaparde --- drivers/net/bnxt/bnxt_hwrm.c | 24 drivers/net/bnxt/bnxt_hwrm.h | 2 ++ drivers/net/bnxt/hsi_struct_def_dpdk.h | 1 + 3 files changed, 27

[dpdk-dev] [PATCH v2 20/40] bnxt: add vnic RSS cos lb cTx alloc/free functions

2016-05-13 Thread Stephen Hurd
More HWRM calls. Signed-off-by: Stephen Hurd Reviewed-by: Ajit Kumar Khaparde --- drivers/net/bnxt/bnxt_hwrm.c | 38 drivers/net/bnxt/bnxt_hwrm.h | 2 + drivers/net/bnxt/hsi_struct_def_dpdk.h | 153 + 3 files changed, 193

[dpdk-dev] [PATCH v2 19/40] bnxt: add HWRM vnic cfg function

2016-05-13 Thread Stephen Hurd
Configurs a vnic allocaed by vnic_alloc function. Signed-off-by: Stephen Hurd Reviewed-by: Ajit Kumar Khaparde --- drivers/net/bnxt/bnxt_hwrm.c | 34 drivers/net/bnxt/bnxt_hwrm.h | 3 +- drivers/net/bnxt/hsi_struct_def_dpdk.h | 155

[dpdk-dev] [PATCH v2 18/40] bnxt: add HWRM vnic free function

2016-05-13 Thread Stephen Hurd
Frees a vnic allocated by vnic_alloc. Signed-off-by: Stephen Hurd Reviewed-by: Ajit Kumar Khaparde --- drivers/net/bnxt/bnxt_hwrm.c | 21 + drivers/net/bnxt/bnxt_hwrm.h | 1 + drivers/net/bnxt/hsi_struct_def_dpdk.h | 82 ++ 3 files

[dpdk-dev] [PATCH v2 17/40] bnxt: add HWRM vnic alloc function

2016-05-13 Thread Stephen Hurd
This requires a group info array in struct bnxt, so add that, save the max size from the func_qcap response, and alloc/free in init/uninit Signed-off-by: Stephen Hurd Reviewed-by: Ajit Kumar Khaparde --- drivers/net/bnxt/bnxt.h| 2 + drivers/net/bnxt/bnxt_hwrm.c | 33

[dpdk-dev] [PATCH v2 16/40] bnxt: add HWRM function reset command

2016-05-13 Thread Stephen Hurd
Add bnxt_hwrm_func_reset() function and supporting structs and macros. Signed-off-by: Stephen Hurd Reviewed-by: Ajit Kumar Khaparde --- drivers/net/bnxt/bnxt_hwrm.c | 17 + drivers/net/bnxt/bnxt_hwrm.h | 1 + drivers/net/bnxt/hsi_struct_def_dpdk.h | 129

[dpdk-dev] [PATCH v2 15/40] bnxt: alloc/free ring information

2016-05-13 Thread Stephen Hurd
Perform allocation and free()ing of ring information structures for TX, RX, and completion rings. Signed-off-by: Stephen Hurd Reviewed-by: Ajit Kumar Khaparde --- drivers/net/bnxt/bnxt_cpr.c | 28 +++- drivers/net/bnxt/bnxt_cpr.h | 2 +- drivers/net/bnxt/bnxt_rxq.c |

[dpdk-dev] [PATCH v2 14/40] bnxt: initial Rx ring code

2016-05-13 Thread Stephen Hurd
Initial implementation of rx_pkt_burst Add code to allocate rings to bnxt_ring.c Signed-off-by: Stephen Hurd Reviewed-by: Ajit Kumar Khaparde --- drivers/net/bnxt/Makefile | 1 + drivers/net/bnxt/bnxt_ethdev.c | 3 +- drivers/net/bnxt/bnxt_ring.c | 20 +-

[dpdk-dev] [PATCH v2 13/40] bnxt: initial Tx ring code

2016-05-13 Thread Stephen Hurd
Initial implementation of rx_pkt_burst Add code to allocate rings to bnxt_ring.c Signed-off-by: Stephen Hurd Reviewed-by: Ajit Kumar Khaparde --- drivers/net/bnxt/Makefile | 1 + drivers/net/bnxt/bnxt_cpr.h| 4 +- drivers/net/bnxt/bnxt_ethdev.c | 5 +-

[dpdk-dev] [PATCH v2 12/40] bnxt: statistics operations

2016-05-13 Thread Stephen Hurd
Add get and clear staitstics operations and the asociated HWRM calls. Signed-off-by: Stephen Hurd Reviewed-by: Ajit Kumar Khaparde --- drivers/net/bnxt/Makefile | 1 + drivers/net/bnxt/bnxt.h| 5 +- drivers/net/bnxt/bnxt_cpr.c| 5 +-

[dpdk-dev] [PATCH v2 11/40] bnxt: add Rx queue create/destroy operations

2016-05-13 Thread Stephen Hurd
Initial create/destroy queue code. Requires RX ring support to be functional. Signed-off-by: Stephen Hurd Reviewed-by: Ajit Kumar Khaparde --- drivers/net/bnxt/Makefile | 1 + drivers/net/bnxt/bnxt.h| 2 + drivers/net/bnxt/bnxt_ethdev.c | 3 +

[dpdk-dev] [PATCH v2 10/40] bnxt: add Tx queue operations (nonfunctional)

2016-05-13 Thread Stephen Hurd
Add code to create/destroy TX queues. This still requires TX ring support to be completed in a future commit. Signed-off-by: Stephen Hurd Reviewed-by: Ajit Kumar Khaparde --- drivers/net/bnxt/Makefile | 1 + drivers/net/bnxt/bnxt_ethdev.c | 3 + drivers/net/bnxt/bnxt_txq.c| 125

[dpdk-dev] [PATCH v2 09/40] bnxt: add L2 filter alloc/init/free

2016-05-13 Thread Stephen Hurd
Add the L2 filter structure and the alloc/init/free functions for dealing with them. Signed-off-by: Stephen Hurd Reviewed-by: Ajit Kumar Khaparde --- drivers/net/bnxt/Makefile | 1 + drivers/net/bnxt/bnxt.h| 3 + drivers/net/bnxt/bnxt_filter.c | 175

[dpdk-dev] [PATCH v2 08/40] bnxt: add completion ring support

2016-05-13 Thread Stephen Hurd
Structures, macros, and functions for working with completion rings in the driver. Signed-off-by: Stephen Hurd Reviewed-by: Ajit Kumar Khaparde --- drivers/net/bnxt/Makefile | 1 + drivers/net/bnxt/bnxt.h| 6 + drivers/net/bnxt/bnxt_cpr.c| 139

[dpdk-dev] [PATCH v2 07/40] bnxt: declare ring structs and free() func

2016-05-13 Thread Stephen Hurd
Declare ring structures and a ring free() function. Signed-off-by: Stephen Hurd Reviewed-by: Ajit Kumar Khaparde --- drivers/net/bnxt/Makefile| 1 + drivers/net/bnxt/bnxt_ring.c | 51 drivers/net/bnxt/bnxt_ring.h | 92

[dpdk-dev] [PATCH v2 06/40] bnxt: add vnic functions and structs

2016-05-13 Thread Stephen Hurd
Add functions to allocate, initialize, and free vnics. Signed-off-by: Stephen Hurd Reviewed-by: Ajit Kumar Khaparde --- drivers/net/bnxt/Makefile | 1 + drivers/net/bnxt/bnxt.h| 14 ++ drivers/net/bnxt/bnxt_vnic.c | 277

[dpdk-dev] [PATCH v2 05/40] bnxt: add dev configure operation

2016-05-13 Thread Stephen Hurd
This adds the bnxt_hwrm_port_phy_cfg() HWRM call, and copies required information into the new struct bnxt_link_info. Signed-off-by: Stephen Hurd Reviewed-by: Ajit Kumar Khaparde --- drivers/net/bnxt/bnxt.h| 32 +++ drivers/net/bnxt/bnxt_ethdev.c | 24 ++

[dpdk-dev] [PATCH v2 04/40] bnxt: add dev infos get operation

2016-05-13 Thread Stephen Hurd
Gets device info from the bp structure filled in the init() function. Signed-off-by: Stephen Hurd Reviewed-by: Ajit Kumar Khaparde --- drivers/net/bnxt/bnxt.h| 3 ++ drivers/net/bnxt/bnxt_ethdev.c | 96 +- 2 files changed, 98 insertions(+), 1

[dpdk-dev] [PATCH v2 03/40] bnxt: add driver register/unregister support

2016-05-13 Thread Stephen Hurd
Move init() cleanup into uninit() function Fix .dev_private_size Add require hwrm calls: bnxt_hwrm_func_driver_register() bnxt_hwrm_func_driver_unregister() Signed-off-by: Stephen Hurd Reviewed-by: Ajit Kumar Khaparde --- drivers/net/bnxt/bnxt.h| 1 +

[dpdk-dev] [PATCH v2 02/40] bnxt: add HWRM init code

2016-05-13 Thread Stephen Hurd
Start adding HWRM support. Initial commit just performs necessary HWRM queries for init, then fails as before. Thee used HWRM calls so far: bnxt_hwrm_func_qcaps: Queries device capabilities. bnxt_hwrm_ver_get: Gets the firmware version and interface specifications.

[dpdk-dev] [PATCH v2 01/40] bnxt: new driver for Broadcom NetXtreme-C devices

2016-05-13 Thread Stephen Hurd
Initial skeleton simply fails init. Add nic guide and tie into build system. Signed-off-by: Stephen Hurd --- MAINTAINERS | 5 ++ config/common_base | 5 ++ doc/guides/nics/bnxt.rst| 49 +++

[dpdk-dev] [PATCH] doc: move rel_notes instructions as comments

2016-05-13 Thread Olivier Matz
We don't want to have this instructions in the generated docs, so use comments. It's also less confusing for people adding entries in the documentation. Signed-off-by: Olivier Matz --- doc/guides/rel_notes/release_16_07.rst | 86 +- 1 file changed, 43

[dpdk-dev] [PATCH 7/7] config: enable virtio-net pmd for ppc64

2016-05-13 Thread Olivier Matz
Now that virtio pmd is supported on ppc, enable it. Signed-off-by: Olivier Matz --- config/defconfig_ppc_64-power8-linuxapp-gcc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/config/defconfig_ppc_64-power8-linuxapp-gcc b/config/defconfig_ppc_64-power8-linuxapp-gcc index

[dpdk-dev] [PATCH 6/7] virtio: fix pci accesses for ppc64 in legacy mode

2016-05-13 Thread Olivier Matz
From: David Marchand Although ppc supports both endianesses, qemu supposes that the cpu is big endian and enforces this for the virtio-net stuff. Fix PCI accesses in legacy mode. Only ppc64le is supported at the moment. Signed-off-by: David Marchand Signed-off-by:

[dpdk-dev] [PATCH 5/7] eal/linux: mmap ioports on ppc64

2016-05-13 Thread Olivier Matz
On PPC64, the ioports are mapped in memory. Implement the missing part of ioport API for PPC64 when using uio. This may also work on other architectures but it has not been tested. Signed-off-by: David Marchand Signed-off-by: Olivier Matz --- lib/librte_eal/common/include/rte_pci.h| 4 +-

[dpdk-dev] [PATCH 4/7] eal/linux: split function parsing pci resources in sysfs

2016-05-13 Thread Olivier Matz
Split pci_parse_sysfs_resource() and introduce pci_parse_one_sysfs_resource() that parses one line of sysfs resource file. This new function will be exported and used in next commits when mapping the ioports resources. No functional change. Signed-off-by: Olivier Matz ---

[dpdk-dev] [PATCH 3/7] eal/linux: remove invalid comment

2016-05-13 Thread Olivier Matz
In a previous commit, the file used to map the PCI resources changed from "/dev/uio" to "/sys/bus/pci/devices//resource", making the comment wrong. Remove it. Fixes: 9e67561acd1a ("eal/linux: mmap uio resources using resourceX files") Signed-off-by: Olivier Matz ---

[dpdk-dev] [PATCH 2/7] eal/linux: only call iopl on x86

2016-05-13 Thread Olivier Matz
>From iopl(2) man page: "This call is mostly for the x86 architecture. On many other architectures it does not exist or will always return an error". This patch removes the call to iopl() in rte_eal_iopl_init() for architectures other than x86, and always return 0 (success). This was already done

[dpdk-dev] [PATCH 1/7] eal: fix typos in ioport API doxygen comments

2016-05-13 Thread Olivier Matz
Fix some typos and add missing comments related to ioports API in rte_pci.h. Fixes: 756ce64b1 ("eal: introduce PCI ioport API") Signed-off-by: Olivier Matz --- lib/librte_eal/common/include/rte_pci.h | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git

[dpdk-dev] [PATCH 0/7] virtio-net support on ppc64

2016-05-13 Thread Olivier Matz
This patchset allows virtio-net pmd to run on ppc64 processors. The main thing that as missing was the support of ioports in EAL. It also fixes some endianess issues in PCI config accesses in legacy mode. This is validated with test-pmd: === HOST mkdir -p /mnt/huge mount -t hugetlbfs nodev

[dpdk-dev] [PATCHv3 1/2] config/armv8a: disable igb_uio

2016-05-13 Thread Thomas Monjalon
2016-05-11 19:17, Hemant Agrawal: > IGB_UIO not supported for arm64 arch in kernel so disable. If I understand well, a patch is needed in the kernel to make igb_uio works? Please confirm. In that case, yes, the default configuration should be to disable igb_uio. Please note it's just a default

[dpdk-dev] [PATCH] arm64: change rte_memcpy to inline function

2016-05-13 Thread Jerin Jacob
On Tue, May 10, 2016 at 02:01:04PM +0530, Jianbo Liu wrote: > Other APP may call rte_memcpy by function pointer, Instead of "Other APP" may be better to use DPDK application > so change it to an inline function. > > Signed-off-by: Jianbo Liu Acked-by: Jerin Jacob > --- >

[dpdk-dev] [PATCH 06/20] thunderx/nicvf: add dev_infos_get support

2016-05-13 Thread Pattan, Reshma
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Jerin Jacob > Sent: Saturday, May 7, 2016 4:16 PM > To: dev at dpdk.org > Cc: thomas.monjalon at 6wind.com; Richardson, Bruce > ; Jerin Jacob > ; Maciej Czekaj > ; Kamil Rytarowski > ; Zyta Szpak > ; Slawomir

[dpdk-dev] [PATCH 04/20] thunderx/nicvf: add get_reg and get_reg_length support

2016-05-13 Thread Jerin Jacob
On Thu, May 12, 2016 at 03:39:56PM +, Pattan, Reshma wrote: > > > > -Original Message- > > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Jerin Jacob > > Sent: Saturday, May 7, 2016 4:16 PM > > To: dev at dpdk.org > > Cc: thomas.monjalon at 6wind.com; Richardson, Bruce > > ;

[dpdk-dev] [PATCHv3 1/2] config/armv8a: disable igb_uio

2016-05-13 Thread Jerin Jacob
On Fri, May 13, 2016 at 03:37:01AM +, Hemant Agrawal wrote: > > > > -Original Message- > > From: Jianbo Liu [mailto:jianbo.liu at linaro.org] > > Sent: Friday, May 13, 2016 7:13 AM > > To: Santosh Shukla > > Cc: Stephen Hemminger ; Jerin Jacob > > ; Hemant Agrawal > > ; dev at

[dpdk-dev] [PATCH] doc: known issue on EAL argv

2016-05-13 Thread Jingjing Wu
This patch docs the issue on EAL argument that the last EAL argument is replaced by program name in argv[]. Reported-by: Ziye Yang Signed-off-by: Jingjing Wu --- doc/guides/rel_notes/known_issues.rst | 19 +++ 1 file changed, 19 insertions(+) diff --git

[dpdk-dev] [PATCH] eal/linuxapp: fix resource leak

2016-05-13 Thread Thomas Monjalon
2016-05-12 08:55, Sergio Gonzalez Monroy: > On 11/05/2016 17:01, Daniel Mrzyglod wrote: > > Fix issue reported by Coverity. > > Coverity ID 97920 > > > > munmap structure of hugepage > > > > leaked_storage: Variable hugepage going out of scope leaks the storage > > it points to. > > > > The system

[dpdk-dev] [PATCH] sched: fix useless call

2016-05-13 Thread Thomas Monjalon
2016-05-11 10:46, Ferruh Yigit: > On 5/10/2016 6:18 PM, Dumitrescu, Cristian wrote: > > As previously discussed on this email list, the rte_bitmap_free() is an API > > function that works as a placeholder for any resource freeing that needs to > > be done for the bitmap. The API function should

[dpdk-dev] [PATCH v1 09/28] eal: introduce --no-soc option

2016-05-13 Thread Jianbo Liu
On 6 May 2016 at 21:47, Jan Viktorin wrote: > This option has the same meaning for the SoC infra as the --no-pci > for the PCI infra. > > Signed-off-by: Jan Viktorin > --- > lib/librte_eal/common/eal_common_options.c | 5 + > lib/librte_eal/common/eal_internal_cfg.h | 1 + >

[dpdk-dev] [PATCH] sched: fix useless call

2016-05-13 Thread Dumitrescu, Cristian
> -Original Message- > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Friday, May 13, 2016 11:12 AM > To: Dumitrescu, Cristian > Cc: dev at dpdk.org; Yigit, Ferruh ; Mrzyglod, > DanielX > T > Subject: Re: [dpdk-dev] [PATCH] sched: fix useless call > > 2016-05-11

[dpdk-dev] [PATCH] tools:new tool for system info CPU, memory and huge pages

2016-05-13 Thread Keith Wiles
The new tool uses /sys/devices instead of /proc directory, which does not exist on all systems. If the procfs is not available then memory and huge page information is skipped. The tool also can emit a json format in short or long form to allow for machine readable information. Here is the usage

[dpdk-dev] [PATCH v4] examples/qos_meter: fix unchecked return value

2016-05-13 Thread Slawomir Mrozowicz
Fix issue reported by Coverity. Coverity ID 30693: Unchecked return value check_return: Calling rte_meter_srtcm_config without checking return value. Fixes: e6541fdec8b2 ("meter: initial import") Signed-off-by: Slawomir Mrozowicz --- examples/qos_meter/main.c | 16

[dpdk-dev] virtio pmd failed in pci probing

2016-05-13 Thread Vincent Li
sorry for the noise, it turned out I need to add the virtio pmd in the mTCP app Makefile as below LIBS += -m64 -g -O3 -pthread -lrt -march=native -Wl,-export-dynamic ${MTCP_FLD}/lib/libmtcp.a -L../../dpdk/lib -Wl,-lnuma -Wl,-lmtcp -Wl,-lpthread -Wl,-lrt -Wl,-ldl -Wl,--whole-archive

[dpdk-dev] [PATCH] qat: change optimization flag for Intel QuickAssist Technology

2016-05-13 Thread Thomas Monjalon
2016-05-10 10:24, Arek Kusztal: > From: Arkadiusz Kusztal > > Changed to -O3 optimization flag in Intel QuickAssist Technology Makefile There is another change below. Should it be a separate patch? > --- a/drivers/crypto/qat/qat_adf/qat_algs_build_desc.c > +++

[dpdk-dev] [PATCH v4 6/8] virtio-user: add new virtual pci driver for virtio

2016-05-13 Thread Tan, Jianfeng
Hi Michael, On 5/13/2016 1:02 AM, Michael S. Tsirkin wrote: > On Thu, May 12, 2016 at 03:08:05PM +0800, Tan, Jianfeng wrote: >> (2) It's more aligned to previous logic to hide the detail to differentiate >> modern/legacy device. > Why is there a need to support legacy interfaces at all? It's a

[dpdk-dev] [PATCH v4 6/8] virtio-user: add new virtual pci driver for virtio

2016-05-13 Thread Tan, Jianfeng
On 5/13/2016 12:40 AM, Yuanhan Liu wrote: > On Thu, May 12, 2016 at 03:08:05PM +0800, Tan, Jianfeng wrote: +static int +vdev_setup_queue(struct virtio_hw *hw __rte_unused, struct virtqueue *vq) +{ + /* Changed to use virtual addr */ + vq->vq_ring_mem =

[dpdk-dev] [PATCHv3 1/2] config/armv8a: disable igb_uio

2016-05-13 Thread Jianbo Liu
On 12 May 2016 at 18:31, Santosh Shukla wrote: > On Thu, May 12, 2016 at 05:52:54PM +0800, Jianbo Liu wrote: >> On 12 May 2016 at 16:57, Santosh Shukla >> wrote: >> > On Thu, May 12, 2016 at 01:54:13PM +0800, Jianbo Liu wrote: >> >> On 12 May 2016 at 13:06, Santosh Shukla >> >> wrote: >> >> >

[dpdk-dev] [dpdk-dev, 2/3] eth_dev: add support for device dma mask

2016-05-13 Thread Alejandro Lucero
On Thu, May 12, 2016 at 4:41 PM, Jan Viktorin wrote: > Hi, > > Just a note, please, when replying inline, do not prepend ">" before your > new text. I could not find your replies. > > My gmail interface does not show that prepend character... It seems I have to leave a white line before my

[dpdk-dev] [PATCH] examples/ip_pipline: fix memory initialization in firewall bulk functions

2016-05-13 Thread Dumitrescu, Cristian
> -Original Message- > From: Mrzyglod, DanielX T > Sent: Friday, May 6, 2016 6:55 PM > To: Kerlin, MarcinX ; Dumitrescu, Cristian > ; Singh, Jasvinder > > Cc: dev at dpdk.org; Mrzyglod, DanielX T > Subject: [PATCH] examples/ip_pipline: fix memory initialization in firewall > bulk

[dpdk-dev] [PATCH v1 03/28] eal/linux: extract function rte_eal_unbind_kernel_driver

2016-05-13 Thread Jianbo Liu
On 6 May 2016 at 21:47, Jan Viktorin wrote: > Generalize the PCI-specific pci_unbind_kernel_driver. It is now divided into > two parts. First, determination of the path and string identification of the > device to be unbound. Second, the actual unbind operation which is generic. > >

[dpdk-dev] [PATCH v4] examples/qos_meter: fix unchecked return value

2016-05-13 Thread Dumitrescu, Cristian
> -Original Message- > From: Mrozowicz, SlawomirX > Sent: Friday, May 13, 2016 9:35 AM > To: Dumitrescu, Cristian > Cc: dev at dpdk.org; Singh, Jasvinder ; > Mrozowicz, SlawomirX > Subject: [PATCH v4] examples/qos_meter: fix unchecked return value > > Fix issue reported by Coverity. >

[dpdk-dev] [PATCH] qat: change optimization flag for Intel QuickAssist Technology

2016-05-13 Thread Jain, Deepak K
Hi Thomas, This change was made to fix the compilation issues arising when optimization flag was changed. Hence its related to same optimization flag patch. Do you want us to separate the Makefile and .c file change? Regards, Deepak -Original Message- From: Thomas Monjalon

[dpdk-dev] [ovs-dev] Traffic scheduling by qos_sched library in DPDK

2016-05-13 Thread gayathri.manepa...@wipro.com
Hi Team, I started working on implementing the QoS Shaping in OVS+DPDK by making use of rte_sched library provided in DPDK. Meanwhile to compare the performance, started performance test with DPDK sample scheduling application. Below are the configuration details of system which I am using,

[dpdk-dev] [PATCHv3 1/2] config/armv8a: disable igb_uio

2016-05-13 Thread Hemant Agrawal
> -Original Message- > From: Jianbo Liu [mailto:jianbo.liu at linaro.org] > Sent: Friday, May 13, 2016 7:13 AM > To: Santosh Shukla > Cc: Stephen Hemminger ; Jerin Jacob > ; Hemant Agrawal > ; dev at dpdk.org; Thomas Monjalon > > Subject: Re: [dpdk-dev] [PATCHv3 1/2] config/armv8a:

[dpdk-dev] [PATCH v1] net: i40e: add VLAN tag size to RXMAX

2016-05-13 Thread Wu, Jingjing
Hi, Nikita dev_conf.rxmode.max_rx_pkt_len is different with MTU concept. The max_rx_pkt_len indicates the maximum packet length it can receive, it should be larger than MTU. There is another patch which is enabling set_mtu ops. http://dpdk.org/dev/patchwork/patch/12218/ You can definitely help

[dpdk-dev] [PATCH v2 06/11] app/test: use linked list to store PCI drivers

2016-05-13 Thread Thomas Monjalon
2016-05-12 23:00, Jan Viktorin: > On Thu, 12 May 2016 18:08:16 +0200 > Thomas Monjalon wrote: > > > 2016-05-12 17:53, Jan Viktorin: > > > On Thu, 12 May 2016 17:31:28 +0200 > > > Thomas Monjalon wrote: > > > > > > > 2016-05-10 20:13, Jan Viktorin: > > > > > The test unregisters all real

[dpdk-dev] [PATCH v2 6/6] vhost: add pmd client and reconnect option

2016-05-13 Thread Yuanhan Liu
Add client and reconnect option to vhost pmd. reconnect only works when client is given. Signed-off-by: Yuanhan Liu --- drivers/net/vhost/rte_eth_vhost.c | 54 ++- 1 file changed, 42 insertions(+), 12 deletions(-) diff --git

[dpdk-dev] [PATCH v2 5/6] examples/vhost: add client and reconnect option

2016-05-13 Thread Yuanhan Liu
Add --client and --reconnect option to enable the client mode and reconnect mode, respectively. --rconnect works only when --client is given as well. Signed-off-by: Yuanhan Liu --- examples/vhost/main.c | 23 +-- 1 file changed, 21 insertions(+), 2 deletions(-) diff --git

[dpdk-dev] [PATCH v2 4/6] vhost: workaround stale vring base

2016-05-13 Thread Yuanhan Liu
When DPDK app crashes (or quits, or gets killed), a restart of DPDK app would get stale vring base from QEMU. That would break the kernel virtio net completely, making it non-work any more, unless a driver reset is done. So, instead of getting the stale vring base from QEMU, Huawei suggested we

[dpdk-dev] [PATCH v2 3/6] vhost: add reconnect ability

2016-05-13 Thread Yuanhan Liu
Allow reconnecting on failure when both RTE_VHOST_USER_RECONNECT and RTE_VHOST_USER_CLIENT flags are set. Reconnecting means two things here: - when DPDK app starts first and QEMU (as the server) is not started, without reconnecting, DPDK app would simply fail on vhost-user registration. -

[dpdk-dev] [PATCH v2 2/6] vhost: add vhost-user client mode

2016-05-13 Thread Yuanhan Liu
Add a new paramter (flags) to rte_vhost_driver_register(). DPDK vhost-user acts as client mode when RTE_VHOST_USER_CLIENT flag is set. The flags would also allow future extensions without breaking the API (again). The rest is straingfoward then: allocate a unix socket, and bind/listen for

  1   2   >