DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH 0/7] bnxt patches
@ 2019-07-19  6:18 Ajit Khaparde
  2019-07-19  6:19 ` [dpdk-dev] [PATCH 1/7] net/bnxt: add new device id Ajit Khaparde
                   ` (8 more replies)
  0 siblings, 9 replies; 14+ messages in thread
From: Ajit Khaparde @ 2019-07-19  6:18 UTC (permalink / raw)
  To: dev; +Cc: ferruh.yigit

Patchset based after an earlier submission
https://patchwork.dpdk.org/patch/56690/

This set mostly contains bug fixes, an update to the bnxt.rst doc
and a adds a VF PCI device ID for Thor based network controller.

Please apply.

Ajit Khaparde (2):
  net/bnxt: remove unnecessary interrupt disable
  doc: update release information for bnxt

Kalesh AP (1):
  net/bnxt: add new device id

Lance Richardson (1):
  net/bnxt: disable vector mode Tx with VLAN offload

Rahul Gupta (1):
  net/bnxt: fix interrupt rearm logic

Santoshkumar Karanappa Rastapur (1):
  net/bnxt: use correct number of RSS contexts for thor

Stephen Hemminger (1):
  net/bnxt: fix receive interrupt vector

 doc/guides/rel_notes/release_19_08.rst |  2 +-
 drivers/net/bnxt/bnxt_ethdev.c         | 26 +++++++++++++-------------
 drivers/net/bnxt/bnxt_hwrm.c           |  2 +-
 drivers/net/bnxt/bnxt_rxq.c            |  2 +-
 4 files changed, 16 insertions(+), 16 deletions(-)

-- 
2.20.1 (Apple Git-117)


^ permalink raw reply	[flat|nested] 14+ messages in thread

* [dpdk-dev] [PATCH 1/7] net/bnxt: add new device id
  2019-07-19  6:18 [dpdk-dev] [PATCH 0/7] bnxt patches Ajit Khaparde
