patch 'net/ice: fix L1 check interval' has been queued to stable release 21.11.6

Kevin Traynor ktraynor at redhat.com
Thu Nov 16 14:23:16 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/da268d8b4ee41218c606176f82e6d6df89dc7cd0

Thanks.

Kevin

---
>From da268d8b4ee41218c606176f82e6d6df89dc7cd0 Mon Sep 17 00:00:00 2001
From: Timothy Miskell <timothy.miskell at intel.com>
Date: Thu, 19 Oct 2023 17:26:10 +0000
Subject: [PATCH] net/ice: fix L1 check interval

[ upstream commit ff628a22c51f9cc5f69c715005a42456a2aec4f6 ]

For edge cases where the transceiver is physically inserted first and
immediately afterwards the DPDK PF is started the LSC event may occur
outside the current setting for the maximum check interval window. This
change lengthens the check interval to account for this along with other
reported cases where the link event may be longer than 1 second.

Fixes: cf911d90e366 ("net/ice: support link update")

Signed-off-by: Timothy Miskell <timothy.miskell at intel.com>
Tested-by: Jonathan Tsai <jonathan1.tsai at intel.com>
Acked-by: Qi Zhang <qi.z.zhang at intel.com>
---
 .mailmap                     | 2 ++
 drivers/net/ice/ice_ethdev.c | 4 ++--
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/.mailmap b/.mailmap
index 4e27a991ba..b92a52a280 100644
--- a/.mailmap
+++ b/.mailmap
@@ -652,4 +652,5 @@ John W. Linville <linville at tuxdriver.com>
 Jonas Pfefferle <jpf at zurich.ibm.com> <pepperjo at japf.ch>
 Jonathan Erb <jonathan.erb at banduracyber.com>
+Jonathan Tsai <jonathan1.tsai at intel.com>
 Jon DeVree <nuxi at vault24.org>
 Jon Loeliger <jdl at netgate.com>
@@ -1365,4 +1366,5 @@ Tianyu Li <tianyu.li at arm.com>
 Timmons C. Player <timmons.player at spirent.com>
 Timothy McDaniel <timothy.mcdaniel at intel.com>
+Timothy Miskell <timothy.miskell at intel.com>
 Timothy Redaelli <tredaelli at redhat.com>
 Tim Shearer <tim.shearer at overturenetworks.com>
diff --git a/drivers/net/ice/ice_ethdev.c b/drivers/net/ice/ice_ethdev.c
index 9768a6eb85..30be64ac40 100644
--- a/drivers/net/ice/ice_ethdev.c
+++ b/drivers/net/ice/ice_ethdev.c
@@ -3838,6 +3838,6 @@ static int
 ice_link_update(struct rte_eth_dev *dev, int wait_to_complete)
 {
-#define CHECK_INTERVAL 100  /* 100ms */
-#define MAX_REPEAT_TIME 10  /* 1s (10 * 100ms) in total */
+#define CHECK_INTERVAL 50  /* 50ms */
+#define MAX_REPEAT_TIME 40  /* 2s (40 * 50ms) in total */
 	struct ice_hw *hw = ICE_DEV_PRIVATE_TO_HW(dev->data->dev_private);
 	struct ice_link_status link_status;
-- 
2.41.0

---
  Diff of the applied patch vs upstream commit (please double-check if non-empty:
---
--- -	2023-11-16 13:21:53.381084151 +0000
+++ 0034-net-ice-fix-L1-check-interval.patch	2023-11-16 13:21:52.488946486 +0000
@@ -1 +1 @@
-From ff628a22c51f9cc5f69c715005a42456a2aec4f6 Mon Sep 17 00:00:00 2001
+From da268d8b4ee41218c606176f82e6d6df89dc7cd0 Mon Sep 17 00:00:00 2001
@@ -5,0 +6,2 @@
+[ upstream commit ff628a22c51f9cc5f69c715005a42456a2aec4f6 ]
+
@@ -13 +14,0 @@
-Cc: stable at dpdk.org
@@ -24 +25 @@
-index c0ab6d090b..d91775daf1 100644
+index 4e27a991ba..b92a52a280 100644
@@ -27 +28 @@
-@@ -677,4 +677,5 @@ John W. Linville <linville at tuxdriver.com>
+@@ -652,4 +652,5 @@ John W. Linville <linville at tuxdriver.com>
@@ -29 +30 @@
- Jonathan Erb <jonathan.erb at threatblockr.com> <jonathan.erb at banduracyber.com>
+ Jonathan Erb <jonathan.erb at banduracyber.com>
@@ -33 +34 @@
-@@ -1418,4 +1419,5 @@ Tianyu Li <tianyu.li at arm.com>
+@@ -1365,4 +1366,5 @@ Tianyu Li <tianyu.li at arm.com>
@@ -40 +41 @@
-index 305077e74e..6ef06b9926 100644
+index 9768a6eb85..30be64ac40 100644
@@ -43 +44 @@
-@@ -3993,6 +3993,6 @@ static int
+@@ -3838,6 +3838,6 @@ static int



More information about the stable mailing list