From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 193BF45FF9; Mon, 6 Jan 2025 04:37:02 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id A74DD4065A; Mon, 6 Jan 2025 04:37:01 +0100 (CET) Received: from lf-2-59.ptr.blmpb.com (lf-2-59.ptr.blmpb.com [101.36.218.59]) by mails.dpdk.org (Postfix) with ESMTP id AF82E4014F for ; Mon, 6 Jan 2025 04:36:58 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; s=feishu2403070942; d=yunsilicon.com; t=1736134611; h=from:subject: mime-version:from:date:message-id:subject:to:cc:reply-to:content-type: mime-version:in-reply-to:message-id; bh=qX/LgIgO+JYqrl/Ki47HNdk8kg4iQy0pyEKTrM2qWCA=; b=UtQ+CmSH91sxfq4/K1Ius9EbEccGgPVIZk/UZ/k9PeG/4ci/1Fc8KldBxWa7rQoS2QOvQ9 jtdyZ2z+2KIwPpaKtoLF4qA0KI32San3V2OfogjAPjbp0VdCTFNWKAfwFbavObd/216NMV zoS2BSTxIAs3ekl6vjV2piR1mB+FdpwDccMDUhF0Rss/M9talqAVWKFcWTeeZalJl4F9xI btq2Vb0DTZj5GF5WrDFhdH7DwAzM6UEBnunADSt+3PwLDsMxNzW5SUN3ZoCMlFeM/PBdfM DYH7pKNZZW60ulHbMVcgTd1oh/UHs1H1+NuPUcPukyPO2CAzaA9fNPZbjUlsoA== Mime-Version: 1.0 In-Reply-To: <20250104111235.7a4c358f@pi5> X-Lms-Return-Path: References: <20240906121405.3404357-1-wanry@yunsilicon.com> <20250104111235.7a4c358f@pi5> To: "Stephen Hemminger" Cc: , , , , , , , Subject: Re: [PATCH 00/19] XSC PMD for Yunsilicon NICs Date: Mon, 6 Jan 2025 11:36:50 +0800 Content-Type: text/plain; charset=UTF-8 Message-Id: <4403fa03-cf3d-45fe-88b7-035f6207d6d4@yunsilicon.com> Content-Transfer-Encoding: 7bit User-Agent: Mozilla Thunderbird From: "WanRenyong" Received: from [127.0.0.1] ([116.231.104.97]) by smtp.feishu.cn with ESMTPS; Mon, 06 Jan 2025 11:36:48 +0800 X-Original-From: WanRenyong 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 On 2025/1/5 3:12, Stephen Hemminger wrote: > On Fri, 6 Sep 2024 20:13:46 +0800 > "WanRenyong" wrote: > >> This xsc PMD (**librte_net_xsc**) provides poll mode driver for Yunsilicon metaScale >> serials NICs. >> >> Features: >> --------- >> - MTU update >> - TSO >> - RSS hash >> - RSS key update >> - RSS reta update >> - L3 checksum offload >> - L4 checksum offload >> - Inner L3 checksum >> - Inner L4 checksum >> - Basic stats >> >> Support NICs: >> ------------- >> - metaScale-200S Single QSFP56 Port 200GE SmartNIC >> - metaScale-200 Quad QSFP28 Ports 100GE SmartNIC >> - metaScale-50 Dual QSFP28 Port 25GE SmartNIC >> - metaScale-100Q Quad QSFP28 Port 25GE SmartNIC >> >> >> ----- >> >> WanRenyong (19): >> net/xsc: add doc and minimum build framework >> net/xsc: add log macro >> net/xsc: add PCI device probe and remove >> net/xsc: add xsc device init and uninit >> net/xsc: add ioctl command interface >> net/xsc: initialize hardware information >> net/xsc: add representor ports probe >> net/xsc: create eth devices for representor ports >> net/xsc: initial representor eth device >> net/xsc: add ethdev configure and rxtx queue setup ops >> net/xsc: add mailbox and structure >> net/xsc: add ethdev RSS hash ops >> net/xsc: add ethdev start and stop ops >> net/xsc: add ethdev Rx burst >> net/xsc: add ethdev Tx burst >> net/xsc: configure xsc device hardware table >> net/xsc: add dev link and MTU ops >> net/xsc: add dev infos get >> net/xsc: add dev basic stats ops >> >> .mailmap | 4 + >> MAINTAINERS | 9 + >> doc/guides/nics/features/xsc.ini | 18 + >> doc/guides/nics/index.rst | 1 + >> doc/guides/nics/xsc.rst | 31 + >> drivers/net/meson.build | 1 + >> drivers/net/xsc/meson.build | 36 + >> drivers/net/xsc/xsc_ctrl.c | 64 ++ >> drivers/net/xsc/xsc_ctrl.h | 314 +++++++ >> drivers/net/xsc/xsc_defs.h | 61 ++ >> drivers/net/xsc/xsc_dev.c | 326 +++++++ >> drivers/net/xsc/xsc_dev.h | 99 +++ >> drivers/net/xsc/xsc_ethdev.c | 1434 ++++++++++++++++++++++++++++++ >> drivers/net/xsc/xsc_ethdev.h | 81 ++ >> drivers/net/xsc/xsc_flow.c | 167 ++++ >> drivers/net/xsc/xsc_flow.h | 67 ++ >> drivers/net/xsc/xsc_log.h | 44 + >> drivers/net/xsc/xsc_rxtx.c | 445 +++++++++ >> drivers/net/xsc/xsc_rxtx.h | 214 +++++ >> drivers/net/xsc/xsc_utils.c | 346 +++++++ >> drivers/net/xsc/xsc_utils.h | 27 + >> 21 files changed, 3789 insertions(+) >> create mode 100644 doc/guides/nics/features/xsc.ini >> create mode 100644 doc/guides/nics/xsc.rst >> create mode 100644 drivers/net/xsc/meson.build >> create mode 100644 drivers/net/xsc/xsc_ctrl.c >> create mode 100644 drivers/net/xsc/xsc_ctrl.h >> create mode 100644 drivers/net/xsc/xsc_defs.h >> create mode 100644 drivers/net/xsc/xsc_dev.c >> create mode 100644 drivers/net/xsc/xsc_dev.h >> create mode 100644 drivers/net/xsc/xsc_ethdev.c >> create mode 100644 drivers/net/xsc/xsc_ethdev.h >> create mode 100644 drivers/net/xsc/xsc_flow.c >> create mode 100644 drivers/net/xsc/xsc_flow.h >> create mode 100644 drivers/net/xsc/xsc_log.h >> create mode 100644 drivers/net/xsc/xsc_rxtx.c >> create mode 100644 drivers/net/xsc/xsc_rxtx.h >> create mode 100644 drivers/net/xsc/xsc_utils.c >> create mode 100644 drivers/net/xsc/xsc_utils.h >> > Please fix the build error: > drivers/libtmp_rte_net_xsc.a.p/net_xsc_xsc_vfio.c.o -c ../drivers/net/xsc/xsc_vfio.c > ../drivers/net/xsc/xsc_vfio.c:482:26: error: cast from 'volatile struct xsc_cqe (*)[]' to 'void *' drops volatile qualifier [-Werror,-Wcast-qual] > 482 | cq_info->cqes = (void *)cqes; > | ^ > > Probably best not to mark that arg as volatile. > Do not try and solve the problem by disabling the warning. > > > And address the review feedback and resubmit when ready. Thanks for review. We will fix it and resubmit next version soon. -- Thanks, WanRenyong