On Fri, Mar 10, 2017 at 07:43:17PM +0000, Harry van Haaren wrote: > 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.haa...@intel.com>
Acked-by: Jerin Jacob <jerin.ja...@caviumnetworks.com> > --- > app/test/test_eventdev.c | 8 ++++++++ > 1 file changed, 8 insertions(+) > > diff --git a/app/test/test_eventdev.c b/app/test/test_eventdev.c > index 042a446..324ef5a 100644 > --- a/app/test/test_eventdev.c > +++ b/app/test/test_eventdev.c > @@ -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); > > -- > 2.7.4 >
