From patchwork Tue Sep 27 07:32:48 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Junfeng Guo X-Patchwork-Id: 116950 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 A1118A00C2; Tue, 27 Sep 2022 09:33:53 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 86C974281C; Tue, 27 Sep 2022 09:33:46 +0200 (CEST) Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) by mails.dpdk.org (Postfix) with ESMTP id 4A24340696 for ; Tue, 27 Sep 2022 09:33:44 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1664264024; x=1695800024; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=EnaPNuffI9yoqO175HNT6TsBZRws4jNWwG+KBn21MMU=; b=GXYyp/4IxATF6ypRlIcY7BBMFwBJM3kazndEmWSaBDWkJhzrpRn+g9qW oCRRPEWWdCn/LujkirYHX77YMXdZ++I3pzVY9tXlPG7eh8JGdLCdrHF7G 4twE9udLc1OmF9y5kNI0/JEeLY0WKdfEH82+QtiDCDbUiOMQXf22eWYxJ kTZT2bYYgJ2oJ/SIZix0IMwzMtik3dasqyPCzPJsh+5cguRScFauQDC4Y TUOubCeplO0ucppw5OGP6DjAWyhdorlHfMbiATiJ2uQMIrrwjatnbAw9m 4tAL551WuzOtr0c3mvdA4o3B54inIOHJINX3q6nQIBlAfe2pyYkdcOp0t A==; X-IronPort-AV: E=McAfee;i="6500,9779,10482"; a="298844106" X-IronPort-AV: E=Sophos;i="5.93,348,1654585200"; d="scan'208";a="298844106" Received: from fmsmga006.fm.intel.com ([10.253.24.20]) by fmsmga102.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 27 Sep 2022 00:33:39 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6500,9779,10482"; a="866470081" X-IronPort-AV: E=Sophos;i="5.93,348,1654585200"; d="scan'208";a="866470081" Received: from dpdk-jf-ntb-one.sh.intel.com ([10.67.111.104]) by fmsmga006.fm.intel.com with ESMTP; 27 Sep 2022 00:33:35 -0700 From: Junfeng Guo To: qi.z.zhang@intel.com, jingjing.wu@intel.com Cc: ferruh.yigit@xilinx.com, dev@dpdk.org, xiaoyun.li@intel.com, awogbemila@google.com, bruce.richardson@intel.com, xueqin.lin@intel.com, junfeng.guo@intel.com, Haiyue Wang Subject: [PATCH v4 2/9] net/gve/base: add logs and OS specific implementation Date: Tue, 27 Sep 2022 15:32:48 +0800 Message-Id: <20220927073255.1803892-3-junfeng.guo@intel.com> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20220927073255.1803892-1-junfeng.guo@intel.com> References: <20220923093829.3019525-2-junfeng.guo@intel.com> <20220927073255.1803892-1-junfeng.guo@intel.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 Add GVE PMD logs. Add some MACRO definitions and memory operations which are specific for DPDK. Signed-off-by: Haiyue Wang Signed-off-by: Xiaoyun Li Signed-off-by: Junfeng Guo --- drivers/net/gve/base/gve_adminq.h | 2 + drivers/net/gve/base/gve_desc.h | 2 + drivers/net/gve/base/gve_desc_dqo.h | 2 + drivers/net/gve/base/gve_osdep.h | 159 ++++++++++++++++++++++++++++ drivers/net/gve/base/gve_register.h | 2 + drivers/net/gve/gve_logs.h | 14 +++ 6 files changed, 181 insertions(+) create mode 100644 drivers/net/gve/base/gve_osdep.h create mode 100644 drivers/net/gve/gve_logs.h diff --git a/drivers/net/gve/base/gve_adminq.h b/drivers/net/gve/base/gve_adminq.h index c7114cc883..cd496760ae 100644 --- a/drivers/net/gve/base/gve_adminq.h +++ b/drivers/net/gve/base/gve_adminq.h @@ -8,6 +8,8 @@ #ifndef _GVE_ADMINQ_H #define _GVE_ADMINQ_H +#include "gve_osdep.h" + /* Admin queue opcodes */ enum gve_adminq_opcodes { GVE_ADMINQ_DESCRIBE_DEVICE = 0x1, diff --git a/drivers/net/gve/base/gve_desc.h b/drivers/net/gve/base/gve_desc.h index 358755b7e0..627b9120dc 100644 --- a/drivers/net/gve/base/gve_desc.h +++ b/drivers/net/gve/base/gve_desc.h @@ -9,6 +9,8 @@ #ifndef _GVE_DESC_H_ #define _GVE_DESC_H_ +#include "gve_osdep.h" + /* A note on seg_addrs * * Base addresses encoded in seg_addr are not assumed to be physical diff --git a/drivers/net/gve/base/gve_desc_dqo.h b/drivers/net/gve/base/gve_desc_dqo.h index 0d533abcd1..5031752b43 100644 --- a/drivers/net/gve/base/gve_desc_dqo.h +++ b/drivers/net/gve/base/gve_desc_dqo.h @@ -9,6 +9,8 @@ #ifndef _GVE_DESC_DQO_H_ #define _GVE_DESC_DQO_H_ +#include "gve_osdep.h" + #define GVE_TX_MAX_HDR_SIZE_DQO 255 #define GVE_TX_MIN_TSO_MSS_DQO 88 diff --git a/drivers/net/gve/base/gve_osdep.h b/drivers/net/gve/base/gve_osdep.h new file mode 100644 index 0000000000..7cb73002f4 --- /dev/null +++ b/drivers/net/gve/base/gve_osdep.h @@ -0,0 +1,159 @@ +/* SPDX-License-Identifier: BSD-3-Clause + * Copyright(C) 2022 Intel Corporation + */ + +#ifndef _GVE_OSDEP_H_ +#define _GVE_OSDEP_H_ + +#include +#include +#include +#include +#include +#include + +#include +#include +#include +#include +#include +#include +#include +#include +#include + +#include "../gve_logs.h" + +typedef uint8_t u8; +typedef uint16_t u16; +typedef uint32_t u32; +typedef uint64_t u64; + +typedef rte_be16_t __sum16; + +typedef rte_be16_t __be16; +typedef rte_be32_t __be32; +typedef rte_be64_t __be64; + +typedef rte_iova_t dma_addr_t; + +#define ETH_MIN_MTU RTE_ETHER_MIN_MTU +#define ETH_ALEN RTE_ETHER_ADDR_LEN + +#ifndef PAGE_SHIFT +#define PAGE_SHIFT 12 +#endif +#ifndef PAGE_SIZE +#define PAGE_SIZE (1UL << PAGE_SHIFT) +#endif + +#define BIT(nr) RTE_BIT32(nr) + +#define be16_to_cpu(x) rte_be_to_cpu_16(x) +#define be32_to_cpu(x) rte_be_to_cpu_32(x) +#define be64_to_cpu(x) rte_be_to_cpu_64(x) + +#define cpu_to_be16(x) rte_cpu_to_be_16(x) +#define cpu_to_be32(x) rte_cpu_to_be_32(x) +#define cpu_to_be64(x) rte_cpu_to_be_64(x) + +#define READ_ONCE32(x) rte_read32(&(x)) + +#ifndef ____cacheline_aligned +#define ____cacheline_aligned __rte_cache_aligned +#endif +#ifndef __packed +#define __packed __rte_packed +#endif +#define __iomem + +#define msleep(ms) rte_delay_ms(ms) + +/* These macros are used to generate compilation errors if a struct/union + * is not exactly the correct length. It gives a divide by zero error if + * the struct/union is not of the correct size, otherwise it creates an + * enum that is never used. + */ +#define GVE_CHECK_STRUCT_LEN(n, X) enum gve_static_assert_enum_##X \ + { gve_static_assert_##X = (n) / ((sizeof(struct X) == (n)) ? 1 : 0) } +#define GVE_CHECK_UNION_LEN(n, X) enum gve_static_asset_enum_##X \ + { gve_static_assert_##X = (n) / ((sizeof(union X) == (n)) ? 1 : 0) } + +static __rte_always_inline u8 +readb(volatile void *addr) +{ + return rte_read8(addr); +} + +static __rte_always_inline void +writeb(u8 value, volatile void *addr) +{ + rte_write8(value, addr); +} + +static __rte_always_inline void +writel(u32 value, volatile void *addr) +{ + rte_write32(value, addr); +} + +static __rte_always_inline u32 +ioread32be(const volatile void *addr) +{ + return rte_be_to_cpu_32(rte_read32(addr)); +} + +static __rte_always_inline void +iowrite32be(u32 value, volatile void *addr) +{ + writel(rte_cpu_to_be_32(value), addr); +} + +/* DMA memory allocation tracking */ +struct gve_dma_mem { + void *va; + rte_iova_t pa; + uint32_t size; + const void *zone; +}; + +static inline void * +gve_alloc_dma_mem(struct gve_dma_mem *mem, u64 size) +{ + static uint16_t gve_dma_memzone_id; + const struct rte_memzone *mz = NULL; + char z_name[RTE_MEMZONE_NAMESIZE]; + + if (!mem) + return NULL; + + snprintf(z_name, sizeof(z_name), "gve_dma_%u", + __atomic_fetch_add(&gve_dma_memzone_id, 1, __ATOMIC_RELAXED)); + mz = rte_memzone_reserve_aligned(z_name, size, SOCKET_ID_ANY, + RTE_MEMZONE_IOVA_CONTIG, + PAGE_SIZE); + if (!mz) + return NULL; + + mem->size = size; + mem->va = mz->addr; + mem->pa = mz->iova; + mem->zone = mz; + PMD_DRV_LOG(DEBUG, "memzone %s is allocated", mz->name); + + return mem->va; +} + +static inline void +gve_free_dma_mem(struct gve_dma_mem *mem) +{ + PMD_DRV_LOG(DEBUG, "memzone %s to be freed", + ((const struct rte_memzone *)mem->zone)->name); + + rte_memzone_free(mem->zone); + mem->zone = NULL; + mem->va = NULL; + mem->pa = 0; +} + +#endif /* _GVE_OSDEP_H_ */ diff --git a/drivers/net/gve/base/gve_register.h b/drivers/net/gve/base/gve_register.h index b65f336be2..a599c1a08e 100644 --- a/drivers/net/gve/base/gve_register.h +++ b/drivers/net/gve/base/gve_register.h @@ -7,6 +7,8 @@ #ifndef _GVE_REGISTER_H_ #define _GVE_REGISTER_H_ +#include "gve_osdep.h" + /* Fixed Configuration Registers */ struct gve_registers { __be32 device_status; diff --git a/drivers/net/gve/gve_logs.h b/drivers/net/gve/gve_logs.h new file mode 100644 index 0000000000..0d02da46e1 --- /dev/null +++ b/drivers/net/gve/gve_logs.h @@ -0,0 +1,14 @@ +/* SPDX-License-Identifier: BSD-3-Clause + * Copyright(C) 2022 Intel Corporation + */ + +#ifndef _GVE_LOGS_H_ +#define _GVE_LOGS_H_ + +extern int gve_logtype_driver; + +#define PMD_DRV_LOG(level, fmt, args...) \ + rte_log(RTE_LOG_ ## level, gve_logtype_driver, "%s(): " fmt "\n", \ + __func__, ## args) + +#endif