From patchwork Fri Jul 12 09:25:54 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Nithin Dabilpuram X-Patchwork-Id: 56391 X-Patchwork-Delegate: thomas@monjalon.net Return-Path: X-Original-To: patchwork@dpdk.org Delivered-To: patchwork@dpdk.org Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 68E441B9B0; Fri, 12 Jul 2019 11:30:25 +0200 (CEST) Received: from mx0b-0016f401.pphosted.com (mx0a-0016f401.pphosted.com [67.231.148.174]) by dpdk.org (Postfix) with ESMTP id E1E531B9AC for ; Fri, 12 Jul 2019 11:30:23 +0200 (CEST) Received: from pps.filterd (m0045849.ppops.net [127.0.0.1]) by mx0a-0016f401.pphosted.com (8.16.0.27/8.16.0.27) with SMTP id x6C9UH5t000891; Fri, 12 Jul 2019 02:30:22 -0700 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=marvell.com; h=from : to : cc : subject : date : message-id : in-reply-to : references : mime-version : content-type; s=pfpt0818; bh=0Q0yleUE/hIPMairH6S/CRn6Za/qvLABslhAGb9QQDc=; b=me6VAqALsk4WAEunqqdj+S+z6/fDE1IGqUcD4+sfjk9b4UAq/zqEOuhYcEgvypmNzORp pqe89gGIpPFk1ujS8rz178Eh4sN7mVTehWbcpNgfpz0zhcjr/IQjChcW0TKJ77EQhy2V 6QFgVle19RGW+OR5yi/WTKweyA9EB+BjFDjDM9OXzmiTTpBPAvsgcHqK8Rt95h6KMlI/ CRKR2BFT7W+QW0I7YpaxAfwBkylWZVxnIAW8cmOLmwHFbbPvo+Vapt3ELTkckIgk0jBJ 2t2BvasA6m91rvYKw2sa2aXjY+bRej0xTBmZYdyiJsT8a+wCJmYKiWIGQIgIvyw32aqb Sg== Received: from sc-exch03.marvell.com ([199.233.58.183]) by mx0a-0016f401.pphosted.com with ESMTP id 2tpjxxs6md-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-SHA384 bits=256 verify=NOT); Fri, 12 Jul 2019 02:30:22 -0700 Received: from SC-EXCH01.marvell.com (10.93.176.81) by SC-EXCH03.marvell.com (10.93.176.83) with Microsoft SMTP Server (TLS) id 15.0.1367.3; Fri, 12 Jul 2019 02:26:02 -0700 Received: from maili.marvell.com (10.93.176.43) by SC-EXCH01.marvell.com (10.93.176.81) with Microsoft SMTP Server id 15.0.1367.3 via Frontend Transport; Fri, 12 Jul 2019 02:26:02 -0700 Received: from hyd1588t430.marvell.com (unknown [10.29.52.204]) by maili.marvell.com (Postfix) with ESMTP id 1186E3F703F; Fri, 12 Jul 2019 02:25:59 -0700 (PDT) From: Nithin Dabilpuram To: Jerin Jacob , Nithin Dabilpuram , Vamsi Attunuru , "John McNamara" , Marko Kovacevic , Kiran Kumar K CC: Date: Fri, 12 Jul 2019 14:55:54 +0530 Message-ID: <20190712092554.1699-1-ndabilpuram@marvell.com> X-Mailer: git-send-email 2.8.4 In-Reply-To: <20190710170142.142517-1-ndabilpuram@marvell.com> References: <20190710170142.142517-1-ndabilpuram@marvell.com> MIME-Version: 1.0 X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10434:, , definitions=2019-07-12_03:, , signatures=0 Subject: [dpdk-dev] [PATCH v2] common/octeontx2: add CNF95xx SoC support 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" Update platform support of CNF95xx in documentation and also, update the HW cap based on PCI subsystem id and revision id. This patch also changes HW capability handling to be based on PCI Revision ID. PCI Revision ID contains a unique identifier to identify chip, major and minor revisions. Signed-off-by: Nithin Dabilpuram Signed-off-by: Jerin Jacob Acked-by: Jerin Jacob --- v2: - Removed code replication related to pci config read doc/guides/platform/octeontx2.rst | 1 + drivers/common/octeontx2/otx2_common.h | 20 ++++++++++++ drivers/common/octeontx2/otx2_dev.c | 14 +------- drivers/common/octeontx2/otx2_dev.h | 37 +++++++++++++++++++--- .../octeontx2/rte_common_octeontx2_version.map | 2 +- drivers/net/octeontx2/otx2_ethdev.c | 4 +-- drivers/net/octeontx2/otx2_flow_ctrl.c | 8 ++--- drivers/net/octeontx2/otx2_tm.c | 2 +- 8 files changed, 63 insertions(+), 25 deletions(-) diff --git a/doc/guides/platform/octeontx2.rst b/doc/guides/platform/octeontx2.rst index 7d1fead..bee56ab 100644 --- a/doc/guides/platform/octeontx2.rst +++ b/doc/guides/platform/octeontx2.rst @@ -15,6 +15,7 @@ Supported OCTEON TX2 SoCs - CN96xx - CN93xx +- CNF95xx OCTEON TX2 Resource Virtualization Unit architecture ---------------------------------------------------- diff --git a/drivers/common/octeontx2/otx2_common.h b/drivers/common/octeontx2/otx2_common.h index cdb25d9..faf206b 100644 --- a/drivers/common/octeontx2/otx2_common.h +++ b/drivers/common/octeontx2/otx2_common.h @@ -108,6 +108,26 @@ extern int otx2_logtype_dpi; #define PCI_DEVID_OCTEONTX2_RVU_AF_VF 0xA0f8 #define PCI_DEVID_OCTEONTX2_DPI_VF 0xA081 +/* Subsystem Device ID */ +#define PCI_SUBSYS_DEVID_96XX_95XX 0xB200 + +/* + * REVID for RVU PCIe devices. + * Bits 0..1: minor pass + * Bits 3..2: major pass + * Bits 7..4: midr id, 0:96, 1:95, 2:loki, f:unknown + */ + +#define RVU_PCI_REV_MIDR_ID(rev_id) (rev_id >> 4) +#define RVU_PCI_REV_MAJOR(rev_id) ((rev_id >> 2) & 0x3) +#define RVU_PCI_REV_MINOR(rev_id) (rev_id & 0x3) + +#define RVU_PCI_CN96XX_MIDR_ID 0x0 +#define RVU_PCI_CNF95XX_MIDR_ID 0x1 + +/* PCI Config offsets */ +#define RVU_PCI_REVISION_ID 0x08 + /* IO Access */ #define otx2_read64(addr) rte_read64_relaxed((void *)(addr)) #define otx2_write64(val, addr) rte_write64_relaxed((val), (void *)(addr)) diff --git a/drivers/common/octeontx2/otx2_dev.c b/drivers/common/octeontx2/otx2_dev.c index 3b51fa9..f58a6e7 100644 --- a/drivers/common/octeontx2/otx2_dev.c +++ b/drivers/common/octeontx2/otx2_dev.c @@ -876,19 +876,8 @@ otx2_dev_active_vfs(void *otx2_dev) } static void -otx2_update_pass_hwcap(struct rte_pci_device *pci_dev, struct otx2_dev *dev) -{ - RTE_SET_USED(pci_dev); - - /* Update this logic when we have A1 */ - dev->hwcap |= OTX2_HWCAP_F_A0; -} - -static void otx2_update_vf_hwcap(struct rte_pci_device *pci_dev, struct otx2_dev *dev) { - dev->hwcap = 0; - switch (pci_dev->id.device_id) { case PCI_DEVID_OCTEONTX2_RVU_PF: break; @@ -907,7 +896,7 @@ otx2_update_vf_hwcap(struct rte_pci_device *pci_dev, struct otx2_dev *dev) * Initialize the otx2 device */ int -otx2_dev_init(struct rte_pci_device *pci_dev, void *otx2_dev) +otx2_dev_priv_init(struct rte_pci_device *pci_dev, void *otx2_dev) { int up_direction = MBOX_DIR_PFAF_UP; int rc, direction = MBOX_DIR_PFAF; @@ -931,7 +920,6 @@ otx2_dev_init(struct rte_pci_device *pci_dev, void *otx2_dev) dev->bar4 = bar4; otx2_update_vf_hwcap(pci_dev, dev); - otx2_update_pass_hwcap(pci_dev, dev); if (otx2_dev_is_vf(dev)) { direction = MBOX_DIR_VFPF; diff --git a/drivers/common/octeontx2/otx2_dev.h b/drivers/common/octeontx2/otx2_dev.h index be862ad..53a3669 100644 --- a/drivers/common/octeontx2/otx2_dev.h +++ b/drivers/common/octeontx2/otx2_dev.h @@ -13,14 +13,18 @@ #include "otx2_mempool.h" /* Common HWCAP flags. Use from LSB bits */ -#define OTX2_HWCAP_F_VF BIT_ULL(0) /* VF device */ +#define OTX2_HWCAP_F_VF BIT_ULL(8) /* VF device */ #define otx2_dev_is_vf(dev) (dev->hwcap & OTX2_HWCAP_F_VF) #define otx2_dev_is_pf(dev) (!(dev->hwcap & OTX2_HWCAP_F_VF)) #define otx2_dev_is_lbk(dev) ((dev->hwcap & OTX2_HWCAP_F_VF) && \ (dev->tx_chan_base < 0x700)) +#define otx2_dev_revid(dev) (dev->hwcap & 0xFF) -#define OTX2_HWCAP_F_A0 BIT_ULL(1) /* A0 device */ -#define otx2_dev_is_A0(dev) (dev->hwcap & OTX2_HWCAP_F_A0) +#define otx2_dev_is_A0(dev) \ + ((RVU_PCI_REV_MAJOR(otx2_dev_revid(dev)) == 0x0) && \ + (RVU_PCI_REV_MINOR(otx2_dev_revid(dev)) == 0x0)) +#define otx2_dev_is_Ax(dev) \ + ((RVU_PCI_REV_MAJOR(otx2_dev_revid(dev)) == 0x0)) struct otx2_dev; @@ -61,7 +65,32 @@ struct otx2_dev { OTX2_DEV; }; -int otx2_dev_init(struct rte_pci_device *pci_dev, void *otx2_dev); +int otx2_dev_priv_init(struct rte_pci_device *pci_dev, void *otx2_dev); + +/* Common dev init and fini routines */ + +static __rte_always_inline int +otx2_dev_init(struct rte_pci_device *pci_dev, void *otx2_dev) +{ + struct otx2_dev *dev = otx2_dev; + uint8_t rev_id; + int rc; + + rc = rte_pci_read_config(pci_dev, &rev_id, + 1, RVU_PCI_REVISION_ID); + if (rc != 1) { + otx2_err("Failed to read pci revision id, rc=%d", rc); + return rc; + } + + if (pci_dev->id.subsystem_device_id == PCI_SUBSYS_DEVID_96XX_95XX) + dev->hwcap = rev_id; + else + dev->hwcap = 0; + + return otx2_dev_priv_init(pci_dev, otx2_dev); +} + void otx2_dev_fini(struct rte_pci_device *pci_dev, void *otx2_dev); int otx2_dev_active_vfs(void *otx2_dev); diff --git a/drivers/common/octeontx2/rte_common_octeontx2_version.map b/drivers/common/octeontx2/rte_common_octeontx2_version.map index 2f48263..4400120 100644 --- a/drivers/common/octeontx2/rte_common_octeontx2_version.map +++ b/drivers/common/octeontx2/rte_common_octeontx2_version.map @@ -3,7 +3,7 @@ DPDK_19.08 { otx2_dev_active_vfs; otx2_dev_fini; - otx2_dev_init; + otx2_dev_priv_init; otx2_logtype_base; otx2_logtype_dpi; diff --git a/drivers/net/octeontx2/otx2_ethdev.c b/drivers/net/octeontx2/otx2_ethdev.c index 156e7d3..fcb1869 100644 --- a/drivers/net/octeontx2/otx2_ethdev.c +++ b/drivers/net/octeontx2/otx2_ethdev.c @@ -1235,7 +1235,7 @@ otx2_nix_configure(struct rte_eth_dev *eth_dev) nix_lf_free(dev); } - if (otx2_dev_is_A0(dev) && + if (otx2_dev_is_Ax(dev) && (txmode->offloads & DEV_TX_OFFLOAD_SCTP_CKSUM) && ((txmode->offloads & DEV_TX_OFFLOAD_OUTER_IPV4_CKSUM) || (txmode->offloads & DEV_TX_OFFLOAD_OUTER_UDP_CKSUM))) { @@ -1787,7 +1787,7 @@ otx2_eth_dev_init(struct rte_eth_dev *eth_dev) dev->tx_offload_capa = nix_get_tx_offload_capa(dev); dev->rx_offload_capa = nix_get_rx_offload_capa(dev); - if (otx2_dev_is_A0(dev)) { + if (otx2_dev_is_Ax(dev)) { dev->hwcap |= OTX2_FIXUP_F_MIN_4K_Q; dev->hwcap |= OTX2_FIXUP_F_LIMIT_CQ_FULL; } diff --git a/drivers/net/octeontx2/otx2_flow_ctrl.c b/drivers/net/octeontx2/otx2_flow_ctrl.c index 0392086..ae56352 100644 --- a/drivers/net/octeontx2/otx2_flow_ctrl.c +++ b/drivers/net/octeontx2/otx2_flow_ctrl.c @@ -162,8 +162,8 @@ otx2_nix_flow_ctrl_set(struct rte_eth_dev *eth_dev, /* Check if TX pause frame is already enabled or not */ if (fc->tx_pause ^ tx_pause) { - if (otx2_dev_is_A0(dev) && eth_dev->data->dev_started) { - /* on A0, CQ should be in disabled state + if (otx2_dev_is_Ax(dev) && eth_dev->data->dev_started) { + /* on Ax, CQ should be in disabled state * while setting flow control configuration. */ otx2_info("Stop the port=%d for setting flow control\n", @@ -207,8 +207,8 @@ otx2_nix_update_flow_ctrl_mode(struct rte_eth_dev *eth_dev) */ otx2_nix_flow_ctrl_get(eth_dev, &fc_conf); - /* To avoid Link credit deadlock on A0, disable Tx FC if it's enabled */ - if (otx2_dev_is_A0(dev) && + /* To avoid Link credit deadlock on Ax, disable Tx FC if it's enabled */ + if (otx2_dev_is_Ax(dev) && (fc_conf.mode == RTE_FC_FULL || fc_conf.mode == RTE_FC_RX_PAUSE)) { fc_conf.mode = (fc_conf.mode == RTE_FC_FULL || diff --git a/drivers/net/octeontx2/otx2_tm.c b/drivers/net/octeontx2/otx2_tm.c index 2469206..12903ec 100644 --- a/drivers/net/octeontx2/otx2_tm.c +++ b/drivers/net/octeontx2/otx2_tm.c @@ -59,7 +59,7 @@ static bool nix_tm_have_tl1_access(struct otx2_eth_dev *dev) { bool is_lbk = otx2_dev_is_lbk(dev); - return otx2_dev_is_pf(dev) && !otx2_dev_is_A0(dev) && + return otx2_dev_is_pf(dev) && !otx2_dev_is_Ax(dev) && !is_lbk && !dev->maxvf; }