[dpdk-stable] patch 'test/distributor: fix worker notification in burst mode' has been queued to stable release 19.11.9

Christian Ehrhardt christian.ehrhardt at canonical.com
Mon May 17 18:10:18 CEST 2021


Hi,

FYI, your patch has been queued to stable release 19.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 05/19/21. 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/cpaelzer/dpdk-stable-queue

This queued commit can be viewed at:
https://github.com/cpaelzer/dpdk-stable-queue/commit/769379e5994aa2a84c69745f621771d2a90eba2b

Thanks.

Christian Ehrhardt <christian.ehrhardt at canonical.com>

---
>From 769379e5994aa2a84c69745f621771d2a90eba2b Mon Sep 17 00:00:00 2001
From: Stanislaw Kardach <kda at semihalf.com>
Date: Wed, 28 Apr 2021 16:25:52 +0200
Subject: [PATCH] test/distributor: fix worker notification in burst mode

[ upstream commit 00dac9a99c783e0d442e73842ebb006f2f39c59d ]

Because a single worker can process more than one packet from the
distributor, the final set of notifications in burst mode should be
sent one-by-one to ensure that each worker has a chance to wake up.

This fix mirrors the change done in the functional test by
commit f72bff0ec272 ("test/distributor: fix quitting workers in burst
mode").

Fixes: c3eabff124e6 ("distributor: add unit tests")

Signed-off-by: Stanislaw Kardach <kda at semihalf.com>
Acked-by: David Hunt <david.hunt at intel.com>
Tested-by: Lukasz Wojciechowski <l.wojciechow at partner.samsung.com>
Reviewed-by: Lukasz Wojciechowski <l.wojciechow at partner.samsung.com>
---
 app/test/test_distributor_perf.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/app/test/test_distributor_perf.c b/app/test/test_distributor_perf.c
index f153bcf9bd..0f3c3d84c1 100644
--- a/app/test/test_distributor_perf.c
+++ b/app/test/test_distributor_perf.c
@@ -188,9 +188,10 @@ quit_workers(struct rte_distributor *d, struct rte_mempool *p)
 	rte_mempool_get_bulk(p, (void *)bufs, num_workers);
 
 	quit = 1;
-	for (i = 0; i < num_workers; i++)
+	for (i = 0; i < num_workers; i++) {
 		bufs[i]->hash.usr = i << 1;
-	rte_distributor_process(d, bufs, num_workers);
+		rte_distributor_process(d, &bufs[i], 1);
+	}
 
 	rte_mempool_put_bulk(p, (void *)bufs, num_workers);
 
-- 
2.31.1

---
  Diff of the applied patch vs upstream commit (please double-check if non-empty:
---
--- -	2021-05-17 17:40:36.794914076 +0200
+++ 0189-test-distributor-fix-worker-notification-in-burst-mo.patch	2021-05-17 17:40:29.547812436 +0200
@@ -1 +1 @@
-From 00dac9a99c783e0d442e73842ebb006f2f39c59d Mon Sep 17 00:00:00 2001
+From 769379e5994aa2a84c69745f621771d2a90eba2b Mon Sep 17 00:00:00 2001
@@ -5,0 +6,2 @@
+[ upstream commit 00dac9a99c783e0d442e73842ebb006f2f39c59d ]
+
@@ -15 +16,0 @@
-Cc: stable at dpdk.org
@@ -26 +27 @@
-index b25f79a348..371a14ba4c 100644
+index f153bcf9bd..0f3c3d84c1 100644


More information about the stable mailing list