Each eventdev pipeline needs to allow different cmdline args combination
based on pipeline type.

Signed-off-by: Pavan Nikhilesh <pbhagavat...@caviumnetworks.com>
---
 examples/eventdev_pipeline_sw_pmd/main.c           | 16 +++------
 .../eventdev_pipeline_sw_pmd/pipeline_common.h     |  4 +++
 .../pipeline_worker_generic.c                      | 40 ++++++++++++++++++++++
 3 files changed, 49 insertions(+), 11 deletions(-)

diff --git a/examples/eventdev_pipeline_sw_pmd/main.c 
b/examples/eventdev_pipeline_sw_pmd/main.c
index bd53acf76..2e80841d0 100644
--- a/examples/eventdev_pipeline_sw_pmd/main.c
+++ b/examples/eventdev_pipeline_sw_pmd/main.c
@@ -254,17 +254,11 @@ parse_app_args(int argc, char **argv)
                }
        }
 
-       if (worker_lcore_mask == 0 || rx_lcore_mask == 0 ||
-           sched_lcore_mask == 0 || tx_lcore_mask == 0) {
-               printf("Core part of pipeline was not assigned any cores. "
-                       "This will stall the pipeline, please check core masks "
-                       "(use -h for details on setting core masks):\n"
-                       "\trx: %"PRIu64"\n\ttx: %"PRIu64"\n\tsched: %"PRIu64
-                       "\n\tworkers: %"PRIu64"\n",
-                       rx_lcore_mask, tx_lcore_mask, sched_lcore_mask,
-                       worker_lcore_mask);
-               rte_exit(-1, "Fix core masks\n");
-       }
+       cdata.worker_lcore_mask = worker_lcore_mask;
+       cdata.sched_lcore_mask = sched_lcore_mask;
+       cdata.rx_lcore_mask = rx_lcore_mask;
+       cdata.tx_lcore_mask = tx_lcore_mask;
+
        if (cdata.num_stages == 0 || cdata.num_stages > MAX_NUM_STAGES)
                usage();
 
diff --git a/examples/eventdev_pipeline_sw_pmd/pipeline_common.h 
b/examples/eventdev_pipeline_sw_pmd/pipeline_common.h
index 0f3426a3a..a5837c99b 100644
--- a/examples/eventdev_pipeline_sw_pmd/pipeline_common.h
+++ b/examples/eventdev_pipeline_sw_pmd/pipeline_common.h
@@ -111,6 +111,10 @@ struct config_data {
        int16_t next_qid[MAX_NUM_STAGES+2];
        int16_t qid[MAX_NUM_STAGES];
        uint8_t rx_adapter_id;
+       uint64_t worker_lcore_mask;
+       uint64_t rx_lcore_mask;
+       uint64_t tx_lcore_mask;
+       uint64_t sched_lcore_mask;
 };
 
 struct port_link {
diff --git a/examples/eventdev_pipeline_sw_pmd/pipeline_worker_generic.c 
b/examples/eventdev_pipeline_sw_pmd/pipeline_worker_generic.c
index 032a4f2d2..a72b7b2f9 100644
--- a/examples/eventdev_pipeline_sw_pmd/pipeline_worker_generic.c
+++ b/examples/eventdev_pipeline_sw_pmd/pipeline_worker_generic.c
@@ -370,6 +370,45 @@ init_rx_adapter(uint16_t nb_ports)
                                cdata.rx_adapter_id);
 }
 
+static void
+generic_opt_check(void)
+{
+       int i;
+       int ret;
+       uint32_t cap = 0;
+       uint8_t rx_needed = 0;
+       struct rte_event_dev_info eventdev_info;
+
+       memset(&eventdev_info, 0, sizeof(struct rte_event_dev_info));
+       rte_event_dev_info_get(0, &eventdev_info);
+
+       for (i = 0; i < rte_eth_dev_count(); i++) {
+               ret = rte_event_eth_rx_adapter_caps_get(0, i, &cap);
+               if (ret)
+                       rte_exit(EXIT_FAILURE,
+                                       "failed to get event rx adapter "
+                                       "capabilities");
+               rx_needed |=
+                       !(cap & RTE_EVENT_ETH_RX_ADAPTER_CAP_INTERNAL_PORT);
+       }
+
+       if (cdata.worker_lcore_mask == 0 ||
+                       (rx_needed && cdata.rx_lcore_mask == 0) ||
+                       cdata.tx_lcore_mask == 0 || (cdata.sched_lcore_mask == 0
+                               && !(eventdev_info.event_dev_cap &
+                                       RTE_EVENT_DEV_CAP_DISTRIBUTED_SCHED))) {
+               printf("Core part of pipeline was not assigned any cores. "
+                       "This will stall the pipeline, please check core masks "
+                       "(use -h for details on setting core masks):\n"
+                       "\trx: %"PRIu64"\n\ttx: %"PRIu64"\n\tsched: %"PRIu64
+                       "\n\tworkers: %"PRIu64"\n",
+                       cdata.rx_lcore_mask, cdata.tx_lcore_mask,
+                       cdata.sched_lcore_mask,
+                       cdata.worker_lcore_mask);
+               rte_exit(-1, "Fix core masks\n");
+       }
+}
+
 void
 set_worker_generic_setup_data(struct setup_data *caps, bool burst)
 {
@@ -377,6 +416,7 @@ set_worker_generic_setup_data(struct setup_data *caps, bool 
burst)
        caps->consumer_loop = consumer_burst;
        caps->worker_loop = worker_generic_burst;
 
+       caps->opt_check = generic_opt_check;
        caps->rx_adapter_setup = init_rx_adapter;
        caps->schedule_loop = schedule_devices;
        caps->eventdev_setup = setup_eventdev_cw;
-- 
2.14.1

Reply via email to