[dpdk-dev] [PATCH v2 6/9] net/bnxt: fix stats context calculation

2019-10-03 Thread Ajit Khaparde
From: Lance Richardson The required number of statistics contexts is computed as the sum of the number of receive and transmit rings plus one for the async completion ring. A statistics context is not actually required for the async completion ring, so remove it from the calculation. Fixes: bd0a

[dpdk-dev] [PATCH v2 8/9] net/bnxt: advertise scatter receive offload capability

2019-10-03 Thread Ajit Khaparde
From: Lance Richardson Scattered receive is supported but not included in receive offload capabilities. Fix by adding it and including in scattered receive calculation. Fixes: 9c1507d96ab8 ("net/bnxt: switch to the new offload API") Cc: sta...@dpdk.org Signed-off-by: Lance Richardson Reviewed-b

[dpdk-dev] [PATCH v2 9/9] net/bnxt: improve CPR handling in vector PMD

2019-10-03 Thread Ajit Khaparde
From: Lance Richardson Reduce overhead of CPR descriptor validity checking in vector receive and transmit functions. Preserve raw cpr consumer index in vector transmit completion function. Remove an unneeded prefetch (per benchmarking) from vector transmit completion function. Fixes: bc4a000f2

[dpdk-dev] [PATCH v2 5/9] net/bnxt: use common receive transmit nq ring

2019-10-03 Thread Ajit Khaparde
From: Lance Richardson Thor queue scaling is currently limited by the number of NQs that can be allocated. Fix by using a common NQ for all receive/transmit rings instead of allocating a separate NQ for each ring. Fixes: f8168ca0e690 ("net/bnxt: support thor controller") Cc: sta...@dpdk.org Sign

[dpdk-dev] [PATCH v10 1/3] ethdev: decouple flow types and RSS offload types

2019-10-03 Thread Simei Su
This patch decouples RTE_ETH_FLOW_* and ETH_RSS_*. The former defines flow types and the latter defines RSS offload types. Signed-off-by: Simei Su Reviewed-by: Qi Zhang Acked-by: Ori Kam Acked-by: Andrew Rybchenko --- lib/librte_ethdev/rte_ethdev.h | 49 ---

[dpdk-dev] [PATCH v10 0/3] extend RSS offload types

2019-10-03 Thread Simei Su
[PATCH v10 1/3] ethdev: decouple flow types and RSS offload types. [PATCH v10 2/3] ethdev: add several bits for extending rss offload types. [PATCH v10 3/3] app/testpmd: add cmdline support for extending rss types. v10: * Modify duplicate logic in rte_eth_dev_configure() and rte_eth_dev_rss_hash

[dpdk-dev] [PATCH v10 2/3] ethdev: extend RSS offload types

2019-10-03 Thread Simei Su
This patch reserves several bits as input set selection from the high end of the 64 bits. It is combined with exisiting ETH_RSS_* to represent RSS types. Signed-off-by: Simei Su Reviewed-by: Qi Zhang Acked-by: Ori Kam --- lib/librte_ethdev/rte_ethdev.c | 42

[dpdk-dev] [PATCH v10 3/3] app/testpmd: add RSS offload types extending support

2019-10-03 Thread Simei Su
This patch adds cmdline support for extended rss types configuration. Signed-off-by: Simei Su Reviewed-by: Qi Zhang --- app/test-pmd/cmdline.c | 14 -- app/test-pmd/config.c | 4 2 files changed, 16 insertions(+), 2 deletions(-) diff --git a/app/test-pmd/cmdline.c b/app/test

Re: [dpdk-dev] [PATCH 0/8] eBPF arm64 JIT support

2019-10-03 Thread Honnappa Nagarahalli
Adding Arm JIT and Kernel experts > -Original Message- > From: Ananyev, Konstantin > Sent: Thursday, October 3, 2019 10:06 AM > To: Jerin Jacob ; tho...@monjalon.net > Cc: jer...@marvell.com; dpdk-dev ; Honnappa Nagarahalli > ; Gavin Hu (Arm Technology China) > > Subject: RE: [dpdk-dev]

[dpdk-dev] [PATCH 05/10] net/bnxt: update trusted VF status only when it changes

2019-10-03 Thread Ajit Khaparde
We are currently logging trusted VF information everytime a VF config change is indicated by the async message. Update it only when new setting is different from the current setting. Fixes: b42c15c83e88 ("net/bnxt: support trusted VF") Cc: sta...@dpdk.org Signed-off-by: Ajit Khaparde Reviewed-by

[dpdk-dev] [PATCH 01/10] net/bnxt: change return value of few routines

2019-10-03 Thread Ajit Khaparde
From: Kalesh AP Change bnxt_init_nic() and bnxt_init_ring_grps() to void functions as these can never fail. Signed-off-by: Kalesh AP Reviewed-by: Somnath Kotur Reviewed-by: Santoshkumar Karanappa Rastapur --- drivers/net/bnxt/bnxt_ethdev.c | 13 +++-- drivers/net/bnxt/bnxt_ring.c |

