[1/2] test/eventdev: fix unintended vdev creation
Checks
Commit Message
From: Pavan Nikhilesh <pbhagavatula@marvell.com>
Virtual eventdevice should only be created when there is no existing
device with the same name.
Fixes: e0f4a0ed4237 ("test: skip tests when missing requirements")
Cc: stable@dpdk.org
Signed-off-by: Pavan Nikhilesh <pbhagavatula@marvell.com>
---
app/test/test_eventdev.c | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)
Comments
On Mon, Jan 6, 2020 at 1:12 PM <pbhagavatula@marvell.com> wrote:
>
> From: Pavan Nikhilesh <pbhagavatula@marvell.com>
>
> Virtual eventdevice should only be created when there is no existing
> device with the same name.
>
> Fixes: e0f4a0ed4237 ("test: skip tests when missing requirements")
> Cc: stable@dpdk.org
>
> Signed-off-by: Pavan Nikhilesh <pbhagavatula@marvell.com>
Acked-by: Jerin Jacob <jerinj@marvell.com>
Series applied to dpdk-next-eventdev/master. Thanks.
> ---
> app/test/test_eventdev.c | 6 +++++-
> 1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/app/test/test_eventdev.c b/app/test/test_eventdev.c
> index 427dbbf77..56155838d 100644
> --- a/app/test/test_eventdev.c
> +++ b/app/test/test_eventdev.c
> @@ -996,9 +996,13 @@ test_eventdev_common(void)
> static int
> test_eventdev_selftest_impl(const char *pmd, const char *opts)
> {
> - rte_vdev_init(pmd, opts);
> + int ret = 0;
> +
> if (rte_event_dev_get_dev_id(pmd) == -ENODEV)
> + ret = rte_vdev_init(pmd, opts);
> + if (ret)
> return TEST_SKIPPED;
> +
> return rte_event_dev_selftest(rte_event_dev_get_dev_id(pmd));
> }
>
> --
> 2.17.1
>
@@ -996,9 +996,13 @@ test_eventdev_common(void)
static int
test_eventdev_selftest_impl(const char *pmd, const char *opts)
{
- rte_vdev_init(pmd, opts);
+ int ret = 0;
+
if (rte_event_dev_get_dev_id(pmd) == -ENODEV)
+ ret = rte_vdev_init(pmd, opts);
+ if (ret)
return TEST_SKIPPED;
+
return rte_event_dev_selftest(rte_event_dev_get_dev_id(pmd));
}