From patchwork Tue Oct 10 20:42:06 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Ed Czeck X-Patchwork-Id: 132485 X-Patchwork-Delegate: ferruh.yigit@amd.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 363DC42350; Tue, 10 Oct 2023 22:42:21 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 8C8F44064E; Tue, 10 Oct 2023 22:42:16 +0200 (CEST) Received: from mail-qk1-f177.google.com (mail-qk1-f177.google.com [209.85.222.177]) by mails.dpdk.org (Postfix) with ESMTP id 3D188402EA for ; Tue, 10 Oct 2023 22:42:15 +0200 (CEST) Received: by mail-qk1-f177.google.com with SMTP id af79cd13be357-77412b91c41so356748585a.1 for ; Tue, 10 Oct 2023 13:42:15 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=atomicrules-com.20230601.gappssmtp.com; s=20230601; t=1696970534; x=1697575334; darn=dpdk.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=/MbgbUEE9tBgbKmHHL8bxPNB54NINvIWnlcbV1g3eG8=; b=bOcFcQ1kHZu5oUdZnM5+aK2HPCExE0hNK3BDRMRAcbT98YdMtjT4Sdi6mZNjwFB/ok kGkFXmK89Oa6kFB9WP+L3fFVaqc1quXFRguixvjsPs74S5n5S1GmDqKltjyY/+ap4bCo swfmOvTnhcAOuXkFHSNYdG8Q97Gpkuty6RA4ylbySBHJH2gcvBvIlBJYxEFtoTjOiAkv DY700XDwAppoSG8rvDM51PD0glJEQaO/CFbHaf60uPBUKsSYwsJQ+CLNEVHRyWMChMfH jnq855m2GwmtLWKdJWrBn/EjvTWZUM8ahME5yXK69BkJYhugBPkJKBQsocE59XoKDNlQ YUVQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1696970534; x=1697575334; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=/MbgbUEE9tBgbKmHHL8bxPNB54NINvIWnlcbV1g3eG8=; b=pwoaqRHzwJhfA7HMmjNegybdZTrTJMacXrjn1cWfsLi6WU6oag2PC3q49s6YcvWrcS w4MDqhv+eYL9XHHS+peDseTGs/jCvNj4j8GcV8YDrJ/Cb9nFEqpT1iMhhDVH7IaoEGlR uzz3opfQRGz0tLY/oA50l1TwdxdN+p7klXpjvOUsxQEujqwT5cYcr/azwlMv0OUHU9Gt AQixHgN8w0z6tOgAs3RiPDk5P8kgQ50zvsrg4PEHaH/PhhYOkPHk9s7ifhGvo5riuLWl PK0K4PIDx7Jok9N1IJaN8e5RJCvHQvppp2OAZ85gNiBhKGH2xihIsHDU9EN4HO4osM6e TP1A== X-Gm-Message-State: AOJu0YwTxofZA0a0w1ntG/oUsIAfUf/WfTCinNPmqm8QmI5PBNSkA5Vt UMbUWap97Eu6saaQXSG6UuuRjsjBCQH+Durn0fY= X-Google-Smtp-Source: AGHT+IHbR8sR8BGC+o7scEzdJaxzj58ioCtIe7Ai83RtB7NzCFruOzU7Ry9NN9GfTvSeH/j+R5rAJQ== X-Received: by 2002:a05:620a:d8c:b0:777:fec:5741 with SMTP id q12-20020a05620a0d8c00b007770fec5741mr8274977qkl.33.1696970534513; Tue, 10 Oct 2023 13:42:14 -0700 (PDT) Received: from z690.czeck.local (pool-173-48-111-149.bstnma.fios.verizon.net. [173.48.111.149]) by smtp.gmail.com with ESMTPSA id c27-20020a05620a165b00b0076cc0a6e127sm4603280qko.116.2023.10.10.13.42.13 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 10 Oct 2023 13:42:13 -0700 (PDT) From: Ed Czeck To: ferruh.yigit@amd.com, dev@dpdk.org Cc: Shepard Siegel , John Miller Subject: [PATCH v2 2/3] net/ark: remove RQ pacing firmware from PMD Date: Tue, 10 Oct 2023 16:42:06 -0400 Message-Id: <20231010204207.1027430-2-ed.czeck@atomicrules.com> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20231010204207.1027430-1-ed.czeck@atomicrules.com> References: <20231005205217.1753187-1-ed.czeck@atomicrules.com> <20231010204207.1027430-1-ed.czeck@atomicrules.com> MIME-Version: 1.0 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 features and function have been removed from FPGA firmware Signed-off-by: Ed Czeck --- v2: * restore previously incorrectly deleted line --- drivers/net/ark/ark_ethdev.c | 60 ++++++++----------------------- drivers/net/ark/ark_global.h | 3 -- drivers/net/ark/ark_rqp.c | 70 ------------------------------------ drivers/net/ark/ark_rqp.h | 58 ------------------------------ drivers/net/ark/meson.build | 1 - 5 files changed, 15 insertions(+), 177 deletions(-) delete mode 100644 drivers/net/ark/ark_rqp.c delete mode 100644 drivers/net/ark/ark_rqp.h diff --git a/drivers/net/ark/ark_ethdev.c b/drivers/net/ark/ark_ethdev.c index 90d3c8abe6..0ffd4b9e9e 100644 --- a/drivers/net/ark/ark_ethdev.c +++ b/drivers/net/ark/ark_ethdev.c @@ -17,7 +17,6 @@ #include "ark_mpu.h" #include "ark_ddm.h" #include "ark_udm.h" -#include "ark_rqp.h" #include "ark_pktdir.h" #include "ark_pktgen.h" #include "ark_pktchkr.h" @@ -107,36 +106,32 @@ static const struct rte_pci_id pci_id_ark_map[] = { * This structure is used to statically define the capabilities * of supported devices. * Capabilities: - * rqpacing - - * Some HW variants require that PCIe read-requests be correctly throttled. - * This is called "rqpacing" and has to do with credit and flow control - * on certain Arkville implementations. + * isvf -- defined for function id that are virtual */ struct ark_caps { - bool rqpacing; bool isvf; }; struct ark_dev_caps { uint32_t device_id; struct ark_caps caps; }; -#define SET_DEV_CAPS(id, rqp, vf) \ - {id, {.rqpacing = rqp, .isvf = vf} } +#define SET_DEV_CAPS(id, vf) \ + {id, {.isvf = vf} } static const struct ark_dev_caps ark_device_caps[] = { - SET_DEV_CAPS(0x100d, true, false), - SET_DEV_CAPS(0x100e, true, false), - SET_DEV_CAPS(0x100f, true, false), - SET_DEV_CAPS(0x1010, false, false), - SET_DEV_CAPS(0x1017, true, false), - SET_DEV_CAPS(0x1018, true, false), - SET_DEV_CAPS(0x1019, true, false), - SET_DEV_CAPS(0x101a, true, false), - SET_DEV_CAPS(0x101b, true, false), - SET_DEV_CAPS(0x101c, true, true), - SET_DEV_CAPS(0x101e, false, false), - SET_DEV_CAPS(0x101f, false, false), + SET_DEV_CAPS(0x100d, false), + SET_DEV_CAPS(0x100e, false), + SET_DEV_CAPS(0x100f, false), + SET_DEV_CAPS(0x1010, false), + SET_DEV_CAPS(0x1017, false), + SET_DEV_CAPS(0x1018, false), + SET_DEV_CAPS(0x1019, false), + SET_DEV_CAPS(0x101a, false), + SET_DEV_CAPS(0x101b, false), + SET_DEV_CAPS(0x101c, true), + SET_DEV_CAPS(0x101e, false), + SET_DEV_CAPS(0x101f, false), {.device_id = 0,} }; @@ -301,7 +296,6 @@ eth_ark_dev_init(struct rte_eth_dev *dev) int port_count = 1; int p; uint16_t num_queues; - bool rqpacing = false; ark->eth_dev = dev; @@ -319,7 +313,6 @@ eth_ark_dev_init(struct rte_eth_dev *dev) p = 0; while (ark_device_caps[p].device_id != 0) { if (pci_dev->id.device_id == ark_device_caps[p].device_id) { - rqpacing = ark_device_caps[p].caps.rqpacing; ark->isvf = ark_device_caps[p].caps.isvf; break; } @@ -344,12 +337,6 @@ eth_ark_dev_init(struct rte_eth_dev *dev) ark->pktgen.v = (void *)&ark->bar0[ARK_PKTGEN_BASE]; ark->pktchkr.v = (void *)&ark->bar0[ARK_PKTCHKR_BASE]; - if (rqpacing) { - ark->rqpacing = - (struct ark_rqpace_t *)(ark->bar0 + ARK_RCPACING_BASE); - } else { - ark->rqpacing = NULL; - } ark->started = 0; ark->pkt_dir_v = ARK_PKT_DIR_INIT_VAL; @@ -368,17 +355,6 @@ eth_ark_dev_init(struct rte_eth_dev *dev) ark->sysctrl.t32[4], __func__); return -1; } - if (ark->sysctrl.t32[3] != 0) { - if (ark->rqpacing) { - if (ark_rqp_lasped(ark->rqpacing)) { - ARK_PMD_LOG(ERR, "Arkville Evaluation System - " - "Timer has Expired\n"); - return -1; - } - ARK_PMD_LOG(WARNING, "Arkville Evaluation System - " - "Timer is Running\n"); - } - } ARK_PMD_LOG(DEBUG, "HW Sanity test has PASSED, expected constant" @@ -550,9 +526,6 @@ ark_config_device(struct rte_eth_dev *dev) mpu = RTE_PTR_ADD(mpu, ARK_MPU_QOFFSET); } - if (!ark->isvf && ark->rqpacing) - ark_rqp_stats_reset(ark->rqpacing); - return 0; } @@ -709,9 +682,6 @@ eth_ark_dev_close(struct rte_eth_dev *dev) /* * This should only be called once for the device during shutdown */ - if (ark->rqpacing) - ark_rqp_dump(ark->rqpacing); - /* return to power-on state */ if (ark->pd) ark_pktdir_setup(ark->pd, ARK_PKT_DIR_INIT_VAL); diff --git a/drivers/net/ark/ark_global.h b/drivers/net/ark/ark_global.h index 2f198edfe4..147b14b6c0 100644 --- a/drivers/net/ark/ark_global.h +++ b/drivers/net/ark/ark_global.h @@ -32,7 +32,6 @@ #define ARK_CMAC_BASE 0x80000 #define ARK_PKTDIR_BASE 0xa0000 #define ARK_PKTCHKR_BASE 0x90000 -#define ARK_RCPACING_BASE 0xb0000 #define ARK_EXTERNAL_BASE 0x100000 #define ARK_MPU_QOFFSET 0x00100 #define ARK_MAX_PORTS RTE_MAX_ETHPORTS @@ -150,8 +149,6 @@ struct ark_adapter { int started; uint16_t rx_queues; uint16_t tx_queues; - - struct ark_rqpace_t *rqpacing; }; typedef uint32_t *ark_t; diff --git a/drivers/net/ark/ark_rqp.c b/drivers/net/ark/ark_rqp.c deleted file mode 100644 index efb9730fe6..0000000000 --- a/drivers/net/ark/ark_rqp.c +++ /dev/null @@ -1,70 +0,0 @@ -/* SPDX-License-Identifier: BSD-3-Clause - * Copyright (c) 2015-2018 Atomic Rules LLC - */ - -#include - -#include "ark_rqp.h" -#include "ark_logs.h" - -/* ************************************************************************* */ -void -ark_rqp_stats_reset(struct ark_rqpace_t *rqp) -{ - rqp->stats_clear = 1; - /* POR 992 */ - /* rqp->cpld_max = 992; */ - /* POR 64 */ - /* rqp->cplh_max = 64; */ -} - -/* ************************************************************************* */ -void -ark_rqp_dump(struct ark_rqpace_t *rqp) -{ - if (rqp->err_count_other || rqp->cmpl_errors) - ARK_PMD_LOG(ERR, - "RQP Errors noted: ctrl: %d cplh_hmax %d cpld_max %d" - ARK_SU32 - ARK_SU32 - ARK_SU32 "\n", - rqp->ctrl, rqp->cplh_max, rqp->cpld_max, - "Error Count", rqp->err_cnt, - "Error General", rqp->err_count_other, - "Cmpl Errors", rqp->cmpl_errors); - - ARK_PMD_LOG(INFO, "RQP Dump: ctrl: %d cplh_hmax %d cpld_max %d" - ARK_SU32 - ARK_SU32 ARK_SU32 ARK_SU32 ARK_SU32 ARK_SU32 ARK_SU32 - ARK_SU32 ARK_SU32 ARK_SU32 ARK_SU32 ARK_SU32 ARK_SU32 - ARK_SU32 ARK_SU32 ARK_SU32 - ARK_SU32 ARK_SU32 ARK_SU32 ARK_SU32 ARK_SU32 "\n", - rqp->ctrl, rqp->cplh_max, rqp->cpld_max, - "Error Count", rqp->err_cnt, - "Error General", rqp->err_count_other, - "stall_pS", rqp->stall_ps, - "stall_pS Min", rqp->stall_ps_min, - "stall_pS Max", rqp->stall_ps_max, - "req_pS", rqp->req_ps, - "req_pS Min", rqp->req_ps_min, - "req_pS Max", rqp->req_ps_max, - "req_dWPS", rqp->req_dw_ps, - "req_dWPS Min", rqp->req_dw_ps_min, - "req_dWPS Max", rqp->req_dw_ps_max, - "cpl_pS", rqp->cpl_ps, - "cpl_pS Min", rqp->cpl_ps_min, - "cpl_pS Max", rqp->cpl_ps_max, - "cpl_dWPS", rqp->cpl_dw_ps, - "cpl_dWPS Min", rqp->cpl_dw_ps_min, - "cpl_dWPS Max", rqp->cpl_dw_ps_max, - "cplh pending", rqp->cplh_pending, - "cpld pending", rqp->cpld_pending, - "cplh pending max", rqp->cplh_pending_max, - "cpld pending max", rqp->cpld_pending_max); -} - -int -ark_rqp_lasped(struct ark_rqpace_t *rqp) -{ - return rqp->lasped; -} diff --git a/drivers/net/ark/ark_rqp.h b/drivers/net/ark/ark_rqp.h deleted file mode 100644 index d09f242e1e..0000000000 --- a/drivers/net/ark/ark_rqp.h +++ /dev/null @@ -1,58 +0,0 @@ -/* SPDX-License-Identifier: BSD-3-Clause - * Copyright (c) 2015-2018 Atomic Rules LLC - */ - -#ifndef _ARK_RQP_H_ -#define _ARK_RQP_H_ - -#include - -#include - -/* The RQP or ReQuest Pacer is an internal Arkville hardware module - * which limits the PCIE data flow to insure correct operation for the - * particular hardware PCIE endpoint. - * This module is *not* intended for end-user manipulation, hence - * there is minimal documentation. - */ - -/* - * RQ Pacing core hardware structure - * This is an overlay structures to a memory mapped FPGA device. These - * structs will never be instantiated in ram memory - */ -struct ark_rqpace_t { - volatile uint32_t ctrl; - volatile uint32_t stats_clear; - volatile uint32_t cplh_max; - volatile uint32_t cpld_max; - volatile uint32_t err_cnt; - volatile uint32_t stall_ps; - volatile uint32_t stall_ps_min; - volatile uint32_t stall_ps_max; - volatile uint32_t req_ps; - volatile uint32_t req_ps_min; - volatile uint32_t req_ps_max; - volatile uint32_t req_dw_ps; - volatile uint32_t req_dw_ps_min; - volatile uint32_t req_dw_ps_max; - volatile uint32_t cpl_ps; - volatile uint32_t cpl_ps_min; - volatile uint32_t cpl_ps_max; - volatile uint32_t cpl_dw_ps; - volatile uint32_t cpl_dw_ps_min; - volatile uint32_t cpl_dw_ps_max; - volatile uint32_t cplh_pending; - volatile uint32_t cpld_pending; - volatile uint32_t cplh_pending_max; - volatile uint32_t cpld_pending_max; - volatile uint32_t err_count_other; - char eval[4]; - volatile int32_t lasped; - volatile uint32_t cmpl_errors; -}; - -void ark_rqp_dump(struct ark_rqpace_t *rqp); -void ark_rqp_stats_reset(struct ark_rqpace_t *rqp); -int ark_rqp_lasped(struct ark_rqpace_t *rqp); -#endif diff --git a/drivers/net/ark/meson.build b/drivers/net/ark/meson.build index 8d87744c22..12b3935b85 100644 --- a/drivers/net/ark/meson.build +++ b/drivers/net/ark/meson.build @@ -16,6 +16,5 @@ sources = files( 'ark_pktchkr.c', 'ark_pktdir.c', 'ark_pktgen.c', - 'ark_rqp.c', 'ark_udm.c', )