d/numam-dpdk
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

eventdev: check timer adapter status before start

Browse Source 
Check if timer adapter is already started before starting it.
Update the unit test accordingly.

Signed-off-by: Pavan Nikhilesh <pbhagavatula@marvell.com>
Acked-by: Erik Gabriel Carrillo <erik.g.carrillo@intel.com>
This commit is contained in:
Pavan Nikhilesh 2019-03-16 20:27:36 +00:00 committed by Jerin Jacob
parent 4b3997680a
commit d69d085883
3 changed files with 9 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
app/test/test_event_timer_adapter.c
View File

@ -1015,8 +1015,8 @@ adapter_start(void)
TEST_ASSERT_SUCCESS(_timdev_setup(180 * NSECPERSEC,
NSECPERSEC / 10),
"Failed to start adapter");
TEST_ASSERT_SUCCESS(rte_event_timer_adapter_start(timdev),
"Failed to repeatedly start adapter");
TEST_ASSERT_EQUAL(rte_event_timer_adapter_start(timdev), -EALREADY,
"Timer adapter started without call to stop.");
return TEST_SUCCESS;
}

6
lib/librte_eventdev/rte_event_timer_adapter.c
View File

@ -261,6 +261,12 @@ rte_event_timer_adapter_start(const struct rte_event_timer_adapter *adapter)
ADAPTER_VALID_OR_ERR_RET(adapter, -EINVAL);
FUNC_PTR_OR_ERR_RET(adapter->ops->start, -EINVAL);
if (adapter->data->started) {
EVTIM_LOG_ERR("event timer adapter %"PRIu8" already started",
adapter->data->id);
return -EALREADY;
}
ret = adapter->ops->start(adapter);
if (ret < 0)
return ret;

1
lib/librte_eventdev/rte_event_timer_adapter.h
View File

@ -339,6 +339,7 @@ rte_event_timer_adapter_get_info(
* - -EINVAL if adapter identifier invalid
* - -ENOENT if software adapter but no service core mapped
* - -ENOTSUP if software adapter and more than one service core mapped
* - -EALREADY if adapter has already been started
*
* @note
* The eventdev to which the event_timer_adapter is connected needs to