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 8F049A051C for ; Tue, 11 Feb 2020 12:27:50 +0100 (CET) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 7C4A11BF7C; Tue, 11 Feb 2020 12:27:50 +0100 (CET) Received: from mail-wr1-f67.google.com (mail-wr1-f67.google.com [209.85.221.67]) by dpdk.org (Postfix) with ESMTP id F0BF01BF7C for ; Tue, 11 Feb 2020 12:27:48 +0100 (CET) Received: by mail-wr1-f67.google.com with SMTP id t3so11895748wru.7 for ; Tue, 11 Feb 2020 03:27:48 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=WdUxAK85h/4aRqGqEgA/rrII680af7is4Z2ap2ymePA=; b=sEMPb3UvtyBZyYsbP8V4oLVpGAvntxg9cDRNkvUTAwkuXrza81x+AidJStY9RAE4nG 0PZIwE5EbZmofFvyuuCZEV2jxgrcdGDN2NExFCAJxQ5sCyhqzGC/wv01g2ymVzQfBioh st8FNeaO2zgr2I3DeNGNOr5+OJTpqbXqeltp4arcEVTWKe7TF0Lm6R/+iGuwmFl86pli VN7XN7RLshaqFS1GkyHzxd7wz4EVOiDyacjapGIei3l10lJSZ00HufrbfQQrExnnewfr Lc5bHC5E8mRy6EAV+4VTavl6Elax52iQeXsYpBDm4Idc33QSfzDm9m8qQ+U5cSp7f0Af x8GA== 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:mime-version:content-transfer-encoding; bh=WdUxAK85h/4aRqGqEgA/rrII680af7is4Z2ap2ymePA=; b=let3BLFndf6NL4EUvqEsBOXAqw9SeSunAoEo9LjjIGMyIJJfPrGL7krSp/BeR/vAUN l5Ffgg1rGLSHCdRBqdt8E5TzsSgR/dctxfTmkz8mcdFEnYYJ0jfl++Nkx24ite6hUvbV zgNNtUoSvj3eFyhaNgIoIbIjXbQqv0nT36h3j0grwut0rfBVrGcY+4Pp8T3kiv5ur9Rs 52h5DOGwL4k4KDCBRiv8vqh5na0FPx/1aOat/PcA6cIArKbDJ7nn8am3j4K9yV6a5aIf lg/ndY+qc6zqyKIILmdGJFAVXP2E9WmQps+1SteeLf2V1RgyEEhZr2pP8Vl/+SMgIbqK 9jgw== X-Gm-Message-State: APjAAAWO3oS7JxLdQJSHhfeRbz07Lk6yxj3WidiITs57xpPcwubC2/DQ uuhqhps2QAJac6Tmj9oUS2KevH9pfYs= X-Google-Smtp-Source: APXvYqxwjtW6SaKbxQM/V5JJVpQFDiNIZpome4XtgE24CLP0oLNWqGZr8qHnGGsghySgwhCMaXijyQ== X-Received: by 2002:adf:fa86:: with SMTP id h6mr7959426wrr.418.1581420467737; Tue, 11 Feb 2020 03:27:47 -0800 (PST) Received: from localhost ([88.98.246.218]) by smtp.gmail.com with ESMTPSA id q10sm3337774wme.16.2020.02.11.03.27.46 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 11 Feb 2020 03:27:47 -0800 (PST) From: luca.boccassi@gmail.com To: Ajit Khaparde Cc: Lance Richardson , dpdk stable Date: Tue, 11 Feb 2020 11:20:03 +0000 Message-Id: <20200211112216.3929-57-luca.boccassi@gmail.com> X-Mailer: git-send-email 2.20.1 In-Reply-To: <20200211112216.3929-1-luca.boccassi@gmail.com> References: <20200211112216.3929-1-luca.boccassi@gmail.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Subject: [dpdk-stable] patch 'net/bnxt: fix overwriting error message' has been queued to stable release 19.11.1 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 stable release 19.11.1 Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet. It will be pushed if I get no objections before 02/13/20. 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. Thanks. Luca Boccassi --- >From d82523e5dc9bbeb10702682401cbb2fcd2bb4128 Mon Sep 17 00:00:00 2001 From: Ajit Khaparde Date: Fri, 20 Dec 2019 18:42:18 -0800 Subject: [PATCH] net/bnxt: fix overwriting error message [ upstream commit 88d57889c9f8034e5075a299e4300a862da6449a ] In some cases when flow creation fails, we overwrite the specific error message with a generic error message. This patch fixes it. Fixes: d24610f7bfda ("net/bnxt: allow flow creation when RSS is enabled") Signed-off-by: Ajit Khaparde Reviewed-by: Lance Richardson --- drivers/net/bnxt/bnxt_flow.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/bnxt/bnxt_flow.c b/drivers/net/bnxt/bnxt_flow.c index 2293de2c37..c694367300 100644 --- a/drivers/net/bnxt/bnxt_flow.c +++ b/drivers/net/bnxt/bnxt_flow.c @@ -1455,7 +1455,7 @@ ret: if (rxq && !vnic->rx_queue_cnt) rxq->vnic = &bp->vnic_info[0]; } - return rc; + return -rte_errno; } static @@ -1775,7 +1775,7 @@ free_flow: rte_flow_error_set(error, 0, RTE_FLOW_ERROR_TYPE_NONE, NULL, "Flow with pattern exists, updating destination queue"); - else + else if (!rte_errno) rte_flow_error_set(error, -ret, RTE_FLOW_ERROR_TYPE_HANDLE, NULL, "Failed to create flow."); -- 2.20.1 --- Diff of the applied patch vs upstream commit (please double-check if non-empty: --- --- - 2020-02-11 11:17:41.044527327 +0000 +++ 0057-net-bnxt-fix-overwriting-error-message.patch 2020-02-11 11:17:38.436002020 +0000 @@ -1,13 +1,14 @@ -From 88d57889c9f8034e5075a299e4300a862da6449a Mon Sep 17 00:00:00 2001 +From d82523e5dc9bbeb10702682401cbb2fcd2bb4128 Mon Sep 17 00:00:00 2001 From: Ajit Khaparde Date: Fri, 20 Dec 2019 18:42:18 -0800 Subject: [PATCH] net/bnxt: fix overwriting error message +[ upstream commit 88d57889c9f8034e5075a299e4300a862da6449a ] + In some cases when flow creation fails, we overwrite the specific error message with a generic error message. This patch fixes it. Fixes: d24610f7bfda ("net/bnxt: allow flow creation when RSS is enabled") -Cc: stable@dpdk.org Signed-off-by: Ajit Khaparde Reviewed-by: Lance Richardson @@ -16,10 +17,10 @@ 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/bnxt/bnxt_flow.c b/drivers/net/bnxt/bnxt_flow.c -index 707aedcec7..cde1fa41c4 100644 +index 2293de2c37..c694367300 100644 --- a/drivers/net/bnxt/bnxt_flow.c +++ b/drivers/net/bnxt/bnxt_flow.c -@@ -1485,7 +1485,7 @@ ret: +@@ -1455,7 +1455,7 @@ ret: if (rxq && !vnic->rx_queue_cnt) rxq->vnic = &bp->vnic_info[0]; } @@ -28,7 +29,7 @@ } static -@@ -1815,7 +1815,7 @@ free_flow: +@@ -1775,7 +1775,7 @@ free_flow: rte_flow_error_set(error, 0, RTE_FLOW_ERROR_TYPE_NONE, NULL, "Flow with pattern exists, updating destination queue");