[dpdk-stable] patch 'event/dsw: avoid reusing previously recorded events' has been queued to LTS release 18.11.9

[Kevin Traynor]

Hi,

FYI, your patch has been queued to LTS release 18.11.9

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

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/b835689b4d292674d43c4dcd11ed935312954e4d

Thanks.

Kevin.

---
>From b835689b4d292674d43c4dcd11ed935312954e4d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Mattias=20R=C3=B6nnblom?= <mattias.ronnblom at ericsson.com>
Date: Mon, 4 May 2020 11:30:59 +0200
Subject: [PATCH] event/dsw: avoid reusing previously recorded events
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

[ upstream commit 52b66b2f208aea74e5fc3b004c1e14056bb22970 ]

Avoid reusing recorded events when performing a migration, since this
may make the migration selection logic pick an already-moved flow.

Fixes: f6257b22e767 ("event/dsw: add load balancing")

Reported-by: Venky Venkatesh <vvenkatesh at paloaltonetworks.com>
Signed-off-by: Mattias Rönnblom <mattias.ronnblom at ericsson.com>
---
 drivers/event/dsw/dsw_event.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/drivers/event/dsw/dsw_event.c b/drivers/event/dsw/dsw_event.c
index 10e44fd497..f3873c54e5 100644
--- a/drivers/event/dsw/dsw_event.c
+++ b/drivers/event/dsw/dsw_event.c
@@ -659,4 +659,7 @@ dsw_port_consider_migration(struct dsw_evdev *dsw,
 		return;
 
+	if (seen_events_len < DSW_MAX_EVENTS_RECORDED)
+		return;
+
 	DSW_LOG_DP_PORT(DEBUG, source_port->id, "Considering migration.\n");
 
-- 
2.21.3

---
  Diff of the applied patch vs upstream commit (please double-check if non-empty:
---
--- -	2020-06-05 19:20:51.913567844 +0100
+++ 0018-event-dsw-avoid-reusing-previously-recorded-events.patch	2020-06-05 19:20:50.738042449 +0100
@@ -1 +1 @@
-From 52b66b2f208aea74e5fc3b004c1e14056bb22970 Mon Sep 17 00:00:00 2001
+From b835689b4d292674d43c4dcd11ed935312954e4d Mon Sep 17 00:00:00 2001
@@ -8,0 +9,2 @@
+[ upstream commit 52b66b2f208aea74e5fc3b004c1e14056bb22970 ]
+
@@ -13 +14,0 @@
-Cc: stable at dpdk.org
@@ -22 +23 @@
-index 8323903736..e5e3597aae 100644
+index 10e44fd497..f3873c54e5 100644
@@ -25 +26 @@
-@@ -822,4 +822,7 @@ dsw_port_consider_emigration(struct dsw_evdev *dsw,
+@@ -659,4 +659,7 @@ dsw_port_consider_migration(struct dsw_evdev *dsw,
@@ -31 +32 @@
- 	DSW_LOG_DP_PORT(DEBUG, source_port->id, "Considering emigration.\n");
+ 	DSW_LOG_DP_PORT(DEBUG, source_port->id, "Considering migration.\n");