[dpdk-dev] Updating http://dpdk.org/doc/nics

2016-07-24 Thread Ajit Khaparde
I don't see the Broadcom NICs listed in the list of supported NICs. Can you add an entry for the Broadcom NICs supported by the bnxt PMD driver? Thanks Ajit

[dpdk-dev] [PATCH v6 00/38] new bnxt poll mode driver library

2016-06-16 Thread Ajit Khaparde
On Thu, Jun 16, 2016 at 9:24 AM, Bruce Richardson < bruce.richardson at intel.com> wrote: > On Wed, Jun 15, 2016 at 02:23:00PM -0700, Stephen Hurd wrote: > > The bnxt poll mode library (librte_pmd_bnxt) implements support for > > Broadcom NetXtreme C-Series. These adapters support Standards- > >

[dpdk-dev] [PATCH v6 00/38] new bnxt poll mode driver library

2016-06-21 Thread Ajit Khaparde
Bruce, We have a set of patches which add support to a couple of new Broadcom PCI devices. Should I use the dpdk-next-net/rel_16_07 to push them out? Let me know if there is any other way to do this. Thanks On Thu, Jun 16, 2016 at 1:51 PM, Ajit Khaparde wrote: > > On Thu, Jun 16, 2016 a

[dpdk-dev] [PATCH] bnxt: Add Cumulus+ PCI ID

2016-06-21 Thread Ajit Khaparde
This patch adds support for Cumulus+ Ethernet adapters. These Cumulus+ Ethernet adapters support 10Gb/25Gb/40Gb/50Gb speeds. Signed-off-by: Ajit Khaparde --- lib/librte_eal/common/include/rte_pci_dev_ids.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/lib/librte_eal/common/include

[dpdk-dev] [PATCH] bnxt: Add Cumulus+ PCI ID

2016-06-24 Thread Ajit Khaparde
On Fri, Jun 24, 2016 at 6:59 AM, Bruce Richardson < bruce.richardson at intel.com> wrote: > On Tue, Jun 21, 2016 at 04:58:20PM -0500, Ajit Khaparde wrote: > > This patch adds support for Cumulus+ Ethernet adapters. > > These Cumulus+ Ethernet adapters support 10Gb/

[dpdk-dev] [PATCH] bnxt: Fix a bug in broadcast/multicast setting

2016-07-04 Thread Ajit Khaparde
Currently we are wrongly setting HWRM_CFA_L2_SET_RX_MASK_INPUT_MASK_MCAST flag in bnxt_hwrm_cfa_l2_set_rx_mask() which is preventing promiscuous and multicast promiscuous settings from working correctly. This patch fixes it. Signed-off-by: Ajit Khaparde --- drivers/net/bnxt/bnxt_hwrm.c | 3

[dpdk-dev] [PATCH] bnxt: use unsigned short instead of signed integer in bnxt_alloc_vnic_attributes

2016-11-01 Thread Ajit Khaparde
Prevent the arithmetic in bnxt_alloc_vnic_attributes from causing any unintentional havoc because of the usage of a signed variable. Coverity: 137874 Signed-off-by: Ajit Khaparde --- drivers/net/bnxt/bnxt_vnic.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net

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

2016-11-03 Thread Ajit Khaparde
Prevent the arithmetic in bnxt_alloc_vnic_attributes from causing any unintentional havoc because of the usage of a signed variable. Coverity: 137874 Signed-off-by: Ajit Khaparde -- v2: Previous attempt did not seem complete. --- drivers/net/bnxt/bnxt_vnic.c | 9 + 1 file changed, 5

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

2016-11-04 Thread Ajit Khaparde
On Fri, Nov 4, 2016 at 5:52 AM, Ferruh Yigit wrote: > On 11/3/2016 6:58 PM, Ajit Khaparde wrote: > > Prevent the arithmetic in bnxt_alloc_vnic_attributes from causing > > any unintentional havoc because of the usage of a signed variable. > > > > Coverity: 137874

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

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

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

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

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