[dpdk-dev] [PATCH 00/10] bnxt patchset with bug fixes

2019-10-03 Thread Ajit Khaparde
This patch against dpdk-next-net contains bug fixes for issues found during testing. This patch is dependent on bnxt patchset https://patchwork.dpdk.org/project/dpdk/list/?series=6686 Please apply. Thanks Ajit Khaparde (2): net/bnxt: update trusted VF status only when it changes net/bnxt: ge

[dpdk-dev] [PATCH 02/10] net/bnxt: free default completion ring before VF configuration

2019-10-03 Thread Ajit Khaparde
From: Santoshkumar Karanappa Rastapur If the VF resources need to be reconfigured using the bnxt_hwrm_func_reserve_vf_resc, make sure that the default completion ring is freed first before the resources are reserved. Reallocate the ring once the VF resources are configured. Signed-off-by: Santo

[dpdk-dev] [PATCH 06/10] net/bnxt: get the default HWRM command timeout from firmware

2019-10-03 Thread Ajit Khaparde
The HWRM command timeout is set to a very high value. VER_GET command response returns the default request timeout value. Use this value for waiting for HWRM commands to complete. Poll for the valid bit every 1us instead of 600us. Signed-off-by: Santoshkumar Karanappa Rastapur Signed-off-by: Ajit

[dpdk-dev] [PATCH 03/10] net/bnxt: return error if setting link up fail

2019-10-03 Thread Ajit Khaparde
From: Kalesh AP Currently bnxt driver does not return error in case setting link up fails. Fixes: 5c206086feaa ("net/bnxt: add link state operations") Cc: sta...@dpdk.org Signed-off-by: Kalesh AP Reviewed-by: Somnath Kotur --- drivers/net/bnxt/bnxt_ethdev.c | 2 +- 1 file changed, 1 insertion

[dpdk-dev] [PATCH 09/10] net/bnxt: fix to handle if change status in port start only

2019-10-03 Thread Ajit Khaparde
From: Kalesh AP Driver should handle if change status during port start only. Also fixed to enable interrupts after issuing hwrm_if_change. Fixes: 92d6617b6e33 ("net/bnxt: inform firmware about IF state changes") Signed-off-by: Kalesh AP --- drivers/net/bnxt/bnxt_ethdev.c | 3 ++- drivers/net

[dpdk-dev] [PATCH 04/10] net/bnxt: remove redundant header file inclusion

2019-10-03 Thread Ajit Khaparde
From: Kalesh AP bnxt.h header includes bnxt_cpr.h and bnxt_util.h. There is no need to include these headers file explicitly. This commit does not cause any functional change. Signed-off-by: Kalesh AP Reviewed-by: Ajit Khaparde --- drivers/net/bnxt/bnxt_cpr.c| 1 - drivers/net/bnxt/bnxt_

[dpdk-dev] [PATCH 08/10] net/bnxt: change msix vector to queue mapping

2019-10-03 Thread Ajit Khaparde
From: Rahul Gupta DPDK PCIe-VFIO framework configures base MSIX vector for interrupts which is supported by other h/w. In case of bnxt, base MSIX vector starts with the RX completion queue 0. To comply with the DPDK framework We need to increase the map index by 1 so that RXTX completion queues e

[dpdk-dev] [PATCH 10/10] net/bnxt: remove a useless check in validate flow routine

2019-10-03 Thread Ajit Khaparde
From: Kalesh AP Since vnic->rx_queue_cnt is already checked against non-zero, removed the useless check from the inside condition check. Fixes: 4be31562a4ba ("net/bnxt: allow flow creation when RSS is enabled") Signed-off-by: Kalesh AP Reviewed-by: Ajit Khaparde --- drivers/net/bnxt/bnxt_flo

[dpdk-dev] [PATCH 07/10] net/bnxt: reduce cleanup time during reset recovery

2019-10-03 Thread Ajit Khaparde
From: Rahul Gupta In stop_op(), don't wait for link down event's ASYNC response from FW. Signed-off-by: Rahul Gupta Reviewed-by: Kalesh Anakkur Purayil Reviewed-by: Ajit Khaparde --- drivers/net/bnxt/bnxt_ethdev.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/dr

Re: [dpdk-dev] [PATCH v3 2/3] lib/rcu: add resource reclamation APIs

2019-10-03 Thread Honnappa Nagarahalli
> > Hi Honnappa, > > > > > Add resource reclamation APIs to make it simple for applications > > > > and libraries to integrate rte_rcu library. > > > > > > > > Signed-off-by: Honnappa Nagarahalli > > > > Reviewed-by: Ola Liljedhal > > > > Reviewed-by: Ruifeng Wang > > > > --- > > > > app/test

<    1   2