event/octeontx: remove selftest option

Since selftest now depends on dynamic mbuf fields it is not
feasible to run selftest on device probe.

Signed-off-by: Pavan Nikhilesh <pbhagavatula@marvell.com>

diff --git a/doc/guides/eventdevs/octeontx.rst b/doc/guides/eventdevs/octeontx.rst
index 79cae9f..435e1e9 100644 (file)
--- a/doc/guides/eventdevs/octeontx.rst
+++ b/doc/guides/eventdevs/octeontx.rst
@@ -63,18 +63,6 @@ Example:
     ./your_eventdev_application --vdev="event_octeontx"
 
 
-Selftest
---------
-
-The functionality of OCTEON TX eventdev can be verified using this option,
-various unit and functional tests are run to verify the sanity.
-The tests are run once the vdev creation is successfully complete.
-
-.. code-block:: console
-
-    --vdev="event_octeontx,selftest=1"
-
-
 Enable TIMvf stats
 ------------------
 TIMvf stats can be enabled by using this option, by default the stats are

diff --git a/drivers/event/octeontx/ssovf_evdev.c b/drivers/event/octeontx/ssovf_evdev.c
index 6f242aa..e60a7dc 100644 (file)
--- a/drivers/event/octeontx/ssovf_evdev.c
+++ b/drivers/event/octeontx/ssovf_evdev.c
@@ -710,8 +710,7 @@ ssovf_close(struct rte_eventdev *dev)
 }
 
 static int
-ssovf_selftest(const char *key __rte_unused, const char *value,
-               void *opaque)
+ssovf_parsekv(const char *key __rte_unused, const char *value, void *opaque)
 {
        int *flag = opaque;
        *flag = !!atoi(value);
@@ -775,10 +774,8 @@ ssovf_vdev_probe(struct rte_vdev_device *vdev)
        const char *name;
        const char *params;
        int ret;
-       int selftest = 0;
 
        static const char *const args[] = {
-               SSOVF_SELFTEST_ARG,
                TIMVF_ENABLE_STATS_ARG,
                NULL
        };
@@ -799,18 +796,9 @@ ssovf_vdev_probe(struct rte_vdev_device *vdev)
                                "Ignoring unsupported params supplied '%s'",
                                name);
                } else {
-                       int ret = rte_kvargs_process(kvlist,
-                                       SSOVF_SELFTEST_ARG,
-                                       ssovf_selftest, &selftest);
-                       if (ret != 0) {
-                               ssovf_log_err("%s: Error in selftest", name);
-                               rte_kvargs_free(kvlist);
-                               return ret;
-                       }
-
-                       ret = rte_kvargs_process(kvlist,
-                                       TIMVF_ENABLE_STATS_ARG,
-                                       ssovf_selftest, &timvf_enable_stats);
+                       ret = rte_kvargs_process(kvlist, TIMVF_ENABLE_STATS_ARG,
+                                                ssovf_parsekv,
+                                                &timvf_enable_stats);
                        if (ret != 0) {
                                ssovf_log_err("%s: Error in timvf stats", name);
                                rte_kvargs_free(kvlist);
@@ -877,8 +865,6 @@ ssovf_vdev_probe(struct rte_vdev_device *vdev)
                        edev->max_event_ports);
 
        ssovf_init_once = 1;
-       if (selftest)
-               test_eventdev_octeontx();
        return 0;
 
 error:

diff --git a/drivers/event/octeontx/ssovf_evdev.h b/drivers/event/octeontx/ssovf_evdev.h
index 90d760a..1016315 100644 (file)
--- a/drivers/event/octeontx/ssovf_evdev.h
+++ b/drivers/event/octeontx/ssovf_evdev.h
@@ -86,8 +86,6 @@
 #define SSO_GRP_GET_PRIORITY              0x7
 #define SSO_GRP_SET_PRIORITY              0x8
 
-#define SSOVF_SELFTEST_ARG               ("selftest")
-
 /*
  * In Cavium OCTEON TX SoC, all accesses to the device registers are
  * implictly strongly ordered. So, The relaxed version of IO operation is