@ 2019-07-19  6:19 ` Ajit Khaparde
  2019-07-19  6:19 ` [dpdk-dev] [PATCH 2/7] net/bnxt: use correct number of RSS contexts for thor Ajit Khaparde
                   ` (7 subsequent siblings)
  8 siblings, 0 replies; 14+ messages in thread
From: Ajit Khaparde @ 2019-07-19  6:19 UTC (permalink / raw)
  To: dev; +Cc: ferruh.yigit, Kalesh AP

From: Kalesh AP <kalesh-anakkur.purayil@broadcom.com>

Added support for new thor device.

Signed-off-by: Kalesh AP <kalesh-anakkur.purayil@broadcom.com>
Signed-off-by: Ajit Khaparde <ajit.khaparde@broadcom.com>
---
 drivers/net/bnxt/bnxt_ethdev.c | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/drivers/net/bnxt/bnxt_ethdev.c b/drivers/net/bnxt/bnxt_ethdev.c
index 87f069caa..6492f5627 100644
--- a/drivers/net/bnxt/bnxt_ethdev.c
+++ b/drivers/net/bnxt/bnxt_ethdev.c
@@ -74,7 +74,8 @@ int bnxt_logtype_driver;
 #define BROADCOM_DEV_ID_57508 0x1750
 #define BROADCOM_DEV_ID_57504 0x1751
 #define BROADCOM_DEV_ID_57502 0x1752
-#define BROADCOM_DEV_ID_57500_VF 0x1807
+#define BROADCOM_DEV_ID_57500_VF1 0x1806
+#define BROADCOM_DEV_ID_57500_VF2 0x1807
 #define BROADCOM_DEV_ID_58802 0xd802
 #define BROADCOM_DEV_ID_58804 0xd804
 #define BROADCOM_DEV_ID_58808 0x16f0
@@ -126,7 +127,8 @@ static const struct rte_pci_id bnxt_pci_id_map[] = {
 	{ RTE_PCI_DEVICE(PCI_VENDOR_ID_BROADCOM, BROADCOM_DEV_ID_57508) },
 	{ RTE_PCI_DEVICE(PCI_VENDOR_ID_BROADCOM, BROADCOM_DEV_ID_57504) },
 	{ RTE_PCI_DEVICE(PCI_VENDOR_ID_BROADCOM, BROADCOM_DEV_ID_57502) },
-	{ RTE_PCI_DEVICE(PCI_VENDOR_ID_BROADCOM, BROADCOM_DEV_ID_57500_VF) },
+	{ RTE_PCI_DEVICE(PCI_VENDOR_ID_BROADCOM, BROADCOM_DEV_ID_57500_VF1) },
+	{ RTE_PCI_DEVICE(PCI_VENDOR_ID_BROADCOM, BROADCOM_DEV_ID_57500_VF2) },
 	{ .vendor_id = 0, /* sentinel */ },
 };
 
@@ -3338,7 +3340,8 @@ static bool bnxt_vf_pciid(uint16_t id)
 	    id == BROADCOM_DEV_ID_STRATUS_NIC_VF1 ||
 	    id == BROADCOM_DEV_ID_STRATUS_NIC_VF2 ||
 	    id == BROADCOM_DEV_ID_58802_VF ||
-	    id == BROADCOM_DEV_ID_57500_VF)
+	    id == BROADCOM_DEV_ID_57500_VF1 ||
+	    id == BROADCOM_DEV_ID_57500_VF2)
 		return true;
 	return false;
 }
@@ -3808,7 +3811,8 @@ bnxt_dev_init(struct rte_eth_dev *eth_dev)
 	if (pci_dev->id.device_id == BROADCOM_DEV_ID_57508 ||
 	    pci_dev->id.device_id == BROADCOM_DEV_ID_57504 ||
 	    pci_dev->id.device_id == BROADCOM_DEV_ID_57502 ||
-	    pci_dev->id.device_id == BROADCOM_DEV_ID_57500_VF)
+	    pci_dev->id.device_id == BROADCOM_DEV_ID_57500_VF1 ||
+	    pci_dev->id.device_id == BROADCOM_DEV_ID_57500_VF2)
 		bp->flags |= BNXT_FLAG_THOR_CHIP;
 
 	rc = bnxt_init_board(eth_dev);
-- 
2.20.1 (Apple Git-117)


^ permalink raw reply	[flat|nested] 14+ messages in thread

* [dpdk-dev] [PATCH 2/7] net/bnxt: use correct number of RSS contexts for thor
  2019-07-19  6:18 [dpdk-dev] [PATCH 0/7] bnxt patches Ajit Khaparde
  2019-07-19  6:19 ` [dpdk-dev] [PATCH 1/7] net/bnxt: add new device id Ajit Khaparde
@ 2019-07-19  6:19 ` Ajit Khaparde
  2019-07-19 21:38   ` Ferruh Yigit
  2019-07-19  6:19 ` [dpdk-dev] [PATCH 3/7] net/bnxt: fix receive interrupt vector Ajit Khaparde
                   ` (6 subsequent siblings)
  8 siblings, 1 reply; 14+ messages in thread
From: Ajit Khaparde @ 2019-07-19  6:19 UTC (permalink / raw)
  To: dev; +Cc: ferruh.yigit, Santoshkumar Karanappa Rastapur, Lance Richardson

From: Santoshkumar Karanappa Rastapur <santosh.rastapur@broadcom.com>

BCM57500-based adapters use a variable number of RSS contexts
depending upon the number of receive rings in use. The current
implementation is erroneously using the maximum possible number
of RSS contexts instead of the actual number allocated when
setting up RSS tables in the adapter. Fix by using the actual
number of allocated contexts.

Fixes: 38412304b50a ("net/bnxt: enable RSS for thor-based controllers")

Reviewed-by: Lance Richardson <lance.richardson@broadcom.com>
Signed-off-by: Santoshkumar Karanappa Rastapur <santosh.rastapur@broadcom.com>
Signed-off-by: Ajit Khaparde <ajit.khaparde@broadcom.com>
---
 drivers/net/bnxt/bnxt_hwrm.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/bnxt/bnxt_hwrm.c b/drivers/net/bnxt/bnxt_hwrm.c
