From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 5EF1BA0613 for ; Wed, 28 Aug 2019 15:43:24 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 54FB01C206; Wed, 28 Aug 2019 15:43:24 +0200 (CEST) Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by dpdk.org (Postfix) with ESMTP id 60FFA1C229 for ; Wed, 28 Aug 2019 15:43:22 +0200 (CEST) Received: from smtp.corp.redhat.com (int-mx08.intmail.prod.int.phx2.redhat.com [10.5.11.23]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id D01C2308219E; Wed, 28 Aug 2019 13:43:21 +0000 (UTC) Received: from rh.redhat.com (ovpn-117-52.ams2.redhat.com [10.36.117.52]) by smtp.corp.redhat.com (Postfix) with ESMTP id 7B64A196AE; Wed, 28 Aug 2019 13:43:20 +0000 (UTC) From: Kevin Traynor To: Kalesh AP Cc: Ajit Khaparde , Somnath Kotur , dpdk stable Date: Wed, 28 Aug 2019 14:42:04 +0100 Message-Id: <20190828134234.20547-28-ktraynor@redhat.com> In-Reply-To: <20190828134234.20547-1-ktraynor@redhat.com> References: <20190828134234.20547-1-ktraynor@redhat.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Scanned-By: MIMEDefang 2.84 on 10.5.11.23 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.47]); Wed, 28 Aug 2019 13:43:21 +0000 (UTC) Subject: [dpdk-stable] patch 'net/bnxt: check invalid VNIC in cleanup path' has been queued to LTS release 18.11.3 X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches for DPDK stable branches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: stable-bounces@dpdk.org Sender: "stable" Hi, FYI, your patch has been queued to LTS release 18.11.3 Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet. It will be pushed if I get no objections before 09/04/19. So please shout if anyone has objections. Also note that after the patch there's a diff of the upstream commit vs the patch applied to the branch. This will indicate if there was any rebasing needed to apply to the stable branch. If there were code changes for rebasing (ie: not only metadata diffs), please double check that the rebase was correctly done. Queued patches are on a temporary branch at: https://github.com/kevintraynor/dpdk-stable-queue This queued commit can be viewed at: https://github.com/kevintraynor/dpdk-stable-queue/commit/0d1f2372d15838f922aa344d885e30e55bf92f78 Thanks. Kevin Traynor --- >From 0d1f2372d15838f922aa344d885e30e55bf92f78 Mon Sep 17 00:00:00 2001 From: Kalesh AP Date: Wed, 17 Jul 2019 16:11:36 +0530 Subject: [PATCH] net/bnxt: check invalid VNIC in cleanup path [ upstream commit fcaf844e993b5850021a7e54e27d21904b805329 ] The cleanup/rollback operation post rte_eth_dev_start failure might end up invoking an HWRM cmd even on an invalid vNIC resulting in error messages being logged needlessly. Fix to check for the same before issuing the HWRM cmd. Fixes: c09f57b49c13 ("net/bnxt: add start/stop/link update operations") Signed-off-by: Kalesh AP Signed-off-by: Ajit Khaparde Reviewed-by: Somnath Kotur --- drivers/net/bnxt/bnxt_hwrm.c | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/drivers/net/bnxt/bnxt_hwrm.c b/drivers/net/bnxt/bnxt_hwrm.c index 6eff260ca..53c31baf4 100644 --- a/drivers/net/bnxt/bnxt_hwrm.c +++ b/drivers/net/bnxt/bnxt_hwrm.c @@ -2119,4 +2119,9 @@ void bnxt_free_all_hwrm_resources(struct bnxt *bp) struct bnxt_vnic_info *vnic = &bp->vnic_info[i]; + if (vnic->fw_vnic_id == INVALID_HW_RING_ID) { + PMD_DRV_LOG(DEBUG, "Invalid vNIC ID\n"); + return; + } + bnxt_clear_hwrm_vnic_flows(bp, vnic); -- 2.20.1 --- Diff of the applied patch vs upstream commit (please double-check if non-empty: --- --- - 2019-08-28 14:32:33.453668545 +0100 +++ 0029-net-bnxt-check-invalid-VNIC-in-cleanup-path.patch 2019-08-28 14:32:31.650957160 +0100 @@ -1 +1 @@ -From fcaf844e993b5850021a7e54e27d21904b805329 Mon Sep 17 00:00:00 2001 +From 0d1f2372d15838f922aa344d885e30e55bf92f78 Mon Sep 17 00:00:00 2001 @@ -5,0 +6,2 @@ +[ upstream commit fcaf844e993b5850021a7e54e27d21904b805329 ] + @@ -12 +13,0 @@ -Cc: stable@dpdk.org @@ -22 +23 @@ -index 84cda5e6c..1a0265f8a 100644 +index 6eff260ca..53c31baf4 100644 @@ -25 +26 @@ -@@ -2326,4 +2326,9 @@ void bnxt_free_all_hwrm_resources(struct bnxt *bp) +@@ -2119,4 +2119,9 @@ void bnxt_free_all_hwrm_resources(struct bnxt *bp)