[dpdk-stable] patch 'test/event: remove eth Rx adapter vdev workaround' has been queued to stable release 18.08.1

Kevin Traynor ktraynor at redhat.com
Wed Nov 21 17:47:18 CET 2018


Hi,

FYI, your patch has been queued to stable release 18.08.1

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/27/18. 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. If the code is different (ie: not only metadata diffs), due for example to
a change in context or macro names, please double check it.

Thanks.

Kevin Traynor

---
>From 9cd00c00ddd2ad0f244455fe68356f9de01d208b Mon Sep 17 00:00:00 2001
From: Nikhil Rao <nikhil.rao at intel.com>
Date: Thu, 6 Sep 2018 14:41:43 +0530
Subject: [PATCH] test/event: remove eth Rx adapter vdev workaround

[ upstream commit 97428d7b8e0ff103369f95dbe35a5c0f8c2ad132 ]

eth Rx adapter has been updated to support hotplugged
devices, devices created after adapter creation can now be
added to the adapter.

Update the adapter_multi_eth_add_del
test case to create the adapter as part of test setup
instead of creating it after creating vdevs.

Fixes: 2a9c83ae3b2e ("test/eventdev: add multi-ports test")

Signed-off-by: Nikhil Rao <nikhil.rao at intel.com>
Acked-by: Jerin Jacob <jerin.jacob at caviumnetworks.com>
---
 test/test/test_event_eth_rx_adapter.c | 8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)

diff --git a/test/test/test_event_eth_rx_adapter.c b/test/test/test_event_eth_rx_adapter.c
index 2337e5435..28f214687 100644
--- a/test/test/test_event_eth_rx_adapter.c
+++ b/test/test/test_event_eth_rx_adapter.c
@@ -497,7 +497,4 @@ adapter_multi_eth_add_del(void)
 	TEST_ASSERT(err == 0, "Port initialization failed err %d\n", err);
 
-	/* creating new instance for all newly added eth devices */
-	adapter_create();
-
 	/* eth_rx_adapter_queue_add for n ports */
 	port_index = 0;
@@ -517,6 +514,4 @@ adapter_multi_eth_add_del(void)
 	}
 
-	adapter_free();
-
 	return TEST_SUCCESS;
 }
@@ -683,5 +678,6 @@ static struct unit_test_suite event_eth_rx_tests = {
 		TEST_CASE_ST(adapter_create, adapter_free,
 					adapter_queue_add_del),
-		TEST_CASE_ST(NULL, NULL, adapter_multi_eth_add_del),
+		TEST_CASE_ST(adapter_create, adapter_free,
+					adapter_multi_eth_add_del),
 		TEST_CASE_ST(adapter_create, adapter_free, adapter_start_stop),
 		TEST_CASE_ST(adapter_create, adapter_free, adapter_stats),
-- 
2.19.0

---
  Diff of the applied patch vs upstream commit (please double-check if non-empty:
---
--- -	2018-11-21 16:44:31.071671627 +0000
+++ 0004-test-event-remove-eth-Rx-adapter-vdev-workaround.patch	2018-11-21 16:44:30.000000000 +0000
@@ -1,8 +1,10 @@
-From 97428d7b8e0ff103369f95dbe35a5c0f8c2ad132 Mon Sep 17 00:00:00 2001
+From 9cd00c00ddd2ad0f244455fe68356f9de01d208b Mon Sep 17 00:00:00 2001
 From: Nikhil Rao <nikhil.rao at intel.com>
 Date: Thu, 6 Sep 2018 14:41:43 +0530
 Subject: [PATCH] test/event: remove eth Rx adapter vdev workaround
 
+[ upstream commit 97428d7b8e0ff103369f95dbe35a5c0f8c2ad132 ]
+
 eth Rx adapter has been updated to support hotplugged
 devices, devices created after adapter creation can now be
 added to the adapter.
@@ -12,7 +14,6 @@
 instead of creating it after creating vdevs.
 
 Fixes: 2a9c83ae3b2e ("test/eventdev: add multi-ports test")
-Cc: stable at dpdk.org
 
 Signed-off-by: Nikhil Rao <nikhil.rao at intel.com>
 Acked-by: Jerin Jacob <jerin.jacob at caviumnetworks.com>


More information about the stable mailing list