index 67b5ff661..692c6c399 100644
--- a/drivers/net/bnxt/bnxt_hwrm.c
+++ b/drivers/net/bnxt/bnxt_hwrm.c
@@ -1736,7 +1736,7 @@ bnxt_hwrm_vnic_rss_cfg_thor(struct bnxt *bp, struct bnxt_vnic_info *vnic)
 {
 	int i;
 	int rc = 0;
-	int nr_ctxs = bp->max_ring_grps;
+	int nr_ctxs = vnic->num_lb_ctxts;
 	struct hwrm_vnic_rss_cfg_input req = {.req_type = 0 };
 	struct hwrm_vnic_rss_cfg_output *resp = bp->hwrm_cmd_resp_addr;
 
-- 
2.20.1 (Apple Git-117)


^ permalink raw reply	[flat|nested] 14+ messages in thread

* [dpdk-dev] [PATCH 3/7] net/bnxt: fix receive interrupt vector
  2019-07-19  6:18 [dpdk-dev] [PATCH 0/7] bnxt patches Ajit Khaparde
  2019-07-19  6:19 ` [dpdk-dev] [PATCH 1/7] net/bnxt: add new device id Ajit Khaparde
  2019-07-19  6:19 ` [dpdk-dev] [PATCH 2/7] net/bnxt: use correct number of RSS contexts for thor Ajit Khaparde
@ 2019-07-19  6:19 ` Ajit Khaparde
  2019-07-19  6:19 ` [dpdk-dev] [PATCH 4/7] net/bnxt: fix interrupt rearm logic Ajit Khaparde
                   ` (5 subsequent siblings)
  8 siblings, 0 replies; 14+ messages in thread
From: Ajit Khaparde @ 2019-07-19  6:19 UTC (permalink / raw)
  To: dev; +Cc: ferruh.yigit, Stephen Hemminger, stable, Rahul Gupta

From: Stephen Hemminger <stephen@networkplumber.org>

The receive interrupt vector should be offset by the constant
RTE_INTR_VEC_RXTX_OFFSET; otherwise setting up some queue interrupts
will fail.

Fixes: 1fe427fd08ee ("net/bnxt: support enable/disable interrupt")
Cc: stable@dpdk.org

Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
Reviewed-by: Ajit Khaparde <ajit.khaparde@broadcom.com>
Tested-by: Rahul Gupta <rahul.gupta@broadcom.com>
Signed-off-by: Ajit Khaparde <ajit.khaparde@broadcom.com>
---
 drivers/net/bnxt/bnxt_ethdev.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/net/bnxt/bnxt_ethdev.c b/drivers/net/bnxt/bnxt_ethdev.c
index 6492f5627..fec1ecbae 100644
--- a/drivers/net/bnxt/bnxt_ethdev.c
+++ b/drivers/net/bnxt/bnxt_ethdev.c
@@ -427,7 +427,8 @@ static int bnxt_init_chip(struct bnxt *bp)
 			intr_handle->max_intr);
 		for (queue_id = 0; queue_id < bp->eth_dev->data->nb_rx_queues;
 		     queue_id++) {
-			intr_handle->intr_vec[queue_id] = vec;
+			intr_handle->intr_vec[queue_id] =
+							vec + BNXT_RX_VEC_START;
 			if (vec < base + intr_handle->nb_efd - 1)
 				vec++;
 		}
-- 
2.20.1 (Apple Git-117)


^ permalink raw reply	[flat|nested] 14+ messages in thread

* [dpdk-dev] [PATCH 4/7] net/bnxt: fix interrupt rearm logic
  2019-07-19  6:18 [dpdk-dev] [PATCH 0/7] bnxt patches Ajit Khaparde
                   ` (2 preceding siblings ...)
  2019-07-19  6:19 ` [dpdk-dev] [PATCH 3/7] net/bnxt: fix receive interrupt vector Ajit Khaparde
@ 2019-07-19  6:19 ` Ajit Khaparde
  2019-07-19  6:19 ` [dpdk-dev] [PATCH 5/7] net/bnxt: disable vector mode Tx with VLAN offload Ajit Khaparde
                   ` (4 subsequent siblings)
  8 siblings, 0 replies; 14+ messages in thread
From: Ajit Khaparde @ 2019-07-19  6:19 UTC (permalink / raw)
  To: dev; +Cc: ferruh.yigit, Rahul Gupta, stable

From: Rahul Gupta <rahul.gupta@broadcom.com>

Rearm will intimate hardware that current interrupts are processed
and it can continue to send more.

Fixes: 1fe427fd08ee ("net/bnxt: support enable/disable interrupt")
Cc: stable@dpdk.org

Signed-off-by: Rahul Gupta <rahul.gupta@broadcom.com>
Signed-off-by: Ajit Khaparde <ajit.khaparde@broadcom.com>
---
 drivers/net/bnxt/bnxt_rxq.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/bnxt/bnxt_rxq.c b/drivers/net/bnxt/bnxt_rxq.c
index c94cac5a9..e0eb890f8 100644
--- a/drivers/net/bnxt/bnxt_rxq.c
+++ b/drivers/net/bnxt/bnxt_rxq.c
@@ -374,7 +374,7 @@ bnxt_rx_queue_intr_enable_op(struct rte_eth_dev *eth_dev, uint16_t queue_id)
 			return rc;
 		}
 		cpr = rxq->cp_ring;
