[dpdk-dev] [PATCH v3 04/24] net/bnxt: fix calculation of number of pools

[Ajit Khaparde]

The calculation for number of pools is wrong.
We are wrongly overwriting the calculated value with ETH_64_POOLS.
Accordingly fix the size of ff_pools array.
Fix the log message as well.

Fixes: 4cfe399f6550 ("net/bnxt: support to set VF rxmode")

Signed-off-by: Ajit Khaparde <ajit.khapa...@broadcom.com>
---
 drivers/net/bnxt/bnxt.h     | 2 +-
 drivers/net/bnxt/bnxt_rxq.c | 3 +--
 2 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/drivers/net/bnxt/bnxt.h b/drivers/net/bnxt/bnxt.h
index 405d94deb..357950509 100644
--- a/drivers/net/bnxt/bnxt.h
+++ b/drivers/net/bnxt/bnxt.h
@@ -217,7 +217,7 @@ struct bnxt {
        STAILQ_HEAD(, bnxt_filter_info) free_filter_list;
 
        /* VNIC pointer for flow filter (VMDq) pools */
-#define MAX_FF_POOLS   ETH_64_POOLS
+#define MAX_FF_POOLS   256
        STAILQ_HEAD(, bnxt_vnic_info)   ff_pool[MAX_FF_POOLS];
 
        struct bnxt_irq         *irq_tbl;
diff --git a/drivers/net/bnxt/bnxt_rxq.c b/drivers/net/bnxt/bnxt_rxq.c
index ef5e47d4f..441e543eb 100644
--- a/drivers/net/bnxt/bnxt_rxq.c
+++ b/drivers/net/bnxt/bnxt_rxq.c
@@ -125,8 +125,7 @@ int bnxt_mq_rx_configure(struct bnxt *bp)
                            RTE_MIN(bp->max_l2_ctx,
                             RTE_MIN(bp->max_rsscos_ctx, ETH_64_POOLS)));
                        RTE_LOG(ERR, PMD,
-                               "VMDq pool not set, defaulted to 64\n");
-                       pools = ETH_64_POOLS;
+                               "VMDq pool not set, defaulted to %d\n", pools);
                }
                nb_q_per_grp = bp->rx_cp_nr_rings / pools;
                start_grp_id = 0;
-- 
2.13.5 (Apple Git-94)