[dpdk-stable] patch 'net/bnxt: fix log message level' has been queued to LTS release 17.11.10

luca.boccassi at gmail.com luca.boccassi at gmail.com
Thu Dec 19 15:34:34 CET 2019


Hi,

FYI, your patch has been queued to LTS release 17.11.10

Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet.
It will be pushed if I get no objections before 12/21/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.

Thanks.

Luca Boccassi

---
>From 0a5547a58201ef66f9fe13cb30ec62eb43f69043 Mon Sep 17 00:00:00 2001
From: Venkat Duvvuru <venkatkumar.duvvuru at broadcom.com>
Date: Wed, 13 Nov 2019 13:59:44 +0530
Subject: [PATCH] net/bnxt: fix log message level

[ upstream commit ae6a941dd9be62a783a8d4e1bafa5df894401936 ]

When an existing mac_addr is tried to get programmed again, a
message is displayed that the mac_addr already exists.
However the message is of type ERR. This patch changes the message
to type DEBUG

Fixes: 938a87db4324 ("net/bnxt: fix redundant MAC address check")

Signed-off-by: Venkat Duvvuru <venkatkumar.duvvuru at broadcom.com>
Reviewed-by: Somnath Kotur <somnath.kotur at broadcom.com>
Reviewed-by: Ajit Khaparde <ajit.khaparde at broadcom.com>
---
 drivers/net/bnxt/bnxt_ethdev.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/bnxt/bnxt_ethdev.c b/drivers/net/bnxt/bnxt_ethdev.c
index 7747c20a58..e73a683bfa 100644
--- a/drivers/net/bnxt/bnxt_ethdev.c
+++ b/drivers/net/bnxt/bnxt_ethdev.c
@@ -715,7 +715,7 @@ static int bnxt_mac_addr_add_op(struct rte_eth_dev *eth_dev,
 	/* Attach requested MAC address to the new l2_filter */
 	STAILQ_FOREACH(filter, &vnic->filter, next) {
 		if (filter->mac_index == index) {
-			RTE_LOG(ERR, PMD,
+			RTE_LOG(DEBUG, PMD,
 				"MAC addr already existed for pool %d\n", pool);
 			return 0;
 		}
-- 
2.20.1

---
  Diff of the applied patch vs upstream commit (please double-check if non-empty:
---
--- -	2019-12-19 14:32:31.381052189 +0000
+++ 0127-net-bnxt-fix-log-message-level.patch	2019-12-19 14:32:26.329302267 +0000
@@ -1,15 +1,16 @@
-From ae6a941dd9be62a783a8d4e1bafa5df894401936 Mon Sep 17 00:00:00 2001
+From 0a5547a58201ef66f9fe13cb30ec62eb43f69043 Mon Sep 17 00:00:00 2001
 From: Venkat Duvvuru <venkatkumar.duvvuru at broadcom.com>
 Date: Wed, 13 Nov 2019 13:59:44 +0530
 Subject: [PATCH] net/bnxt: fix log message level
 
+[ upstream commit ae6a941dd9be62a783a8d4e1bafa5df894401936 ]
+
 When an existing mac_addr is tried to get programmed again, a
 message is displayed that the mac_addr already exists.
 However the message is of type ERR. This patch changes the message
 to type DEBUG
 
 Fixes: 938a87db4324 ("net/bnxt: fix redundant MAC address check")
-Cc: stable at dpdk.org
 
 Signed-off-by: Venkat Duvvuru <venkatkumar.duvvuru at broadcom.com>
 Reviewed-by: Somnath Kotur <somnath.kotur at broadcom.com>
@@ -19,18 +20,18 @@
  1 file changed, 1 insertion(+), 1 deletion(-)
 
 diff --git a/drivers/net/bnxt/bnxt_ethdev.c b/drivers/net/bnxt/bnxt_ethdev.c
-index b00966942b..393706b800 100644
+index 7747c20a58..e73a683bfa 100644
 --- a/drivers/net/bnxt/bnxt_ethdev.c
 +++ b/drivers/net/bnxt/bnxt_ethdev.c
-@@ -1024,7 +1024,7 @@ static int bnxt_add_mac_filter(struct bnxt *bp, struct bnxt_vnic_info *vnic,
+@@ -715,7 +715,7 @@ static int bnxt_mac_addr_add_op(struct rte_eth_dev *eth_dev,
  	/* Attach requested MAC address to the new l2_filter */
  	STAILQ_FOREACH(filter, &vnic->filter, next) {
  		if (filter->mac_index == index) {
--			PMD_DRV_LOG(ERR,
-+			PMD_DRV_LOG(DEBUG,
- 				    "MAC addr already existed for pool %d\n",
- 				    pool);
+-			RTE_LOG(ERR, PMD,
++			RTE_LOG(DEBUG, PMD,
+ 				"MAC addr already existed for pool %d\n", pool);
  			return 0;
+ 		}
 -- 
 2.20.1
 


More information about the stable mailing list