test/eventdev: link all queues before start
authorHarry van Haaren <harry.van.haaren@intel.com>
Fri, 10 Mar 2017 19:43:17 +0000 (19:43 +0000)
committerJerin Jacob <jerin.jacob@caviumnetworks.com>
Tue, 4 Apr 2017 17:19:51 +0000 (19:19 +0200)
The software eventdev can lock-up if not all queues are
linked to a port. For this reason, the software evendev
fails to start if queues are not linked to anything.

This commit creates dummy links from all queues to port
0 in the eventdev setup function and start/stop test,
which would otherwise fail due to unlinked queues.

Signed-off-by: Harry van Haaren <harry.van.haaren@intel.com>
Acked-by: Jerin Jacob <jerin.jacob@caviumnetworks.com>
test/test/test_eventdev.c

index e087af2..0f1deb6 100644 (file)
@@ -543,6 +543,10 @@ test_eventdev_start_stop(void)
                TEST_ASSERT_SUCCESS(ret, "Failed to setup port%d", i);
        }
 
+       ret = rte_event_port_link(TEST_DEV_ID, 0, NULL, NULL, 0);
+       TEST_ASSERT(ret == rte_event_queue_count(TEST_DEV_ID),
+                       "Failed to link port, device %d", TEST_DEV_ID);
+
        ret = rte_event_dev_start(TEST_DEV_ID);
        TEST_ASSERT_SUCCESS(ret, "Failed to start device%d", TEST_DEV_ID);
 
@@ -569,6 +573,10 @@ eventdev_setup_device(void)
                TEST_ASSERT_SUCCESS(ret, "Failed to setup port%d", i);
        }
 
+       ret = rte_event_port_link(TEST_DEV_ID, 0, NULL, NULL, 0);
+       TEST_ASSERT(ret == rte_event_queue_count(TEST_DEV_ID),
+                       "Failed to link port, device %d", TEST_DEV_ID);
+
        ret = rte_event_dev_start(TEST_DEV_ID);
        TEST_ASSERT_SUCCESS(ret, "Failed to start device%d", TEST_DEV_ID);