From patchwork Fri Apr 30 20:14:13 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Ajit Khaparde X-Patchwork-Id: 92582 X-Patchwork-Delegate: ajit.khaparde@broadcom.com Return-Path: X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 764EBA0546; Fri, 30 Apr 2021 22:10:49 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 6627341160; Fri, 30 Apr 2021 22:10:32 +0200 (CEST) Received: from mail-pf1-f171.google.com (mail-pf1-f171.google.com [209.85.210.171]) by mails.dpdk.org (Postfix) with ESMTP id 8CA6E41150 for ; Fri, 30 Apr 2021 22:10:28 +0200 (CEST) Received: by mail-pf1-f171.google.com with SMTP id v191so7433097pfc.8 for ; Fri, 30 Apr 2021 13:10:28 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=broadcom.com; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=DNLUxuIm+XTdkVye36dXjRyOs560xx+J7vcr8WA3D80=; b=bOXVWSt7tZRszuMz19mMac3A0OPN2BKuRSEe6WQtjgsThfNFoEp0BNendFkViTHqNk iIjsc59Ko+2KsJE7x0y8QWTzavrRMODeoMICma8CUB8SYkFMz0u0US33I8Grr++8sr3S PDh0FGFtCXnsoTSzyM01CsJ2XisPbuwlioS5Q= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=DNLUxuIm+XTdkVye36dXjRyOs560xx+J7vcr8WA3D80=; b=GkmjXmV9NDkOZOMkdvauCq/7BAn7ow2NQQOYUKV9Yyj57+Bj/cdS07OMyKg2eouqQz Nqe7Jlb84UhzVO6rfsMZ7mSrzIsIzQ6tWa5mNOQvqaTV7fKKnCuExunwGe5TNa4lKfi0 k/n8SDmvVjgFkRjEw9mh6t64AKa/H0M8ZMw6KgkcHLqANYfduKy1IcOFbg3AlRNgBbe7 deAAt73T0AKJan9OjmHzB0qW/BEi39tz73WzVJR57KcSmRKjbEXG1W8idRyucHCcL8Ro zLOkndtk1MPFw1QIt7gup7C2EPZUXq8qQK8f252/TcMb74cy+Dj1SfgG1rrx3GpPRpLB 48WQ== X-Gm-Message-State: AOAM532fxTQyPLTf8MerhCTlTw4rFRscBXOWh//qip8JuhHge53HzRdX 3C+j96TC1VtPv1+B8/BXK0XFnoyNxHcnoEeX46XyVb3/ht5/MhOwT+6DOX+DgineJ3ZpNbZw/Zp gcmmfo8nc9WRodrUTiDEuViH99h5vIooVg0rqEJKngiASXxjdRR2Xqs2L8UcwtLI= X-Google-Smtp-Source: ABdhPJwu3/LcvQRCwGcFU8Y98eoplhhZrtjV36kZ4xJHl9CqJxBR1SM74FtPXl/jI9pWCCVY1jPKuA== X-Received: by 2002:a63:ff66:: with SMTP id s38mr6083018pgk.154.1619813427278; Fri, 30 Apr 2021 13:10:27 -0700 (PDT) Received: from u099.swdvt.lab.broadcom.net ([192.19.223.252]) by smtp.gmail.com with ESMTPSA id j12sm3430587pfc.17.2021.04.30.13.10.25 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Fri, 30 Apr 2021 13:10:26 -0700 (PDT) From: Ajit Khaparde To: dev@dpdk.org Cc: ferruh.yigit@intel.com, Ajit Khaparde Date: Fri, 30 Apr 2021 13:14:13 -0700 Message-Id: <1619813653-5161-5-git-send-email-ajit.khaparde@broadcom.com> X-Mailer: git-send-email 1.8.3.1 In-Reply-To: <1619813653-5161-1-git-send-email-ajit.khaparde@broadcom.com> References: <1619813653-5161-1-git-send-email-ajit.khaparde@broadcom.com> X-Content-Filtered-By: Mailman/MimeDel 2.1.29 Subject: [dpdk-dev] [PATCH 4/4] net/bnxt: fix ring count calculation X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Fix ring count calculation for Thor. VNIC count does not have a direct bearing on the number of rings that can be used. Fixes: fe8dd26f86c78 ("net/bnxt: cap max Rx rings for Thor") Signed-off-by: Ajit Khaparde --- drivers/net/bnxt/bnxt.h | 18 ++++++------------ 1 file changed, 6 insertions(+), 12 deletions(-) diff --git a/drivers/net/bnxt/bnxt.h b/drivers/net/bnxt/bnxt.h index bd2dec4..013e291 100644 --- a/drivers/net/bnxt/bnxt.h +++ b/drivers/net/bnxt/bnxt.h @@ -836,7 +836,6 @@ struct bnxt { static inline uint16_t bnxt_max_rings(struct bnxt *bp) { - struct rte_eth_conf *dev_conf = &bp->eth_dev->data->dev_conf; uint16_t max_tx_rings = bp->max_tx_rings; uint16_t max_rx_rings = bp->max_rx_rings; uint16_t max_cp_rings = bp->max_cp_rings; @@ -854,17 +853,12 @@ inline uint16_t bnxt_max_rings(struct bnxt *bp) bp->max_stat_ctx / 2U); } - if (BNXT_CHIP_P5(bp)) { - /* RSS table size in Thor is 512. - * Cap max Rx rings to the same value for RSS. - * For non-RSS case cap it to the max VNIC count. - */ - if (dev_conf->rxmode.mq_mode & ETH_MQ_RX_RSS_FLAG) - max_rx_rings = RTE_MIN(max_rx_rings, - BNXT_RSS_TBL_SIZE_P5); - else - max_rx_rings = RTE_MIN(max_rx_rings, bp->max_vnics); - } + /* + * RSS table size in Thor is 512. + * Cap max Rx rings to the same value for RSS. + */ + if (BNXT_CHIP_P5(bp)) + max_rx_rings = RTE_MIN(max_rx_rings, BNXT_RSS_TBL_SIZE_P5); max_tx_rings = RTE_MIN(max_tx_rings, max_rx_rings); if (max_cp_rings > BNXT_NUM_ASYNC_CPR(bp))