2016-11-04 Thread Ajit Khaparde
Some of the production parts will arrive after the 16.11 release. Back off support for those devices. We will add these IDs again at an appropriate time. --- drivers/net/bnxt/bnxt_ethdev.c | 30 -- 1 file changed, 30 deletions(-) diff --git

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

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

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

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

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

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

[dpdk-dev] [PATCH] net/bnxt: fix a segfault encountered during PMD exit

2016-11-15 Thread Ajit Khaparde
This patch fixes segfault encountered during dev_uninit/close routine. KNI sample app can be used to reproduce the issue. backported from upstream commit 316e412 Cc: Signed-off-by: Ajit Khaparde --- drivers/net/bnxt/bnxt.h| 1 + drivers/net/bnxt/bnxt_ethdev.c | 28

[dpdk-dev] [PATCH v12 0/6] add Tx preparation

2016-11-30 Thread Ajit Khaparde
On Mon, ?? Nov 28, 2016 at 5:03 AM, Thomas Monjalon wrote: > We need attention of every PMD developers on this thread. > > Reminder of what Konstantin suggested: > " > - if the PMD supports TX offloads AND > - if to be able use any of these offloads the upper layer SW would have to: > *

[dpdk-dev] [PATCH 1/4] testpmd: Add support to configure 25G and 50G speeds

2016-10-10 Thread Ajit Khaparde
On Mon, Oct 10, 2016 at 10:01 AM, Ferruh Yigit wrote: > Hi Ajit, > > On 9/29/2016 6:03 PM, Ajit Khaparde wrote: > > Support to configure 25G and 50G speeds is missing from testpmd. > > This patch also updates the testpmd user guide accordingly. > > >

[dpdk-dev] [PATCH 2/4] bnxt: Fix a segfault encountered during KNI exit

2016-10-10 Thread Ajit Khaparde
On Mon, Oct 10, 2016 at 10:24 AM, Ferruh Yigit wrote: > On 9/29/2016 8:06 PM, Ajit Khaparde wrote: > > On Thu, Sep 29, 2016 at 12:41 PM, Ferruh Yigit > <mailto:ferruh.yigit at intel.com>>wrote: > > > > On 9/29/2016 6:03 PM, Ajit Khaparde wrote:

[dpdk-dev] [PATCH v2 4/4] net/bnxt: add ULL suffix to constant 1 for bit shift in bnxt_mac_addr_remove_op

