patch 'net/enic: avoid extra unlock in MTU set' has been queued to stable release 21.11.6

Kevin Traynor ktraynor at redhat.com
Thu Nov 16 14:23:03 CET 2023


Hi,

FYI, your patch has been queued to stable release 21.11.6

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

This queued commit can be viewed at:
https://github.com/kevintraynor/dpdk-stable/commit/48ce6323ad78840b36664e12c50d10e17f19c519

Thanks.

Kevin

---
>From 48ce6323ad78840b36664e12c50d10e17f19c519 Mon Sep 17 00:00:00 2001
From: Weiguo Li <liweiguo at xencore.cn>
Date: Wed, 1 Nov 2023 15:28:09 +0800
Subject: [PATCH] net/enic: avoid extra unlock in MTU set

[ upstream commit e90884a65bf6d8ed0e1e5618af42cf5856a422f1 ]

The 'set_mtu_done' goto statement is being executed in a context
where the 'mtu_lock' has not been previously locked.

To avoid the extra unlocking operation, replace the goto statement
with a return statement.

Fixes: c3e09182bcd6 ("net/enic: support scatter Rx in MTU update")

Signed-off-by: Weiguo Li <liweiguo at xencore.cn>
Reviewed-by: Ferruh Yigit <ferruh.yigit at amd.com>
---
 .mailmap                     | 2 +-
 drivers/net/enic/enic_main.c | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/.mailmap b/.mailmap
index 1111c386e2..3ba8d68383 100644
--- a/.mailmap
+++ b/.mailmap
@@ -1448,5 +1448,5 @@ Weichun Chen <weichunx.chen at intel.com>
 Wei Dai <wei.dai at intel.com>
 Weifeng Li <liweifeng96 at 126.com>
-Weiguo Li <liwg06 at foxmail.com>
+Weiguo Li <liweiguo at xencore.cn> <liwg06 at foxmail.com>
 Wei Huang <wei.huang at intel.com>
 Wei Hu (Xavier) <xavier.huwei at huawei.com>
diff --git a/drivers/net/enic/enic_main.c b/drivers/net/enic/enic_main.c
index 97d97ea793..2ea26c0407 100644
--- a/drivers/net/enic/enic_main.c
+++ b/drivers/net/enic/enic_main.c
@@ -1640,5 +1640,5 @@ int enic_set_mtu(struct enic *enic, uint16_t new_mtu)
 	 */
 	if (!eth_dev->data->dev_started)
-		goto set_mtu_done;
+		return rc;
 
 	/*
-- 
2.41.0

---
  Diff of the applied patch vs upstream commit (please double-check if non-empty:
---
--- -	2023-11-16 13:21:53.070885029 +0000
+++ 0021-net-enic-avoid-extra-unlock-in-MTU-set.patch	2023-11-16 13:21:52.441946349 +0000
@@ -1 +1 @@
-From e90884a65bf6d8ed0e1e5618af42cf5856a422f1 Mon Sep 17 00:00:00 2001
+From 48ce6323ad78840b36664e12c50d10e17f19c519 Mon Sep 17 00:00:00 2001
@@ -5,0 +6,2 @@
+[ upstream commit e90884a65bf6d8ed0e1e5618af42cf5856a422f1 ]
+
@@ -13 +14,0 @@
-Cc: stable at dpdk.org
@@ -23 +24 @@
-index 1e6d750979..c384d97a1a 100644
+index 1111c386e2..3ba8d68383 100644
@@ -26 +27 @@
-@@ -1503,5 +1503,5 @@ Weichun Chen <weichunx.chen at intel.com>
+@@ -1448,5 +1448,5 @@ Weichun Chen <weichunx.chen at intel.com>
@@ -32 +33 @@
- Wei Hu <weh at microsoft.com>
+ Wei Hu (Xavier) <xavier.huwei at huawei.com>
@@ -34 +35 @@
-index 19a99a82c5..a6aaa760ca 100644
+index 97d97ea793..2ea26c0407 100644



More information about the stable mailing list