From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 8780EA00BE; Fri, 12 Jun 2020 23:26:30 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 380A91BF31; Fri, 12 Jun 2020 23:26:22 +0200 (CEST) Received: from mga07.intel.com (mga07.intel.com [134.134.136.100]) by dpdk.org (Postfix) with ESMTP id F1D621BED9 for ; Fri, 12 Jun 2020 23:26:19 +0200 (CEST) IronPort-SDR: xetu9DQjiHmgD14QV2eaM3J6EPuFSS9XuY/Fp1TqURPYQTr1dnxAJOPOjsFw902gpZ43FmmnMx McY62xTNPfnA== X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga003.jf.intel.com ([10.7.209.27]) by orsmga105.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 12 Jun 2020 14:26:19 -0700 IronPort-SDR: qjnrd32dLRqNtAPr+aKDR/7Qp9pPB5HCNvoh8GajSiMuuAPlIenIQKQv3n13EIJfNoulrXW83e XEEbcSFGjWHQ== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.73,504,1583222400"; d="scan'208";a="272035764" Received: from txasoft-yocto.an.intel.com ([10.123.72.192]) by orsmga003.jf.intel.com with ESMTP; 12 Jun 2020 14:26:18 -0700 From: "McDaniel, Timothy" To: jerinj@marvell.com Cc: dev@dpdk.org, gage.eads@intel.com, harry.van.haaren@intel.com Date: Fri, 12 Jun 2020 16:24:09 -0500 Message-Id: <20200612212434.6852-3-timothy.mcdaniel@intel.com> X-Mailer: git-send-email 2.13.6 In-Reply-To: <20200612212434.6852-1-timothy.mcdaniel@intel.com> References: <20200612212434.6852-1-timothy.mcdaniel@intel.com> Subject: [dpdk-dev] [PATCH 02/27] eventdev: do not pass disable_implicit_release bit to trace macro X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 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" The disable implicit_release indicator is now a bitfield, so it must be converted to uint8_t when calling the tracepoint macro. Change-Id: Iab83bbc295de2431a936ce2a73ed6801366d392f Signed-off-by: McDaniel, Timothy --- lib/librte_eventdev/rte_event_crypto_adapter.c | 7 ++++++- lib/librte_eventdev/rte_event_eth_tx_adapter.c | 14 ++++++++++++-- lib/librte_eventdev/rte_eventdev.c | 10 +++++++--- lib/librte_eventdev/rte_eventdev_trace.h | 15 +++++++++------ 4 files changed, 34 insertions(+), 12 deletions(-) diff --git a/lib/librte_eventdev/rte_event_crypto_adapter.c b/lib/librte_eventdev/rte_event_crypto_adapter.c index 8f2593316..1b56f2eb4 100644 --- a/lib/librte_eventdev/rte_event_crypto_adapter.c +++ b/lib/librte_eventdev/rte_event_crypto_adapter.c @@ -205,8 +205,10 @@ rte_event_crypto_adapter_create_ext(uint8_t id, uint8_t dev_id, struct rte_event_crypto_adapter *adapter; char mem_name[CRYPTO_ADAPTER_NAME_LEN]; struct rte_event_dev_info dev_info; + struct rte_event_port_conf *port_conf = conf_arg; int socket_id; uint8_t i; + uint8_t disable_impl_rel; int ret; EVENT_CRYPTO_ADAPTER_ID_VALID_OR_ERR_RET(id, -EINVAL); @@ -268,8 +270,11 @@ rte_event_crypto_adapter_create_ext(uint8_t id, uint8_t dev_id, event_crypto_adapter[id] = adapter; + disable_impl_rel = !!(port_conf->event_port_cfg & + RTE_EVENT_PORT_CFG_DISABLE_IMPL_REL); + rte_eventdev_trace_crypto_adapter_create(id, dev_id, adapter, conf_arg, - mode); + disable_impl_rel, mode); return 0; } diff --git a/lib/librte_eventdev/rte_event_eth_tx_adapter.c b/lib/librte_eventdev/rte_event_eth_tx_adapter.c index 8a72256de..e872f7c8c 100644 --- a/lib/librte_eventdev/rte_event_eth_tx_adapter.c +++ b/lib/librte_eventdev/rte_event_eth_tx_adapter.c @@ -910,6 +910,7 @@ rte_event_eth_tx_adapter_create(uint8_t id, uint8_t dev_id, struct rte_event_port_conf *port_conf) { struct rte_eventdev *dev; + uint8_t disable_impl_rel; int ret; if (port_conf == NULL) @@ -943,8 +944,12 @@ rte_event_eth_tx_adapter_create(uint8_t id, uint8_t dev_id, txa_dev_id_array[id] = TXA_INVALID_DEV_ID; return ret; } + + disable_impl_rel = !!(port_conf->event_port_cfg & + RTE_EVENT_PORT_CFG_DISABLE_IMPL_REL); + rte_eventdev_trace_eth_tx_adapter_create(id, dev_id, NULL, port_conf, - ret); + disable_impl_rel, ret); txa_dev_id_array[id] = dev_id; return 0; } @@ -955,6 +960,8 @@ rte_event_eth_tx_adapter_create_ext(uint8_t id, uint8_t dev_id, void *conf_arg) { struct rte_eventdev *dev; + struct rte_event_port_conf *port_conf = conf_arg; + uint8_t disable_impl_rel; int ret; RTE_EVENT_ETH_TX_ADAPTER_ID_VALID_OR_ERR_RET(id, -EINVAL); @@ -986,8 +993,11 @@ rte_event_eth_tx_adapter_create_ext(uint8_t id, uint8_t dev_id, return ret; } + disable_impl_rel = !!(port_conf->event_port_cfg & + RTE_EVENT_PORT_CFG_DISABLE_IMPL_REL); + rte_eventdev_trace_eth_tx_adapter_create(id, dev_id, conf_cb, conf_arg, - ret); + disable_impl_rel, ret); txa_dev_id_array[id] = dev_id; return 0; } diff --git a/lib/librte_eventdev/rte_eventdev.c b/lib/librte_eventdev/rte_eventdev.c index 4955ab1a0..67c9c8357 100644 --- a/lib/librte_eventdev/rte_eventdev.c +++ b/lib/librte_eventdev/rte_eventdev.c @@ -736,6 +736,7 @@ rte_event_port_setup(uint8_t dev_id, uint8_t port_id, { struct rte_eventdev *dev; struct rte_event_port_conf def_conf; + uint8_t disable_impl_rel; int diag; RTE_EVENTDEV_VALID_DEVID_OR_ERR_RET(dev_id, -EINVAL); @@ -812,7 +813,12 @@ rte_event_port_setup(uint8_t dev_id, uint8_t port_id, if (!diag) diag = rte_event_port_unlink(dev_id, port_id, NULL, 0); - rte_eventdev_trace_port_setup(dev_id, port_id, port_conf, diag); + disable_impl_rel = !!(port_conf->event_port_cfg & + RTE_EVENT_PORT_CFG_DISABLE_IMPL_REL); + + rte_eventdev_trace_port_setup(dev_id, port_id, port_conf, + disable_impl_rel, diag); + if (diag < 0) return diag; @@ -1422,10 +1428,8 @@ rte_event_pmd_allocate(const char *name, int socket_id) eventdev->data = eventdev_data; if (rte_eal_process_type() == RTE_PROC_PRIMARY) { - strlcpy(eventdev->data->name, name, RTE_EVENTDEV_NAME_MAX_LEN); - eventdev->data->dev_id = dev_id; eventdev->data->socket_id = socket_id; eventdev->data->dev_started = 0; diff --git a/lib/librte_eventdev/rte_eventdev_trace.h b/lib/librte_eventdev/rte_eventdev_trace.h index 4de6341ca..f56063976 100644 --- a/lib/librte_eventdev/rte_eventdev_trace.h +++ b/lib/librte_eventdev/rte_eventdev_trace.h @@ -53,13 +53,14 @@ RTE_TRACE_POINT( RTE_TRACE_POINT( rte_eventdev_trace_port_setup, RTE_TRACE_POINT_ARGS(uint8_t dev_id, uint8_t port_id, - const struct rte_event_port_conf *port_conf, int rc), + const struct rte_event_port_conf *port_conf, + uint8_t disable_impl_rel, int rc), rte_trace_point_emit_u8(dev_id); rte_trace_point_emit_u8(port_id); rte_trace_point_emit_i32(port_conf->new_event_threshold); rte_trace_point_emit_u16(port_conf->dequeue_depth); rte_trace_point_emit_u16(port_conf->enqueue_depth); - rte_trace_point_emit_u8(port_conf->disable_implicit_release); + rte_trace_point_emit_u8(disable_impl_rel); rte_trace_point_emit_int(rc); ) @@ -159,13 +160,14 @@ RTE_TRACE_POINT( RTE_TRACE_POINT( rte_eventdev_trace_eth_tx_adapter_create, RTE_TRACE_POINT_ARGS(uint8_t adptr_id, uint8_t dev_id, void *conf_cb, - struct rte_event_port_conf *port_conf, int rc), + struct rte_event_port_conf *port_conf, uint8_t disable_impl_rel, + int rc), rte_trace_point_emit_u8(adptr_id); rte_trace_point_emit_u8(dev_id); rte_trace_point_emit_i32(port_conf->new_event_threshold); rte_trace_point_emit_u16(port_conf->dequeue_depth); rte_trace_point_emit_u16(port_conf->enqueue_depth); - rte_trace_point_emit_u8(port_conf->disable_implicit_release); + rte_trace_point_emit_u8(disable_impl_rel); rte_trace_point_emit_ptr(conf_cb); rte_trace_point_emit_int(rc); ) @@ -249,7 +251,8 @@ RTE_TRACE_POINT( RTE_TRACE_POINT( rte_eventdev_trace_crypto_adapter_create, RTE_TRACE_POINT_ARGS(uint8_t adptr_id, uint8_t dev_id, void *adapter, - struct rte_event_port_conf *port_conf, uint8_t mode), + struct rte_event_port_conf *port_conf, uint8_t disable_impl_rel, + uint8_t mode), rte_trace_point_emit_u8(adptr_id); rte_trace_point_emit_u8(dev_id); rte_trace_point_emit_ptr(adapter); @@ -257,7 +260,7 @@ RTE_TRACE_POINT( rte_trace_point_emit_i32(port_conf->new_event_threshold); rte_trace_point_emit_u16(port_conf->dequeue_depth); rte_trace_point_emit_u16(port_conf->enqueue_depth); - rte_trace_point_emit_u8(port_conf->disable_implicit_release); + rte_trace_point_emit_u8(disable_impl_rel); ) RTE_TRACE_POINT( -- 2.13.6