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 C889AA034E; Sat, 12 Feb 2022 15:05:39 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 8DE5A40683; Sat, 12 Feb 2022 15:05:39 +0100 (CET) Received: from VLXDG1SPAM1.ramaxel.com (email.ramaxel.com [221.4.138.186]) by mails.dpdk.org (Postfix) with ESMTP id 68F114067C for ; Sat, 12 Feb 2022 15:05:37 +0100 (CET) Received: from V12DG1MBS01.ramaxel.local (v12dg1mbs01.ramaxel.local [172.26.18.31]) by VLXDG1SPAM1.ramaxel.com with ESMTPS id 21CE1mu3023246 (version=TLSv1.2 cipher=AES256-GCM-SHA384 bits=256 verify=FAIL); Sat, 12 Feb 2022 22:01:48 +0800 (GMT-8) (envelope-from songyl@ramaxel.com) Received: from localhost (2.0.0.25) by V12DG1MBS01.ramaxel.local (172.26.18.31) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2375.17; Sat, 12 Feb 2022 22:01:47 +0800 Date: Sat, 12 Feb 2022 22:01:46 +0800 From: Yanling Song To: Ferruh Yigit CC: , , , , , , Ziyang Xuan , Xiaoyun Wang , Guoyang Zhou , Hemant Agrawal Subject: Re: [PATCH v6 00/26] Net/SPNIC: support SPNIC into DPDK 22.03 Message-ID: <20220212220146.00007431@ramaxel.com> In-Reply-To: <3982cadb-5b7e-22b4-d8e7-091ff22e4702@intel.com> References: <3030c4f9-d3f8-bf36-bba1-00656bcc7e2e@intel.com> <20220121172713.00000853@ramaxel.com> <3982cadb-5b7e-22b4-d8e7-091ff22e4702@intel.com> X-Mailer: Claws Mail 4.0.0 (GTK+ 3.24.29; x86_64-w64-mingw32) MIME-Version: 1.0 Content-Type: text/plain; charset="US-ASCII" Content-Transfer-Encoding: 7bit X-Originating-IP: [2.0.0.25] X-ClientProxiedBy: V12DG1MBS03.ramaxel.local (172.26.18.33) To V12DG1MBS01.ramaxel.local (172.26.18.31) X-DNSRBL: X-MAIL: VLXDG1SPAM1.ramaxel.com 21CE1mu3023246 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 Fri, 21 Jan 2022 10:22:10 +0000 Ferruh Yigit wrote: > On 1/21/2022 9:27 AM, Yanling Song wrote: > > On Wed, 19 Jan 2022 16:56:52 +0000 > > Ferruh Yigit wrote: > > > >> On 12/30/2021 6:08 AM, Yanling Song wrote: > >>> The patchsets introduce SPNIC driver for Ramaxel's SPNxx serial > >>> NIC cards into DPDK 22.03. Ramaxel Memory Technology is a company > >>> which supply a lot of electric products: storage, communication, > >>> PCB... SPNxxx is a serial PCIE interface NIC cards: > >>> SPN110: 2 PORTs *25G > >>> SPN120: 4 PORTs *25G > >>> SPN130: 2 PORTs *100G > >>> > >> > >> Hi Yanling, > >> > >> As far as I can see hnic (from Huawei) and this spnic drivers are > >> alike, what is the relation between these two? > >> > > It is hard to create a brand new driver from scratch, so we > > referenced to hinic driver when developing spnic. > > > > That is OK, but based on the familiarity of the code you may consider > keeping the original code Copyright, I didn't investigate in > that level but cc'ed hinic maintainers for info. > Also cc'ed Hemant for guidance. > > Sorry for late reponse since it was Spring Festival and I was in vacation, just back to work. Hemant gave the guidance already, but we do not want to keep another company's copyright in our code. How should we modify code so that the code meet DPDK's requirment and can be accepted with our copyright only? > But my question was more related to the HW, is there any relation > between the hinic HW and spnic HW? Like one is derived from other > etc... I'm not clear what's the relation between hinic/spnic hw since we do not know what's the hinic hw. > > >>> The following is main features of our SPNIC: > >>> - TSO > >>> - LRO > >>> - Flow control > >>> - SR-IOV(Partially supported) > >>> - VLAN offload > >>> - VLAN filter > >>> - CRC offload > >>> - Promiscuous mode > >>> - RSS > >>> > >>> v6->v5, No real changes: > >>> 1. Move the fix of RTE_ETH_DEV_AUTOFILL_QUEUE_XSTATS from patch 26 > >>> to patch 2; 2. Change the description of patch 26. > >>> > >>> v5->v4: > >>> 1. Add prefix "spinc_" for external functions; > >>> 2. Remove temporary MACRO: RTE_ETH_DEV_AUTOFILL_QUEUE_XSTATS > >>> 3. Do not use void* for keeping the type information > >>> > >>> v3->v4: > >>> 1. Fix ABI test failure; > >>> 2. Remove some descriptions in spnic.rst. > >>> > >>> v2->v3: > >>> 1. Fix clang compiling failure. > >>> > >>> v1->v2: > >>> 1. Fix coding style issues and compiling failures; > >>> 2. Only support linux in meson.build; > >>> 3. Use CLOCK_MONOTONIC_COARSE instead of > >>> CLOCK_MONOTONIC/CLOCK_MONOTONIC_RAW; 4. Fix time_before(); > >>> 5. Remove redundant checks in spnic_dev_configure(); > >>> > >>> Yanling Song (26): > >>> drivers/net: introduce a new PMD driver > >>> net/spnic: initialize the HW interface > >>> net/spnic: add mbox message channel > >>> net/spnic: introduce event queue > >>> net/spnic: add mgmt module > >>> net/spnic: add cmdq and work queue > >>> net/spnic: add interface handling cmdq message > >>> net/spnic: add hardware info initialization > >>> net/spnic: support MAC and link event handling > >>> net/spnic: add function info initialization > >>> net/spnic: add queue pairs context initialization > >>> net/spnic: support mbuf handling of Tx/Rx > >>> net/spnic: support Rx congfiguration > >>> net/spnic: add port/vport enable > >>> net/spnic: support IO packets handling > >>> net/spnic: add device configure/version/info > >>> net/spnic: support RSS configuration update and get > >>> net/spnic: support VLAN filtering and offloading > >>> net/spnic: support promiscuous and allmulticast Rx modes > >>> net/spnic: support flow control > >>> net/spnic: support getting Tx/Rx queues info > >>> net/spnic: net/spnic: support xstats statistics > >>> net/spnic: support VFIO interrupt > >>> net/spnic: support Tx/Rx queue start/stop > >>> net/spnic: add doc infrastructure > >>> net/spnic: fixes unsafe C style code > >> > >> <...> > > >