-		B_CP_DB_ARM(cpr);
+		B_CP_DB_REARM(cpr, cpr->cp_raw_cons);
 	}
 	return rc;
 }
-- 
2.20.1 (Apple Git-117)


^ permalink raw reply	[flat|nested] 14+ messages in thread

* [dpdk-dev] [PATCH 5/7] net/bnxt: disable vector mode Tx with VLAN offload
  2019-07-19  6:18 [dpdk-dev] [PATCH 0/7] bnxt patches Ajit Khaparde
                   ` (3 preceding siblings ...)
  2019-07-19  6:19 ` [dpdk-dev] [PATCH 4/7] net/bnxt: fix interrupt rearm logic Ajit Khaparde
@ 2019-07-19  6:19 ` Ajit Khaparde
  2019-07-19  6:19 ` [dpdk-dev] [PATCH 6/7] net/bnxt: remove unnecessary interrupt disable Ajit Khaparde
                   ` (3 subsequent siblings)
  8 siblings, 0 replies; 14+ messages in thread
From: Ajit Khaparde @ 2019-07-19  6:19 UTC (permalink / raw)
  To: dev; +Cc: ferruh.yigit, Lance Richardson

From: Lance Richardson <lance.richardson@broadcom.com>

The vector mode transmit path does not currently support VLAN tag
insertion, so we need to disable vector transmit when transmit
VLAN insertion offload is enabled.

Fixes: bc4a000f2f53 ("net/bnxt: implement SSE vector mode")
Reviewed-by: Ajit Khaparde <ajit.khaparde@broadcom.com>
Signed-off-by: Lance Richardson <lance.richardson@broadcom.com>
Signed-off-by: Ajit Khaparde <ajit.khaparde@broadcom.com>
---
 drivers/net/bnxt/bnxt_ethdev.c | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/drivers/net/bnxt/bnxt_ethdev.c b/drivers/net/bnxt/bnxt_ethdev.c
index fec1ecbae..d6176a686 100644
--- a/drivers/net/bnxt/bnxt_ethdev.c
+++ b/drivers/net/bnxt/bnxt_ethdev.c
@@ -743,13 +743,11 @@ bnxt_transmit_function(__rte_unused struct rte_eth_dev *eth_dev)
 {
 #ifdef RTE_ARCH_X86
 	/*
-	 * Vector mode receive can be enabled only if scatter tx is not
-	 * in use and tx offloads other than VLAN insertion are not
-	 * in use.
+	 * Vector mode transmit can be enabled only if not using scatter rx
+	 * or tx offloads.
 	 */
 	if (!eth_dev->data->scattered_rx &&
-	    !(eth_dev->data->dev_conf.txmode.offloads &
-	      ~DEV_TX_OFFLOAD_VLAN_INSERT)) {
+	    !eth_dev->data->dev_conf.txmode.offloads) {
 		PMD_DRV_LOG(INFO, "Using vector mode transmit for port %d\n",
 			    eth_dev->data->port_id);
 		return bnxt_xmit_pkts_vec;
-- 
2.20.1 (Apple Git-117)


^ permalink raw reply	[flat|nested] 14+ messages in thread

* [dpdk-dev] [PATCH 6/7] net/bnxt: remove unnecessary interrupt disable
  2019-07-19  6:18 [dpdk-dev] [PATCH 0/7] bnxt patches Ajit Khaparde
                   ` (4 preceding siblings ...)
  2019-07-19  6:19 ` [dpdk-dev] [PATCH 5/7] net/bnxt: disable vector mode Tx with VLAN offload Ajit Khaparde
@ 2019-07-19  6:19 ` Ajit Khaparde
  2019-07-19  6:19 ` [dpdk-dev] [PATCH 7/7] doc: update release information for bnxt Ajit Khaparde
                   ` (2 subsequent siblings)
  8 siblings, 0 replies; 14+ messages in thread
