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 A63E345C73; Mon, 4 Nov 2024 09:46:11 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 96A54402AF; Mon, 4 Nov 2024 09:46:11 +0100 (CET) Received: from NAM12-BN8-obe.outbound.protection.outlook.com (mail-bn8nam12on2056.outbound.protection.outlook.com [40.107.237.56]) by mails.dpdk.org (Postfix) with ESMTP id A1AD54021F for ; Mon, 4 Nov 2024 09:46:09 +0100 (CET) ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=gIqTWbHux6f8z+yCfsaGfyyydzXJ4r5qW8+Bd6Tq+JKHFI2nvW4NjDkcQApcguYokvldSvY8BszxajfGWSgNpFcFtXkSR14FJb5aZfRulVALjEhizGawUzoR1UJb51pDsZsODQn/n2QnsAqtfGOWVZBzaV/0Ls76szAlue4Dl6ds0jP/0/EpmkonlN7V7qDSctj/j27KmK8E+oXh7JX7XY76Vt3Y80VVFekykvl7IBsD9FIV6mMBs822Uz3PXcHdsrk9EwnTPLQwR6c0AFloSxduOi/3ju6LN1HCnfChfZ5ji71/xJA3kKMGLXQ2paNyK3lfI0CDKdnVzWtbkHkFQA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; 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=uWyM9WTI+X1wVAbOtBK71I2RD+njzW3RH06npQrWPdc=; b=Cn6Ow/V6jmXls6rpd9byqv11h9ujqOUag653gsgPQt/oOofj4HZaoICXkZZ/0ha27RwD7okBHWvBrHhJgUb3wfr9BdAg9mPLUIKHxZFD6eOWWVL5ooawhWJL72+RPFlqPQiZKRBse/t/eaL6jZjC1rSdhkYc0wHE9wef7lPpzWXjUhWq0Vc0cUN6rA4mbePMBEp4wihXJb8twcjbQVLuB8JVjk9OkuVf5FXjR4nRRA2WKadwn1WozJ5VRCKUasNYnvuP0k8CL2TWyMV63fiawW2065iibTFUZZ7zhiO5/ng/iUhVn65WDptcqyokIXRblxNMcpwbslpRxet3IQ9FRw== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=amd.com; dmarc=pass action=none header.from=amd.com; dkim=pass header.d=amd.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amd.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=uWyM9WTI+X1wVAbOtBK71I2RD+njzW3RH06npQrWPdc=; b=11YiXpDCoQ+kcI73e/MY8ztBqoP7cvzdoovj2/J298wxZNBJ7tUdp6ET+D/R8hji//esX+zHOl7yeJP/gRpyyIuSjWoa31SX/IMi20FoTKclwIllgmlCckj4LAdFeUkT8MxqUOMPknpl44Dv92RpBIKjga9Zxe4EMAFD2hLVqaQ= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=amd.com; Received: from SJ2PR12MB8830.namprd12.prod.outlook.com (2603:10b6:a03:4d0::9) by SA1PR12MB6679.namprd12.prod.outlook.com (2603:10b6:806:252::14) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8114.30; Mon, 4 Nov 2024 08:46:07 +0000 Received: from SJ2PR12MB8830.namprd12.prod.outlook.com ([fe80::c3eb:df02:eaa9:2055]) by SJ2PR12MB8830.namprd12.prod.outlook.com ([fe80::c3eb:df02:eaa9:2055%4]) with mapi id 15.20.8114.028; Mon, 4 Nov 2024 08:46:07 +0000 Message-ID: <5668533d-4105-4803-9d99-b45a4a2b86e1@amd.com> Date: Mon, 4 Nov 2024 08:46:00 +0000 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v9 0/9] net/zxdh: introduce net zxdh driver To: Junlong Wang , Aaron Conole Cc: dev@dpdk.org, dpdklab , David Marchand , Stephen Hemminger , Thomas Monjalon References: <20241101062156.2637339-1-wang.junlong1@zte.com.cn> <20241104024233.2775676-1-wang.junlong1@zte.com.cn> Content-Language: en-US From: Ferruh Yigit In-Reply-To: <20241104024233.2775676-1-wang.junlong1@zte.com.cn> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-ClientProxiedBy: PA7P264CA0079.FRAP264.PROD.OUTLOOK.COM (2603:10a6:102:349::14) To SJ2PR12MB8830.namprd12.prod.outlook.com (2603:10b6:a03:4d0::9) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: SJ2PR12MB8830:EE_|SA1PR12MB6679:EE_ X-MS-Office365-Filtering-Correlation-Id: 973e314f-3926-435b-4d3f-08dcfcad1fba X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|376014|1800799024|366016; X-Microsoft-Antispam-Message-Info: =?utf-8?B?MzFCaWJ6UWQ4allrQlJTWUUrcDc1N2pGSnJRaG1RYTB6Sm9YZ1RGa21ibG9r?= =?utf-8?B?WExkU3FWcGdtN2x1UG40TDJwZHhmbmJlaFNUYmVHempuMlp2cjBVMUpSdFpk?= =?utf-8?B?V1JFdG1wcHhldHBIb3FNcGhqRmtxWW5QK0FGSm4ySXc0Vk01TnlSNXJnNS9Q?= =?utf-8?B?c3haRU4rOFBUTFczOTNVL2cyS2xQcWdzTlppZXFOZEU0Z1ZSWW5HRERWNXdR?= =?utf-8?B?ZXBUY0hTZll0MnJVVHZZK2FPR3ZZMks3K0grV05FdXUyRGY1L0pGTTNnaHhq?= =?utf-8?B?dDBaZW8ycHp5ckJnY0QzdkxGV091Z1lteWR5TUJRcmhUbnMwbU1zK2E1dnVH?= =?utf-8?B?Wi9ZSTdGSXhOK3NnOFVwbEp6MFpJWTN2dk1ybi9uazJZNWZlRGlCN05QdkZN?= =?utf-8?B?OVZUVkd5UkcyQmpnelVDY3pNbldPTGNMMEpaK2JMRktyWVNpSmJKQ3YvYkV0?= =?utf-8?B?L3kvUERORjNTck9EaXNOLzMrTHI3N2xCeVROUTlvS2VYcGlVVEtWbXlFd1hS?= =?utf-8?B?QVNtQnU5bnI4UG4vd3UvL24ySjR0ZDVXMzlyc0JxTnNVRTZTelFNWWV3Yzd1?= =?utf-8?B?MkV0OVUwNjE5THRoTnp5enpycGZQc2NJMldQOHBKaitWK3pyZDFEWTkxdkZ2?= =?utf-8?B?OTZMd0ticXpMeFllaEJrMUxVR3NXdTJ1RnhpMjdDaU1LSW1pWU1iYjhwU3hn?= =?utf-8?B?cjNGNGhnRHZyVzgrOUhhZXFvZG43UGVHWDZObHpyZFd3MCtWQ3QzSXVSM1Zx?= =?utf-8?B?VlFUcllOdHV3eG9NYVE5dkE1M2lXTTNtS3ZPRTJwRFFwR0VIT2QwN0pqUzlU?= =?utf-8?B?QXhqRmRzQUlmVlY1VC9YVFhFQUJTaTBxRmpQL1VNVU9mQ1ZGKzIraXZYclRQ?= =?utf-8?B?Z3NSRlMzV3l6V1RBdjVHWWx2aWVjOGNPV0ovZ0szRVo2WkRaS09oZDdDM2pH?= =?utf-8?B?WEJpaUR2T3pzWlBpallnT2ZBcW56SXkrb2JXRmJqSXhOYWFEZTMyZGRNdjZE?= =?utf-8?B?dTUrQ01yY3VRS0xzLzZ5YjZ3dGgvRksyRXlnOXMzOTdjYWt1NW5KSFJnU3dK?= =?utf-8?B?OWx4R0pyTmVHSUhwSGE5RjdpbWRGcmxETVUvZGlseGIza25uYnRVTzF2ajJD?= =?utf-8?B?MUp3S2RtaTRhVjJHWEhRU1Z0OVk3U3ZuSWlxenFFYUhOUFVqc1RDVjUweGdN?= =?utf-8?B?dTdCQlJOcDlyL1MzaEJMakFFN0ZleXV3SU5zYk4zUGlCWHI3MVg4R2dQTVpN?= =?utf-8?B?dW1FVW1oelV3Tm9BN2RTbGYzVExRaDA3WHVIb0IzMVVQRjErbUQ1TU1IUnhV?= =?utf-8?B?MHpFbll3TUZsVnM0TE1hcHZ5YUpBZm5EYVh4ZjBPWWFUTy9KNWIwSm1maENu?= =?utf-8?B?cHN6RUh0NEM1S0FWRkRrVEM5MDBldGNVaTlSc3B3d3JENUNWdGNLSk5jQXhh?= =?utf-8?B?RWU3N3M5TFNZMFE4azVURURoaHJYck9ORTVZbm9pemRLVEVxR2psSUxUekFE?= =?utf-8?B?cWtvZXgwc3Znd3MzOU1XUnV0MWRFNWpYWk4vc1h3TnNKa0I5UjI4ZFB1U2tH?= =?utf-8?B?SExJc0ZoMlhUdjNTUEkxT3ZGem0rUTRkaGgycUtHUlYxaHE0eXBrL2hXSFRI?= =?utf-8?B?K3hsSlBZOEQ2bnJzV2xZZjBrOHJGZVZQdnk4cE50VGhCdnhVR1ZYK09FaUJI?= =?utf-8?B?b0NraUlyc1RGV3MyWmJzQitHb3gzTEx1WHpRZVVZaFVUOVB2VWh1clZBT1NM?= =?utf-8?Q?8JNP8i/CGw7/33D3XQ0krzLciHrH8D3vuANzT9c?= X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:SJ2PR12MB8830.namprd12.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230040)(376014)(1800799024)(366016); DIR:OUT; SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?WW4vMTNiK0h6bEhRblpPTURJMmV4dXFhY1lET1RZL25udkhsMnpLWWl4RWsr?= =?utf-8?B?WFB1dG9GTkNFcnFIUThydEhFdXd3ckhwcXYwNDhrYzk5STRvcEFaUmZVcVJO?= =?utf-8?B?UEVwbWRuUUhpYmZ1VmNadUE4TnJvc0E3aGc0SFUzdkVWYlZUZFJsQ21ERzZ4?= =?utf-8?B?aEtlcWNFWWlpTWd3b3dtUzJuZ09lbm42M1g3Yy9rd2lXSkxjcEpUbE93eGNh?= =?utf-8?B?NGNQTjhHUGQ2b2RyMDNNTnlzRkhTUXhzaWhBNXEwN1h6am5kQVhqNkgxbUFK?= =?utf-8?B?bkdZRFRMVys5aTJ6RXhZM1EzWmU4ZzVyZmdLQlZ0VGxpWitRVUxLYWFhTGMy?= =?utf-8?B?aUU1b0tNckY4S0d3QndrRU16aDlCNTdFbkduNVB2K1pWbTFZd21iY01EVEFm?= =?utf-8?B?aUpRM0FuYlNGZWg4bWZqbmo2T3lxRjd2U0VxZXhCYmc1cVRrRmhEaVd2MC9a?= =?utf-8?B?SnptWkFVR1dxZzdrK3U5SnJkd2prY3JubkFvRjVCSVVCaGNaR0lwaEN0UUdD?= =?utf-8?B?L1JXTFdvZDNWRU9zTDBaYWdRMVRJUzFiMGNCSTNodnZwUDQ2WWpKYTdKT29W?= =?utf-8?B?dDlpZVMyYlYwT2FsZm5ad2FwY2RCRnBpNytwOFphdGZvWjd5M2wxdzB1YUJI?= =?utf-8?B?MmNRRkRtczVibkFqS2lJQlBmcVhhUGdCQXM5cnh0S1N6VEdud2w0L1I2TStk?= =?utf-8?B?amNuV3JIYmhQSEl4dmhBS3NjMDRzTTczTUlQNUhJa3JCNk5Jc2V0dElpc3d1?= =?utf-8?B?SktqVE5WQTMxcXJ5SHI5REgyai8vcnNQUDJzN0tJZE1pQVRRUFhSZ2cwOGti?= =?utf-8?B?V3JoUHpHc0xUZCt3UFRPNTZlNjgrNFA3bE5SSWlONHd3dVZxVWtuZFJ1M1hm?= =?utf-8?B?U21YUGxKYVZTMG5JdmM1S1RCQnRES0VPeG5xVm5DWG02RTJBazNGenI0RmY2?= =?utf-8?B?N2JKYmsxdmtwNlBpVHlrZnpIVVlBWXYxUUp0ZFg0WmxvNG02YTVHVU1Cc3Fn?= =?utf-8?B?RjJSZlFQdVdvb0RyUStVZE9oMHlnMzZ5bStLUlI1MDd0eXA3bCsvekVvdEZs?= =?utf-8?B?UHplQmFIV2J6aGRBeHhNZXZtbVlFbkxISkQyRFlHcG9ZQWNid0J6R3BSVGxT?= =?utf-8?B?dmxILzNOcm5UZXhNcnRHSEtYaTRtNU01Z2lvT1VGYkdIOE9YaWpvSHpWSXRx?= =?utf-8?B?ZDU0WDIrdit1eTNVaVAwb2VSdmsxUVo1TkYrdExYNVp5OHN5bzQ3RzMwckhM?= =?utf-8?B?L3p1T2pUUUhXM3NMVDNRVENFUFBZRWhtaHVBV05LZWNZQm0zanM0UnJDMmhL?= =?utf-8?B?aEREd1RFS0g4cGlpbXlGRXJCUWdEVjQyVG51TmdrS1JtejJrSVAzQ3BsYkhF?= =?utf-8?B?dUtUQUpHVHdwS3JvSzNKT1BzaFdyUEs1T290a0s3QXlJckpIRGR0eWhXNmc3?= =?utf-8?B?bnZESEV5TnYyNXBCVGJ3QWwvWTNFOXkyVVcyTlEwR1BMVGNPYklCMkdIZkxt?= =?utf-8?B?Z1dpM3lBalVBQmtzOHMyckZZVUpqak1aejJGZnNydFRmUUVmRTZqYkFYM1FP?= =?utf-8?B?TXREVDU4b2p4bGIzdEpKN3VpVWxTVFRMc1hZdHhrZFdway84L3dDTExmOUw1?= =?utf-8?B?Vzd4TnZiVk1IdWF2VDZBNUtaZHNoRHRMNjVFTWI1K2x0NXdpblZ5eUdpa0l5?= =?utf-8?B?OWc1bkpSdkZ2dThjZHJ4NkhjUkNHRzFmTWtCYmRvSy9qdVI0NzN6U0RQbVpG?= =?utf-8?B?YkZyVkpsMzhkWGJVcThMbFN5NWpvOFBVd01sVXdRR0dIdElqaGFwaDh2bUpH?= =?utf-8?B?UFBiYXhXcU9yM2F5VEtFT3gzbU5aOURwRzBmeUt1NUQwYVhDbnNkTHdDQ016?= =?utf-8?B?dWVmNUwxcFFBMkQ5R09FM3JQcXJOTCtwRk90b2JOV0t4UGFWVEJ0MWxRbjd3?= =?utf-8?B?WGUya3pENVBod21wbHdIeHc3OXBGQVF3YWxMWTJ6Wkh5UzRuZ0Q3UDJkOEVO?= =?utf-8?B?SEZUZmFmbDlCTHY3VWEzQktkWnRza1IxR21xRWFyU0dKY2s2T05md29QU0Zk?= =?utf-8?B?cGRyRm5DeWc0Sm1oUy9zb1BhaDVnRWNVcHJuZ3NKaENndFRIc1hjWC9hdjlX?= =?utf-8?Q?IYz7G9h+WUDmcfk8HbLxXMkiH?= X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-Network-Message-Id: 973e314f-3926-435b-4d3f-08dcfcad1fba X-MS-Exchange-CrossTenant-AuthSource: SJ2PR12MB8830.namprd12.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 04 Nov 2024 08:46:07.3978 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: BZ0jCS7s3siOAKgheVUUtGSuu3y+/sdB78zHEWxxZu6iDNS5ahKdCpGDlpC1g8W0 X-MS-Exchange-Transport-CrossTenantHeadersStamped: SA1PR12MB6679 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 11/4/2024 2:42 AM, Junlong Wang wrote: >>>   net/zxdh: add zxdh get device backend infos >>>   net/zxdh: add configure zxdh intr implementation >>>   net/zxdh: add zxdh dev infos get ops >>>   net/zxdh: add zxdh dev configure ops >>>  > >> Hi Junlong, > >> I can see not all of the eth_dev_ops implemented, and datapath not >> implemented, so driver is not functional right now. > >> What happens if you want to run testpmd with the current state of the >> driver, I assume it crashes? > >> And what is the plan for the driver? Are you planning to upstream >> remaining support in this release or in future releases? > >> As the driver is not functional yet, to set the expectation right for >> the users, I suggest marking driver as experimental in the maintainers >> file and document the restrictions in the driver documentation, also >> clarify this in the release notes update, what do you think? > > Hi Ferruh, >       The complete driver function has been implemented. At present, the driver function is integrated in batches. >       The integrated PMD does not cause the crash but cannot support all PMD functions.We plan to integrate the driver in this version and finally provide a complete and available driver. >       In the current phase, we will marking driver as experimental in the maintainers file and document the restrictions in the driver documentation, also clarify this in the release notes update. >       Thanks. Sounds good, thanks. Btw, build should be fine after each patch, but in this patch series there are warnings in some early patches, can you please fix them in next version? @Aaron, @David, what do you think to add patch by patch build to the CI? I hit the same issue in multiple series in this release.