From patchwork Sun Sep 12 15:58:57 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Ganapati Kundapura X-Patchwork-Id: 98722 X-Patchwork-Delegate: jerinj@marvell.com Return-Path: X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 6BFB5A0C45; Sun, 12 Sep 2021 17:59:11 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 8B5A2410E0; Sun, 12 Sep 2021 17:59:09 +0200 (CEST) Received: from mga05.intel.com (mga05.intel.com [192.55.52.43]) by mails.dpdk.org (Postfix) with ESMTP id 5F9364003D for ; Sun, 12 Sep 2021 17:59:05 +0200 (CEST) X-IronPort-AV: E=McAfee;i="6200,9189,10105"; a="307028352" X-IronPort-AV: E=Sophos;i="5.85,287,1624345200"; d="scan'208";a="307028352" Received: from fmsmga003.fm.intel.com ([10.253.24.29]) by fmsmga105.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 12 Sep 2021 08:59:03 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.85,287,1624345200"; d="scan'208";a="542494108" Received: from txandevlnx322.an.intel.com ([10.123.117.44]) by FMSMGA003.fm.intel.com with ESMTP; 12 Sep 2021 08:59:01 -0700 From: Ganapati Kundapura To: jay.jayatheerthan@intel.com, jerinjacobk@gmail.com Cc: dev@dpdk.org Date: Sun, 12 Sep 2021 10:58:57 -0500 Message-Id: <20210912155858.1397453-1-ganapati.kundapura@intel.com> X-Mailer: git-send-email 2.23.0 In-Reply-To: <20210912150152.1386089-2-ganapati.kundapura@intel.com> References: <20210912150152.1386089-2-ganapati.kundapura@intel.com> MIME-Version: 1.0 Subject: [dpdk-dev] [PATCH v4 1/2] eventdev: add rx queue conf get api X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Added rte_event_eth_rx_adapter_queue_conf_get() API to get rx queue information - event queue identifier, flags for handling received packets, scheduler type, event priority, polling frequency of the receive queue and flow identifier in rte_event_eth_rx_adapter_queue_conf structure Signed-off-by: Ganapati Kundapura --- v4: * squashed 1/3 and 3/3 * reused rte_event_eth_rx_adapter_queue_conf structure in place of rte_event_eth_rx_adapter_queue_info * renamed to rte_event_eth_rx_adapter_queue_info_get() to rte_event_eth_rx_adapter_queue_conf_get to align with rte_event_eth_rx_adapter_queue_conf structure v3: * Split single patch into implementaion, test and document updation patches separately v2: * Fixed build issue due to missing entry in version.map v1: * Initial patch with implementaion, test and doc together --- .../prog_guide/event_ethernet_rx_adapter.rst | 8 +++ lib/eventdev/eventdev_pmd.h | 29 +++++++++ lib/eventdev/rte_event_eth_rx_adapter.c | 75 ++++++++++++++++++++++ lib/eventdev/rte_event_eth_rx_adapter.h | 26 ++++++++ lib/eventdev/version.map | 1 + 5 files changed, 139 insertions(+) diff --git a/doc/guides/prog_guide/event_ethernet_rx_adapter.rst b/doc/guides/prog_guide/event_ethernet_rx_adapter.rst index c01e5a9..62f6413 100644 --- a/doc/guides/prog_guide/event_ethernet_rx_adapter.rst +++ b/doc/guides/prog_guide/event_ethernet_rx_adapter.rst @@ -146,6 +146,14 @@ if the callback is supported, and the counts maintained by the service function, if one exists. The service function also maintains a count of cycles for which it was not able to enqueue to the event device. +Getting Adapter queue config +~~~~~~~~~~~~~~~~~~~~~~~~~~~~ + +The ``rte_event_eth_rx_adapter_queue_conf_get()`` function reports +flags for handling received packets, event queue identifier, scheduler type, +event priority, polling frequency of the receive queue and flow identifier +in struct ``rte_event_eth_rx_adapter_queue_conf``. + Interrupt Based Rx Queues ~~~~~~~~~~~~~~~~~~~~~~~~~~ diff --git a/lib/eventdev/eventdev_pmd.h b/lib/eventdev/eventdev_pmd.h index 0f724ac..5b58385 100644 --- a/lib/eventdev/eventdev_pmd.h +++ b/lib/eventdev/eventdev_pmd.h @@ -562,6 +562,33 @@ typedef int (*eventdev_eth_rx_adapter_queue_del_t) int32_t rx_queue_id); /** + * Retrieve information about Rx queue. This callback is invoked if + * the caps returned from the eventdev_eth_rx_adapter_caps_get(, eth_port_id) + * has RTE_EVENT_ETH_RX_ADAPTER_CAP_INTERNAL_PORT set. + * + * @param dev + * Event device pointer + * + * @param eth_dev + * Ethernet device pointer + * + * @param rx_queue_id + * Ethernet device receive queue index. + * + * @param[out] queue_conf + * Pointer to rte_event_eth_rx_adapter_queue_conf structure + * + * @return + * - 0: Success + * - <0: Error code on failure. + */ +typedef int (*eventdev_eth_rx_adapter_queue_conf_get_t) + (const struct rte_eventdev *dev, + const struct rte_eth_dev *eth_dev, + uint16_t rx_queue_id, + struct rte_event_eth_rx_adapter_queue_conf *queue_conf); + +/** * Start ethernet Rx adapter. This callback is invoked if * the caps returned from eventdev_eth_rx_adapter_caps_get(.., eth_port_id) * has RTE_EVENT_ETH_RX_ADAPTER_CAP_INTERNAL_PORT set and Rx queues @@ -1107,6 +1134,8 @@ struct rte_eventdev_ops { /**< Add Rx queues to ethernet Rx adapter */ eventdev_eth_rx_adapter_queue_del_t eth_rx_adapter_queue_del; /**< Delete Rx queues from ethernet Rx adapter */ + eventdev_eth_rx_adapter_queue_conf_get_t eth_rx_adapter_queue_conf_get; + /**< Get Rx adapter queue info */ eventdev_eth_rx_adapter_start_t eth_rx_adapter_start; /**< Start ethernet Rx adapter */ eventdev_eth_rx_adapter_stop_t eth_rx_adapter_stop; diff --git a/lib/eventdev/rte_event_eth_rx_adapter.c b/lib/eventdev/rte_event_eth_rx_adapter.c index 7c94c73..de8ab05 100644 --- a/lib/eventdev/rte_event_eth_rx_adapter.c +++ b/lib/eventdev/rte_event_eth_rx_adapter.c @@ -2811,3 +2811,78 @@ rte_event_eth_rx_adapter_cb_register(uint8_t id, return 0; } + +int +rte_event_eth_rx_adapter_queue_conf_get(uint8_t id, + uint16_t eth_dev_id, + uint16_t rx_queue_id, + struct rte_event_eth_rx_adapter_queue_conf *queue_conf) +{ + struct rte_eventdev *dev; + struct eth_device_info *dev_info; + struct rte_event_eth_rx_adapter *rx_adapter; + struct eth_rx_queue_info *queue_info; + struct rte_event *qi_ev; + int ret; + uint32_t cap; + + RTE_EVENT_ETH_RX_ADAPTER_ID_VALID_OR_ERR_RET(id, -EINVAL); + RTE_ETH_VALID_PORTID_OR_ERR_RET(eth_dev_id, -EINVAL); + + if (rx_queue_id >= rte_eth_devices[eth_dev_id].data->nb_rx_queues) { + RTE_EDEV_LOG_ERR("Invalid rx queue_id %u", rx_queue_id); + return -EINVAL; + } + + if (queue_conf == NULL) { + RTE_EDEV_LOG_ERR("Rx queue conf struct cannot be NULL"); + return -EINVAL; + } + + rx_adapter = rxa_id_to_adapter(id); + if (rx_adapter == NULL) + return -EINVAL; + + dev = &rte_eventdevs[rx_adapter->eventdev_id]; + ret = rte_event_eth_rx_adapter_caps_get(rx_adapter->eventdev_id, + eth_dev_id, + &cap); + if (ret) { + RTE_EDEV_LOG_ERR("Failed to get adapter caps edev %" PRIu8 + "eth port %" PRIu16, id, eth_dev_id); + return ret; + } + + if (cap & RTE_EVENT_ETH_RX_ADAPTER_CAP_INTERNAL_PORT) { + RTE_FUNC_PTR_OR_ERR_RET( + *dev->dev_ops->eth_rx_adapter_queue_conf_get, + -ENOTSUP); + ret = (*dev->dev_ops->eth_rx_adapter_queue_conf_get)(dev, + &rte_eth_devices[eth_dev_id], + rx_queue_id, + queue_conf); + return ret; + } + + dev_info = &rx_adapter->eth_devices[eth_dev_id]; + + queue_info = &dev_info->rx_queue[rx_queue_id]; + if (!queue_info->queue_enabled) { + RTE_EDEV_LOG_ERR("Rx queue %u not added", rx_queue_id); + return -EINVAL; + } + + qi_ev = (struct rte_event *)&queue_info->event; + + memset(queue_conf, 0, sizeof(*queue_conf)); + queue_conf->rx_queue_flags = 0; + if (queue_info->flow_id_mask != 0) + queue_conf->rx_queue_flags |= + RTE_EVENT_ETH_RX_ADAPTER_QUEUE_FLOW_ID_VALID; + + queue_conf->servicing_weight = queue_info->wt; + + memcpy(&queue_conf->ev, qi_ev, sizeof(*qi_ev)); + + return 0; +} diff --git a/lib/eventdev/rte_event_eth_rx_adapter.h b/lib/eventdev/rte_event_eth_rx_adapter.h index 182dd2e..2f24c34 100644 --- a/lib/eventdev/rte_event_eth_rx_adapter.h +++ b/lib/eventdev/rte_event_eth_rx_adapter.h @@ -33,6 +33,7 @@ * - rte_event_eth_rx_adapter_stop() * - rte_event_eth_rx_adapter_stats_get() * - rte_event_eth_rx_adapter_stats_reset() + * - rte_event_eth_rx_adapter_queue_conf_get() * * The application creates an ethernet to event adapter using * rte_event_eth_rx_adapter_create_ext() or rte_event_eth_rx_adapter_create() @@ -575,6 +576,31 @@ int rte_event_eth_rx_adapter_queue_event_vector_config( uint8_t id, uint16_t eth_dev_id, int32_t rx_queue_id, struct rte_event_eth_rx_adapter_event_vector_config *config); +/** + * Retrieve Rx queue config information. + * + * @param id + * Adapter identifier. + + * @param eth_dev_id + * Port identifier of Ethernet device. + + * @param rx_queue_id + * Ethernet device receive queue index. + + * @param[out] queue_conf + * Pointer to struct rte_event_eth_rx_adapter_queue_conf + + * @return + * - 0: Success, Receive queue added correctly. + * - <0: Error code on failure. + */ +__rte_experimental +int rte_event_eth_rx_adapter_queue_conf_get(uint8_t id, + uint16_t eth_dev_id, + uint16_t rx_queue_id, + struct rte_event_eth_rx_adapter_queue_conf *queue_conf); + #ifdef __cplusplus } #endif diff --git a/lib/eventdev/version.map b/lib/eventdev/version.map index 8862562..5562c78 100644 --- a/lib/eventdev/version.map +++ b/lib/eventdev/version.map @@ -143,6 +143,7 @@ EXPERIMENTAL { rte_event_vector_pool_create; rte_event_eth_rx_adapter_vector_limits_get; rte_event_eth_rx_adapter_queue_event_vector_config; + rte_event_eth_rx_adapter_queue_conf_get; __rte_eventdev_trace_crypto_adapter_enqueue; }; From patchwork Sun Sep 12 15:58:58 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Ganapati Kundapura X-Patchwork-Id: 98721 X-Patchwork-Delegate: jerinj@marvell.com Return-Path: X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 079A8A0C45; Sun, 12 Sep 2021 17:59:05 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 87D7940042; Sun, 12 Sep 2021 17:59:05 +0200 (CEST) Received: from mga05.intel.com (mga05.intel.com [192.55.52.43]) by mails.dpdk.org (Postfix) with ESMTP id B926D4003D for ; Sun, 12 Sep 2021 17:59:04 +0200 (CEST) X-IronPort-AV: E=McAfee;i="6200,9189,10105"; a="307028350" X-IronPort-AV: E=Sophos;i="5.85,287,1624345200"; d="scan'208";a="307028350" Received: from fmsmga003.fm.intel.com ([10.253.24.29]) by fmsmga105.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 12 Sep 2021 08:59:03 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.85,287,1624345200"; d="scan'208";a="542494104" Received: from txandevlnx322.an.intel.com ([10.123.117.44]) by FMSMGA003.fm.intel.com with ESMTP; 12 Sep 2021 08:59:03 -0700 From: Ganapati Kundapura To: jay.jayatheerthan@intel.com, jerinjacobk@gmail.com Cc: dev@dpdk.org Date: Sun, 12 Sep 2021 10:58:58 -0500 Message-Id: <20210912155858.1397453-2-ganapati.kundapura@intel.com> X-Mailer: git-send-email 2.23.0 In-Reply-To: <20210912155858.1397453-1-ganapati.kundapura@intel.com> References: <20210912150152.1386089-2-ganapati.kundapura@intel.com> <20210912155858.1397453-1-ganapati.kundapura@intel.com> MIME-Version: 1.0 Subject: [dpdk-dev] [PATCH v4 2/2] test/event: Add rx queue conf get test in rx adapter autotest X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Add unit tests for rte_event_eth_rx_adapter_queue_conf_get() in rx adapter autotest Signed-off-by: Ganapati Kundapura --- app/test/test_event_eth_rx_adapter.c | 22 ++++++++++++++++++++++ 1 file changed, 22 insertions(+) diff --git a/app/test/test_event_eth_rx_adapter.c b/app/test/test_event_eth_rx_adapter.c index 9198767..ac6cb18 100644 --- a/app/test/test_event_eth_rx_adapter.c +++ b/app/test/test_event_eth_rx_adapter.c @@ -750,6 +750,27 @@ adapter_stats(void) return TEST_SUCCESS; } +static int +adapter_queue_conf(void) +{ + int err; + struct rte_event_eth_rx_adapter_queue_conf queue_conf; + + err = rte_event_eth_rx_adapter_queue_conf_get(TEST_INST_ID, TEST_DEV_ID, + 0, &queue_conf); + TEST_ASSERT(err == 0, "Expected 0 got %d", err); + + err = rte_event_eth_rx_adapter_queue_conf_get(TEST_INST_ID, TEST_DEV_ID, + -1, &queue_conf); + TEST_ASSERT(err == -EINVAL, "Expected -EINVAL got %d", err); + + err = rte_event_eth_rx_adapter_queue_conf_get(TEST_INST_ID, TEST_DEV_ID, + 0, NULL); + TEST_ASSERT(err == -EINVAL, "Expected -EINVAL got %d", err); + + return TEST_SUCCESS; +} + static struct unit_test_suite event_eth_rx_tests = { .suite_name = "rx event eth adapter test suite", .setup = testsuite_setup, @@ -762,6 +783,7 @@ static struct unit_test_suite event_eth_rx_tests = { adapter_multi_eth_add_del), TEST_CASE_ST(adapter_create, adapter_free, adapter_start_stop), TEST_CASE_ST(adapter_create, adapter_free, adapter_stats), + TEST_CASE_ST(adapter_create, adapter_free, adapter_queue_conf), TEST_CASES_END() /**< NULL terminate unit test array */ } };