From: Ajit Khaparde @ 2019-07-19  6:19 UTC (permalink / raw)
  To: dev; +Cc: ferruh.yigit, stable, Kalesh AP

Remove an unnecessary rte_intr_disable() call to disable interrupt
during device init.

Fixes: c09f57b49c13 ("net/bnxt: add start/stop/link update operations")
Cc: stable@dpdk.org

Signed-off-by: Kalesh AP <kalesh-anakkur.purayil@broadcom.com>
Signed-off-by: Ajit Khaparde <ajit.khaparde@broadcom.com>
---
 drivers/net/bnxt/bnxt_ethdev.c | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/drivers/net/bnxt/bnxt_ethdev.c b/drivers/net/bnxt/bnxt_ethdev.c
index d6176a686..357ec830d 100644
--- a/drivers/net/bnxt/bnxt_ethdev.c
+++ b/drivers/net/bnxt/bnxt_ethdev.c
@@ -248,9 +248,6 @@ static int bnxt_init_chip(struct bnxt *bp)
 	unsigned int i, j;
 	int rc;
 
-	/* disable uio/vfio intr/eventfd mapping */
-	rte_intr_disable(intr_handle);
-
 	if (bp->eth_dev->data->mtu > RTE_ETHER_MTU) {
 		bp->eth_dev->data->dev_conf.rxmode.offloads |=
 			DEV_RX_OFFLOAD_JUMBO_FRAME;
-- 
2.20.1 (Apple Git-117)


^ permalink raw reply	[flat|nested] 14+ messages in thread

* [dpdk-dev] [PATCH 7/7] doc: update release information for bnxt
  2019-07-19  6:18 [dpdk-dev] [PATCH 0/7] bnxt patches Ajit Khaparde
                   ` (5 preceding siblings ...)
  2019-07-19  6:19 ` [dpdk-dev] [PATCH 6/7] net/bnxt: remove unnecessary interrupt disable Ajit Khaparde
@ 2019-07-19  6:19 ` Ajit Khaparde
  2019-07-19 21:38   ` Ferruh Yigit
  2019-07-19 12:33 ` [dpdk-dev] [PATCH 0/7] bnxt patches Ferruh Yigit
  2019-07-19 21:39 ` Ferruh Yigit
  8 siblings, 1 reply; 14+ messages in thread
From: Ajit Khaparde @ 2019-07-19  6:19 UTC (permalink / raw)
  To: dev; +Cc: ferruh.yigit

Update the HWRM version to the latest version used by the bnxt PMD
from 1.10.0.74 to 1.10.0.91

Signed-off-by: Ajit Khaparde <ajit.khaparde@broadcom.com>
---
 doc/guides/rel_notes/release_19_08.rst | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/doc/guides/rel_notes/release_19_08.rst b/doc/guides/rel_notes/release_19_08.rst
index f24e7301e..a0a7de88a 100644
--- a/doc/guides/rel_notes/release_19_08.rst
+++ b/doc/guides/rel_notes/release_19_08.rst
@@ -83,7 +83,7 @@ New Features
 
   * Performance optimizations in non-vector Tx path
   * Added support for SSE vector mode
-  * Updated HWRM API to version 1.10.0.74
+  * Updated HWRM API to version 1.10.0.91
 
 * **Added support for Broadcom NetXtreme-E BCM57500 Ethernet controllers.**
 
-- 
2.20.1 (Apple Git-117)


^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [dpdk-dev] [PATCH 0/7] bnxt patches
  2019-07-19  6:18 [dpdk-dev] [PATCH 0/7] bnxt patches Ajit Khaparde
                   ` (6 preceding siblings ...)
  2019-07-19  6:19 ` [dpdk-dev] [PATCH 7/7] doc: update release information for bnxt Ajit Khaparde
@ 2019-07-19 12:33 ` Ferruh Yigit
  2019-07-19 16:18   ` Stephen Hemminger
  2019-07-19 21:39 ` Ferruh Yigit
  8 siblings, 1 reply; 14+ messages in thread
From: Ferruh Yigit @ 2019-07-19 12:33 UTC (permalink / raw)
  To: Ajit Khaparde, dev

On 7/19/2019 7:18 AM, Ajit Khaparde wrote:
> Patchset based after an earlier submission
> https://patchwork.dpdk.org/patch/56690/
> 
> This set mostly contains bug fixes, an update to the bnxt.rst doc
> and a adds a VF PCI device ID for Thor based network controller.
> 
> Please apply.
> 
> Ajit Khaparde (2):
>   net/bnxt: remove unnecessary interrupt disable
>   doc: update release information for bnxt
> 
> Kalesh AP (1):
>   net/bnxt: add new device id
> 
> Lance Richardson (1):
>   net/bnxt: disable vector mode Tx with VLAN offload
> 
> Rahul Gupta (1):
>   net/bnxt: fix interrupt rearm logic
> 
> Santoshkumar Karanappa Rastapur (1):
>   net/bnxt: use correct number of RSS contexts for thor
> 
> Stephen Hemminger (1):
>   net/bnxt: fix receive interrupt vector
> 

Same for this patchset, it has been sent to late in the release cycle, will be
considered for next release.

Regards,
ferruh

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [dpdk-dev] [PATCH 0/7] bnxt patches
  2019-07-19 12:33 ` [dpdk-dev] [PATCH 0/7] bnxt patches Ferruh Yigit
@ 2019-07-19 16:18   ` Stephen Hemminger
  2019-07-19 16:58     ` Ferruh Yigit
  0 siblings, 1 reply; 14+ messages in thread
From: Stephen Hemminger @ 2019-07-19 16:18 UTC (permalink / raw)
  To: Ferruh Yigit; +Cc: Ajit Khaparde, dev

On Fri, 19 Jul 2019 13:33:32 +0100
Ferruh Yigit <ferruh.yigit@intel.com> wrote:

> On 7/19/2019 7:18 AM, Ajit Khaparde wrote:
> > Patchset based after an earlier submission
> > https://patchwork.dpdk.org/patch/56690/
> > 
> > This set mostly contains bug fixes, an update to the bnxt.rst doc
> > and a adds a VF PCI device ID for Thor based network controller.
> > 
> > Please apply.
> > 
> > Ajit Khaparde (2):
> >   net/bnxt: remove unnecessary interrupt disable
> >   doc: update release information for bnxt
> > 
> > Kalesh AP (1):
> >   net/bnxt: add new device id
> > 
> > Lance Richardson (1):
> >   net/bnxt: disable vector mode Tx with VLAN offload
> > 
> > Rahul Gupta (1):
> >   net/bnxt: fix interrupt rearm logic
> > 
> > Santoshkumar Karanappa Rastapur (1):
> >   net/bnxt: use correct number of RSS contexts for thor
> > 
> > Stephen Hemminger (1):
> >   net/bnxt: fix receive interrupt vector
> >   
> 
> Same for this patchset, it has been sent to late in the release cycle, will be
> considered for next release.
> 
> Regards,
> ferruh

Some of these are bug fixes. They should go in.
Also, adding new device id's should always be allowed.

Ajit, could you split the set?

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [dpdk-dev] [PATCH 0/7] bnxt patches
  2019-07-19 16:18   ` Stephen Hemminger
@ 2019-07-19 16:58     ` Ferruh Yigit
  0 siblings, 0 replies; 14+ messages in thread
From: Ferruh Yigit @ 2019-07-19 16:58 UTC (permalink / raw)
  To: Stephen Hemminger; +Cc: Ajit Khaparde, dev

On 7/19/2019 5:18 PM, Stephen Hemminger wrote:
> On Fri, 19 Jul 2019 13:33:32 +0100
> Ferruh Yigit <ferruh.yigit@intel.com> wrote:
> 
>> On 7/19/2019 7:18 AM, Ajit Khaparde wrote:
>>> Patchset based after an earlier submission
>>> https://patchwork.dpdk.org/patch/56690/
>>>
>>> This set mostly contains bug fixes, an update to the bnxt.rst doc
>>> and a adds a VF PCI device ID for Thor based network controller.
>>>
>>> Please apply.
>>>
>>> Ajit Khaparde (2):
>>>   net/bnxt: remove unnecessary interrupt disable
>>>   doc: update release information for bnxt
>>>
>>> Kalesh AP (1):
>>>   net/bnxt: add new device id
>>>
>>> Lance Richardson (1):
>>>   net/bnxt: disable vector mode Tx with VLAN offload
>>>
>>> Rahul Gupta (1):
>>>   net/bnxt: fix interrupt rearm logic
>>>
>>> Santoshkumar Karanappa Rastapur (1):
>>>   net/bnxt: use correct number of RSS contexts for thor
>>>
>>> Stephen Hemminger (1):
>>>   net/bnxt: fix receive interrupt vector
>>>   
>>
>> Same for this patchset, it has been sent to late in the release cycle, will be
>> considered for next release.
>>
>> Regards,
>> ferruh
> 
> Some of these are bug fixes. They should go in.
> Also, adding new device id's should always be allowed.
> 
> Ajit, could you split the set?
> 

Indeed I overlooked, all (for both patchset) looks like bug fixes, patchsets
with 22 and 7 patches made me think these are new features, I will get them for rc2.

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [dpdk-dev] [PATCH 2/7] net/bnxt: use correct number of RSS contexts for thor
  2019-07-19  6:19 ` [dpdk-dev] [PATCH 2/7] net/bnxt: use correct number of RSS contexts for thor Ajit Khaparde
@ 2019-07-19 21:38   ` Ferruh Yigit
  0 siblings, 0 replies; 14+ messages in thread
From: Ferruh Yigit @ 2019-07-19 21:38 UTC (permalink / raw)
  To: Ajit Khaparde, dev; +Cc: Santoshkumar Karanappa Rastapur, Lance Richardson

On 7/19/2019 7:19 AM, Ajit Khaparde wrote:
> From: Santoshkumar Karanappa Rastapur <santosh.rastapur@broadcom.com>
> 
> BCM57500-based adapters use a variable number of RSS contexts
> depending upon the number of receive rings in use. The current
> implementation is erroneously using the maximum possible number
> of RSS contexts instead of the actual number allocated when
> setting up RSS tables in the adapter. Fix by using the actual
> number of allocated contexts.
> 
> Fixes: 38412304b50a ("net/bnxt: enable RSS for thor-based controllers")
> 
> Reviewed-by: Lance Richardson <lance.richardson@broadcom.com>
> Signed-off-by: Santoshkumar Karanappa Rastapur <santosh.rastapur@broadcom.com>
> Signed-off-by: Ajit Khaparde <ajit.khaparde@broadcom.com>


Squashed into fix of same issue from previous patchset

[17/22] https://patches.dpdk.org/patch/56685/

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [dpdk-dev] [PATCH 7/7] doc: update release information for bnxt
  2019-07-19  6:19 ` [dpdk-dev] [PATCH 7/7] doc: update release information for bnxt Ajit Khaparde
@ 2019-07-19 21:38   ` Ferruh Yigit
  0 siblings, 0 replies; 14+ messages in thread
From: Ferruh Yigit @ 2019-07-19 21:38 UTC (permalink / raw)
  To: Ajit Khaparde, dev

On 7/19/2019 7:19 AM, Ajit Khaparde wrote:
> Update the HWRM version to the latest version used by the bnxt PMD
> from 1.10.0.74 to 1.10.0.91
> 
> Signed-off-by: Ajit Khaparde <ajit.khaparde@broadcom.com>

Squashed into patch that does the code update in previous patchset

[22/22] https://patches.dpdk.org/patch/56690/


^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [dpdk-dev] [PATCH 0/7] bnxt patches
  2019-07-19  6:18 [dpdk-dev] [PATCH 0/7] bnxt patches Ajit Khaparde
                   ` (7 preceding siblings ...)
  2019-07-19 12:33 ` [dpdk-dev] [PATCH 0/7] bnxt patches Ferruh Yigit
@ 2019-07-19 21:39 ` Ferruh Yigit
  8 siblings, 0 replies; 14+ messages in thread
From: Ferruh Yigit @ 2019-07-19 21:39 UTC (permalink / raw)
  To: Ajit Khaparde, dev

On 7/19/2019 7:18 AM, Ajit Khaparde wrote:
> Patchset based after an earlier submission
> https://patchwork.dpdk.org/patch/56690/
> 
> This set mostly contains bug fixes, an update to the bnxt.rst doc
> and a adds a VF PCI device ID for Thor based network controller.
> 
> Please apply.
> 
> Ajit Khaparde (2):
>   net/bnxt: remove unnecessary interrupt disable
>   doc: update release information for bnxt
> 
> Kalesh AP (1):
>   net/bnxt: add new device id
> 
> Lance Richardson (1):
>   net/bnxt: disable vector mode Tx with VLAN offload
> 
> Rahul Gupta (1):
>   net/bnxt: fix interrupt rearm logic
> 
> Santoshkumar Karanappa Rastapur (1):
>   net/bnxt: use correct number of RSS contexts for thor
> 
> Stephen Hemminger (1):
>   net/bnxt: fix receive interrupt vector

Series applied to dpdk-next-net/master, thanks.

^ permalink raw reply	[flat|nested] 14+ messages in thread

end of thread, other threads:[~2019-07-19 21:39 UTC | newest]

Thread overview: 14+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-07-19  6:18 [dpdk-dev] [PATCH 0/7] bnxt patches Ajit Khaparde
2019-07-19  6:19 ` [dpdk-dev] [PATCH 1/7] net/bnxt: add new device id Ajit Khaparde
2019-07-19  6:19 ` [dpdk-dev] [PATCH 2/7] net/bnxt: use correct number of RSS contexts for thor Ajit Khaparde
2019-07-19 21:38   ` Ferruh Yigit
2019-07-19  6:19 ` [dpdk-dev] [PATCH 3/7] net/bnxt: fix receive interrupt vector Ajit Khaparde
2019-07-19  6:19 ` [dpdk-dev] [PATCH 4/7] net/bnxt: fix interrupt rearm logic Ajit Khaparde
2019-07-19  6:19 ` [dpdk-dev] [PATCH 5/7] net/bnxt: disable vector mode Tx with VLAN offload Ajit Khaparde
2019-07-19  6:19 ` [dpdk-dev] [PATCH 6/7] net/bnxt: remove unnecessary interrupt disable Ajit Khaparde
2019-07-19  6:19 ` [dpdk-dev] [PATCH 7/7] doc: update release information for bnxt Ajit Khaparde
2019-07-19 21:38   ` Ferruh Yigit
2019-07-19 12:33 ` [dpdk-dev] [PATCH 0/7] bnxt patches Ferruh Yigit
2019-07-19 16:18   ` Stephen Hemminger
2019-07-19 16:58     ` Ferruh Yigit
2019-07-19 21:39 ` Ferruh Yigit

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).