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 4E67AA04A6; Mon, 24 Jan 2022 06:13:04 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id BBDE94278E; Mon, 24 Jan 2022 06:13:03 +0100 (CET) Received: from EUR05-AM6-obe.outbound.protection.outlook.com (mail-am6eur05on2057.outbound.protection.outlook.com [40.107.22.57]) by mails.dpdk.org (Postfix) with ESMTP id 7931D40040 for ; Mon, 24 Jan 2022 06:13:02 +0100 (CET) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=ZcAjknwSZ0dqTXAiAUAHQo8ZZNpMWJBc0gOBiXfErhx0+PoQdPli86BZl/diTvY9F20ii6QY2efVfw+WiLeCzr2Cp3weHRISetuJZ0byBNL+sUWvGE+cXNBIR6ujhPj2XzdqyvmxOGM+DD3crYo87iTGtKYm+gXZUD3pjHI1HYlnYNF2JmtiL9930J4UmM/1jT+sgSeSD44dfDW0GsDDepm++yYyJok2flr6HZb7zU9l9NMR01CDyzpgnToRlTd95EG8wbRtYZJTckye4yBlcJCI+9rdMobFUYwFmjlvyYoPH8nDBWX4ByNvHe46lM9iN2aO31+PMZHlp5IAC1Zudw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=Zj+VnuhXAvrfi/zOz5XMPDFcPsrRKxcx1zgCzKtMYK8=; b=hohJLtLe2d1GY2A/BnNMw/gtgbftsNxzG4Qsoq2IKH7PfYldHHHdSSW4g5o5sL+fo9yXAzqZjiNy4SAUbSJ9j4zf4x7MXdrOQDVpf1vQaQwO/ZWQ7M6+HgVqxWsKdBwTPmv192XbNbpA/y2Sb6dRVoTBzMZTQpLYbti+Ppn/vE74UjVkQbIvhA0TiFZNxJLZ4EBrS2cHRTuJ03tdiUma7ufOWJoKn4sGuH4rwJ2zoqvp+3uIFO9rdwhbF1SKP55YrSbgnvl7C0LHUY8ugFEAKzZzfND420LRmY0D6dB4cNAo7TDxXctmMUVRYV7Pqfn46o7vdzKW4XZQNexI1a7k4w== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=none; dmarc=none; dkim=none; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=NXP1.onmicrosoft.com; s=selector2-NXP1-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=Zj+VnuhXAvrfi/zOz5XMPDFcPsrRKxcx1zgCzKtMYK8=; b=j1iFHPpkJ4dtVJrFLtGCiHHt2n0R6Szvnq3UDk0Km5axXYDmxHqDejMTi2q9YTbo7VZUaU1uV6R12S8KxunZLwV5psV77XHrVmbotW6J+sxEWOrgYfstTZ+8mdHZtxvtdfzU8sMycgQXHL1AWjRWfIpRHh1LM6gBEYMAjIpIPbs= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=oss.nxp.com; Received: from AS8PR04MB9064.eurprd04.prod.outlook.com (2603:10a6:20b:447::17) by VI1PR0402MB3821.eurprd04.prod.outlook.com (2603:10a6:803:1b::33) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4909.17; Mon, 24 Jan 2022 05:12:59 +0000 Received: from AS8PR04MB9064.eurprd04.prod.outlook.com ([fe80::6844:bed9:f897:9429]) by AS8PR04MB9064.eurprd04.prod.outlook.com ([fe80::6844:bed9:f897:9429%6]) with mapi id 15.20.4909.017; Mon, 24 Jan 2022 05:12:59 +0000 Message-ID: Date: Mon, 24 Jan 2022 10:42:48 +0530 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101 Thunderbird/91.3.0 Subject: Re: [PATCH v6 00/26] Net/SPNIC: support SPNIC into DPDK 22.03 Content-Language: en-US To: Ferruh Yigit , Yanling Song Cc: dev@dpdk.org, yanling.song@linux.dev, yanggan@ramaxel.com, xuyun@ramaxel.com, stephen@networkplumber.org, lihuisong@huawei.com, Ziyang Xuan , Xiaoyun Wang , Guoyang Zhou , Hemant Agrawal References: <3030c4f9-d3f8-bf36-bba1-00656bcc7e2e@intel.com> <20220121172713.00000853@ramaxel.com> <3982cadb-5b7e-22b4-d8e7-091ff22e4702@intel.com> From: Hemant Agrawal In-Reply-To: <3982cadb-5b7e-22b4-d8e7-091ff22e4702@intel.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit X-ClientProxiedBy: SI2PR01CA0032.apcprd01.prod.exchangelabs.com (2603:1096:4:192::18) To AS8PR04MB9064.eurprd04.prod.outlook.com (2603:10a6:20b:447::17) MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: bc23cd50-6dd9-4385-5d6a-08d9def8306b X-MS-TrafficTypeDiagnostic: VI1PR0402MB3821:EE_ X-MS-Exchange-SharedMailbox-RoutingAgent-Processed: True X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:8273; X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: Y9M1GR/nBKm9xJ2IPJx0junxi6TIpsLj4HF8gIM8VdCI4CgvZC/f874NBx7DPZNyV7MlfOsJ2ACYjRiOT4llQ3hZ6DRzfWKOIyTIFYSduqjWTL6mS3FCztB+5DzmFPGrYsJ5sRVEq942CPTIblg72kOTEojrVomhlJrQN669vrTbxm6s765zpD/Pio4RvuIi0UkE60crS9X9wJyHVJvR6J67dralLFLxqSCvk82kPOU0CVbRyOtN+tLdHVdkpUhGgcm1qIohZMamW2KgehJAc/OyrASILHGJuVlVFOwdvcZqmMErxX5mxZt7pGAv7ENlCexTbz3KdxZicoTKhfPr61BV0yx9F78K0o/HZLDkne2lE/5ddjhMR9Vb25pb1WUDXDH4O5BJwpjwrdykts9JGVoriQ4KlofuJq1pTzK+L05F8D+UVk7dwmZmJtJQzbeSfyXd6WdIKN9NlnH53CkFZaocu22W9ASrYeiV6DYG1yJaB7+EFEjDZZqbQgi+5sjwKnn7IU3QteP8hBkpsDWG+KzWP6bcbaYnSIe7dyY5ZNJ54pb9QsHd3EDmqGxCDixm2unVPY9vx1GwmY5Y7iEncwfZtJyVCzl5C9dFsHcnIp1kT9gcWyvEGA9yf4JuDyUX4ri1PkhHI+c3y1lAVIx6xcd61o0sPyd5abHYKkwaocJWQVMpXOi28s9xsLWtX27ZwQwbmYDGRPbBd46Q5pBaKQqQBo5fF716VAoPKMNU3a+4CUoCSp6jMTMzjVHlTndMu8/beM55Q0Q21i/aswzRaw== X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:AS8PR04MB9064.eurprd04.prod.outlook.com; PTR:; CAT:NONE; SFS:(4636009)(366004)(508600001)(66556008)(66476007)(66946007)(31696002)(53546011)(26005)(6506007)(38350700002)(5660300002)(38100700002)(6512007)(44832011)(8936002)(86362001)(2616005)(7416002)(110136005)(54906003)(4326008)(8676002)(83380400001)(6486002)(186003)(52116002)(2906002)(6666004)(31686004)(316002)(45980500001)(43740500002); DIR:OUT; SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?UXpLeUVvUkhwVnlwL3d1aFRHN2s0RHJHeUFjQVd4bVZiUGk2WG1XQ2NVREY3?= =?utf-8?B?YW4vQ2ZnNlBXekdDS1pYSnNzQmNvTGF6WUx5ZjFJWE5NeE9CRlBzVFluc1FV?= =?utf-8?B?Q1R4VkxmbmhJeG1vK2NwSXl5NXNvNGhVeU1zTkQxRDNuZjdzVnczMzBZaXJO?= =?utf-8?B?ZDdnSVJFdG1tdWo0U1JDbzFNbGoxd3FzNkFUejVnTzVvbFF6dTd5eTFCSGNX?= =?utf-8?B?dWpMVUJjSXBxcW4rcnAzSm5vakFhZUVaRlZicWlEaXlJY29hNFp5bFVmeSsz?= =?utf-8?B?bk9TSk80WE53S3JHY0pNZUJYOCsxdnpEWUhkSDdUTXVBenkwZlRZNXk0alZZ?= =?utf-8?B?N2JtZzVZbW9keW0vcVlCZ0tmWFFwZFlISXBybUU5U1d2RlBhYllQZmh6bTNZ?= =?utf-8?B?WktNMVdxdzRiVDZhTzFDTXUxVDI0N2N1MmtsQWFYVmFhcnN4THErcVpVazVQ?= =?utf-8?B?aUxvejhLd05aMDU5a2RsM1ZNRlVnMENpc3JSdmE1VjhLam12bDVmQWFNSzVE?= =?utf-8?B?NTQ5dTFlSEJkMFF6WGtTOS8wUVluamV1VVdvTDcwdEc5aE9tTWNpK3VTLzg5?= =?utf-8?B?b2dhd0tZNDgvc3lLSlh6eVZLUGpJcGx3enB6cmw0c1RUMURvYzFaUWY3dUt4?= =?utf-8?B?aHVyTmJhVU95Vlh4OU9TK2FyQkREYUdaVkdXRkw3ZldpbUo4SEhxQS9tdnNr?= =?utf-8?B?UHpNT3ExdldNdUIrV0R6OEdJY2NpbHZGNk1aWHJ3ODN0dE9SOWtmQ05TdUVN?= =?utf-8?B?aStuekwzaXdodE5aUW1YRGQ5K3AzUHlRbHlXRHNqOWFXbjZ6MC9mQkdWdWJK?= =?utf-8?B?NWNwQ2lBeVh1ZXE2U2d3dUNtTVNJWENQVmNuNFJjQnZmVnk3UWdzdjdJZ1d5?= =?utf-8?B?dm5jN05hNS9pd0FxNmN5THQ0MzNaSGRCbjZwOStNc3dabDBXZ1ZuZjU3RlN1?= =?utf-8?B?ZHJQQ3UvYWpMTHpIYTcyM3p5STBrZW96STRIUS81WWlNeExjVkc1M0Y4SnBH?= =?utf-8?B?ZW5GaFE4eVd1czVaZlgrUXJMVlVucTZwQVhmWTJjS1ZBTCtSZWpDZlk0L0Fu?= =?utf-8?B?eVJjUGVMSkk5czZSaDAzMlJlVk1sZTFJNUZTVHhqSnJEVnRTb0o1eitqbVJi?= =?utf-8?B?V2owSmJzLzk5MmQwNHIrdmgvVmQ4K3Jsano4dFY4amkwbFlIa1pmUjZteGFB?= =?utf-8?B?QTBwNUtvZGNzZU4relNpWFBnMUxvblpDNDBUdG1aeVBkWUVaM2lOK0VIYjM0?= =?utf-8?B?c3ZWeFlXd2hTT1BkRjRDMEYzL0lhMHQ3WnVIcE1hS0N3SmFmM0tmSnMxT1Fl?= =?utf-8?B?SUJRS01JaTcvenU0MGRtWjRJTlZTcW9aeXlxN1RTMXR4NjJrbE5lYkVtbzU1?= =?utf-8?B?ZldWcnB5NVBiaW9oUkZXS2JxT2orZ3J1bmxwaEk4dTFMa0J5dDlPMTA0QU9a?= =?utf-8?B?Yzc0OHZDTlF3NFJjRW1UdVUwbFdUblQ3V3RDeEx2UXkwdU90ZnJ6U2I1OE1q?= =?utf-8?B?SWMvUnZVNTltVm9jUVNSVGgxSWpoQWdBSWlNWjJSYWl2UVY1cFdMY0E5NXNC?= =?utf-8?B?aVcxQXp3ZDBnbkhhemFBQmZMZXVMdE85QkVqVU5QVEh2cmV3Rml5clU4Ty83?= =?utf-8?B?N2p3UWEvb2hUaGt5UHpaQ2VJVEhvRVZYaTRjV0FxK2xGYWpIaWlaUG5TcS9o?= =?utf-8?B?TThsVnQ0bkMxY1had1RnV29idStRcTRYbE1HMVlkZDhYRTBJVFM0V2M4Qzhm?= =?utf-8?B?MlMrZnBrMWdVVUpIOFllaGJ0WGY5MkU3a0wxb1lrbEkxd1BkeHMzV3h6Z1ky?= =?utf-8?B?OWNuZTNVdncvV2ZEQXdhTVVNQWF5RmI2aWd4N3lNcEoxRy8zSEVZTHZzaUc3?= =?utf-8?B?N2NkU0RTKzR4ZW5IMlROaWhOd3A0bUNaR0tmZVdaSFdJSTc5VGRZQ1ZOODdx?= =?utf-8?B?clhEeHErRUFSRGdGWkJuOE55bDVsL1hTYUJhcDhtMUtLVUpuUStsNUY2cjBG?= =?utf-8?B?cnpJaGpJTFVJOTlpMnp4QUpsZlBONE14YWtRdlljb1Y3UHJEeTJCUE5DNytR?= =?utf-8?B?MndjcldwVGIrMGp2N0JPZlhRMytXVkxYODdaSU83TnY1MHp2U0VvdE9TbzNG?= =?utf-8?B?amY0UUJCSjdibkN2RUFpWjIxaEU2ekxlVkxQZTJRRGJMREVYbXRhZmx2SGRr?= =?utf-8?Q?vXR51GtPwkkQ22Vz3QPDuLM=3D?= X-OriginatorOrg: oss.nxp.com X-MS-Exchange-CrossTenant-Network-Message-Id: bc23cd50-6dd9-4385-5d6a-08d9def8306b X-MS-Exchange-CrossTenant-AuthSource: AS8PR04MB9064.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 24 Jan 2022 05:12:59.8671 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635 X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: iNCTeAUaW+RnGi94xZcji6GP1OpVknULx4JtbmzSxJO3/LNR7FR8tCt05DrB6Ea9J4PEvFVNUvFSz+Y3KUModQ== X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR0402MB3821 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: , Reply-To: hemant.agrawal@nxp.com Errors-To: dev-bounces@dpdk.org On 1/21/2022 3:52 PM, 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. > > Yes, if large part of code is driven from an existing driver, it is advisable to keep the original copyright. You can add your copyright as well, if you have changed the code. Alternately, you provide reference in the file header, that this file id driver from xx file having license type Y. > 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... > >>>> 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 >>> >>> <...> >> >