2016-10-10 Thread Ajit Khaparde
"net/bnxt: add MAC address") > > Signed-off-by: John W. Linville > ?Acked -by: Ajit Khaparde > --- > v2: fix-up changelog entries based-on comments from Ferruh Yigit > > drivers/net/bnxt/bnxt_ethdev.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(

[dpdk-dev] [PATCH 3/4] bnxt: Add support for Async Link Notification

2016-10-10 Thread Ajit Khaparde
On Mon, Oct 10, 2016 at 10:40 AM, Ferruh Yigit wrote: > Hi Ajit, > > On 9/29/2016 6:03 PM, Ajit Khaparde wrote: > > This patch adds support to get Link notification asynchronously. > > The HW sends Async notifications on default completion ring. The > > PMD processes t

[dpdk-dev] [PATCH v2] bnxt: Add support for Async Link Notification

2016-10-11 Thread Ajit Khaparde
This patch adds support to get Link notification asynchronously. The HW sends Async notifications on default completion ring. The PMD processes these notifications and logs a message appropriately. Signed-off-by: Ajit Khaparde -- v2: Remove unused function rte_bnxt_atomic_read_link_status which

[dpdk-dev] [PATCH v2] app/testpmd: add support to configure 25G and 50G speeds

2016-10-12 Thread Ajit Khaparde
Support to configure 25G and 50G speeds is missing from testpmd. This patch also updates the testpmd user guide accordingly. Signed-off-by: Ajit Khaparde -- v2: shorten lines > 80 character limit --- app/test-pmd/cmdline.c | 16 +++- doc/guides/testpmd_app

[dpdk-dev] [PATCH 0/5] bnxt patchset

2016-08-15 Thread Ajit Khaparde
Please consider applying the following patches: bnxt: Add support for Broadcom StrataGX Communication Processors bnxt: Enable support for NPAR 1.0 feature bnxt: Add support for new HWRM version bnxt: Add support for new DIDs bnxt: Update Broadcom PMD driver documentation

[dpdk-dev] [PATCH 1/5] bnxt: Add support for Broadcom StrataGX Communication Processors

2016-08-15 Thread Ajit Khaparde
writes before the doorbell writes happen. Signed-off-by: John Carney Signed-off-by: Ajit Khaparde --- drivers/net/bnxt/bnxt_cpr.h| 1 + drivers/net/bnxt/bnxt_ethdev.c | 2 ++ drivers/net/bnxt/bnxt_txr.h| 1 + 3 files changed, 4 insertions(+) diff --git a/drivers/net/bnxt/bnxt_cpr.h b

[dpdk-dev] [PATCH 2/5] bnxt: Enable support for NPAR 1.0 feature

2016-08-15 Thread Ajit Khaparde
Adding code to enable support for NIC Partitioning or NPAR 1.0 As a part of NPAR, we don't allow port settings like speed or flow control to be changed. Signed-off-by: Ajit Khaparde --- drivers/net/bnxt/bnxt.h| 4 + drivers/net/bnxt/bnxt_ethdev.c | 13 +- drivers/net

[dpdk-dev] [PATCH 5/5] bnxt: Update Broadcom PMD driver documentation

2016-08-15 Thread Ajit Khaparde
Update doc/guides/nics/bnxt.rst to indicate that the bnxt PMD driver supports Broadcom NetXtreme-C/NetXtreme-E BCM5730X/BCM5740X family of network controllers and Broadcom StrataGX BCM5871X family of communications processors. Signed-off-by: Ajit Khaparde --- doc/guides/nics/bnxt.rst | 23

[dpdk-dev] [PATCH 4/5] bnxt: Add support for new DIDs

2016-08-15 Thread Ajit Khaparde
whether they are configured in copper or fiber mode. This patch adds the necessary DIDs. Signed-off-by: Ajit Khaparde Reviewed-by: David Christensen --- drivers/net/bnxt/bnxt_ethdev.c | 46 -- 1 file changed, 44 insertions(+), 2 deletions(-) diff --git

[dpdk-dev] [PATCH 3/5] bnxt: Add support for new HWRM version

2016-08-15 Thread Ajit Khaparde
Update HWRM specification to 1.3.0 release. Hardware Resource Manager or HWRM in short, is a set of API provided by the firmware running in the ASIC to manage the various resources. Signed-off-by: Ajit Khaparde --- drivers/net/bnxt/hsi_struct_def_dpdk.h | 4731

[dpdk-dev] [PATCH] net/bnxt: make driver less verbose

2016-08-25 Thread Ajit Khaparde
On Thu, Aug 25, 2016 at 6:05 AM, Ferruh Yigit wrote: > This line printed for every application even if driver is not used at > all. Removing the line to reduce the noise. > > Signed-off-by: Ferruh Yigit > --- > drivers/net/bnxt/bnxt_ethdev.c | 1 - > 1 file changed, 1 deletion(-) > > diff

[dpdk-dev] [PATCH] net/bnxt: make driver less verbose

2016-08-26 Thread Ajit Khaparde
On Fri, Aug 26, 2016 at 3:39 AM, Ferruh Yigit wrote: > On 8/25/2016 8:03 PM, Ajit Khaparde wrote: > > > > > > On Thu, Aug 25, 2016 at 6:05 AM, Ferruh Yigit > <mailto:ferruh.yigit at intel.com>> wrote: > > > > This line printed for every applicat

[dpdk-dev] Unsupported SFP+ Module - Hardware Initialization Failure -19

2016-08-30 Thread Ajit Khaparde
On Tue, Aug 30, 2016 at 11:02 AM, Alex Forster wrote: > Hi guys, > > I have a problem again that I had about a year ago[1]. My Finisar > FTL410QE2C (multimode QSFP) transceivers aren?t working with my Intel > X520-QDA1?s. I have 7 servers, each with two X520?s and two QSFP > transceivers in