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 67F37A0C4C; Tue, 16 Nov 2021 21:08:09 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 0FBA640141; Tue, 16 Nov 2021 21:08:09 +0100 (CET) Received: from mga05.intel.com (mga05.intel.com [192.55.52.43]) by mails.dpdk.org (Postfix) with ESMTP id 3734140040 for ; Tue, 16 Nov 2021 21:08:07 +0100 (CET) X-IronPort-AV: E=McAfee;i="6200,9189,10170"; a="320030096" X-IronPort-AV: E=Sophos;i="5.87,239,1631602800"; d="scan'208";a="320030096" Received: from orsmga004.jf.intel.com ([10.7.209.38]) by fmsmga105.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 16 Nov 2021 12:08:06 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.87,239,1631602800"; d="scan'208";a="604447404" Received: from orsmsx606.amr.corp.intel.com ([10.22.229.19]) by orsmga004.jf.intel.com with ESMTP; 16 Nov 2021 12:08:05 -0800 Received: from orsmsx611.amr.corp.intel.com (10.22.229.24) by ORSMSX606.amr.corp.intel.com (10.22.229.19) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2242.12; Tue, 16 Nov 2021 12:08:04 -0800 Received: from orsmsx608.amr.corp.intel.com (10.22.229.21) by ORSMSX611.amr.corp.intel.com (10.22.229.24) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2242.12; Tue, 16 Nov 2021 12:08:04 -0800 Received: from orsedg603.ED.cps.intel.com (10.7.248.4) by orsmsx608.amr.corp.intel.com (10.22.229.21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2242.12 via Frontend Transport; Tue, 16 Nov 2021 12:08:04 -0800 Received: from NAM10-DM6-obe.outbound.protection.outlook.com (104.47.58.106) by edgegateway.intel.com (134.134.137.100) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2242.12; Tue, 16 Nov 2021 12:08:04 -0800 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=iFuN9iFCNcAD9FTHUQ730a62iu+I2nQc+l8d6st3sLwBSTYulOdtGKRoK2AnmL5pmwL8ND4LbppohGpoRWMwUi/TGUx2MjVuhThdNgnIGWXBmq4ej0GlbwFdXcVGF5UhIy8QZ6ByYahiM89e5b5b6jgXBC2mkDKejmDb55VNz9Wy3kGBjT3S96oD2z5Fkh7dP9xLJuWA0kSUE3vfe0+WN3n0nZfaNYb4ghFT52WsdbyaZmgTxPPPXFVtVkJog1GLL8YWBznHuysyPLGRByY85HSniI/9+WzK1aplo+q49h2povmsQz4qfyMzrgktPqF+dw0sr+n4t13pW49tDVczdA== 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=6bU0karsZypymq6tURXH6XSbLkV7ZHX1qfgk4yCJd7k=; b=CjRKOUnx8jI2qEF02c17TemdXrkn2dIoZoRsZ3U4hr6z/TwTiJG3fDCBo6mCX8OiXgMrLm9ukvl3vO2/G7iwYwcT+RPXjE4cndP13NDR3e2wujmPg8a4PvtEur0hmvxPol8J5FclAYl6+mLAUYtlutRFmQdoNtwZO6ezyzptYQfIgjDbGtdbwHO12FLyzXcOnE5izUFrMA7VwXRpb35PKa6tqqMd2lkP7Qv4H5cp3VMbWLWfwBcM0iDcC/nJFbWBSBPcqQ0ufNpWAFKxumPnePQFFs+rGHrEhCT1Q1BKFsoKrEmA35obYa9iyeGb2twQQDba5X3tZwZnzSOj7aQRBg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=intel.com; dmarc=pass action=none header.from=intel.com; dkim=pass header.d=intel.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=intel.onmicrosoft.com; s=selector2-intel-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=6bU0karsZypymq6tURXH6XSbLkV7ZHX1qfgk4yCJd7k=; b=SYEG3WfWcw6o8ByAvCQkOQOCQMNGU4mc8/YEwTfhdAdmDGXF7yyBx0wC92+rhj/aPMwKjfhl71ByW0wws5uUlKBfy3GbZvcTJjA+ySSPkbEb1438YoWBMR8W6MK+74QYDAqqIyTf9R5j1ONlc3o6Sr59dsbl6uefrcZC2qf6wdw= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=intel.com; Received: from PH0PR11MB5000.namprd11.prod.outlook.com (2603:10b6:510:41::19) by PH0PR11MB5061.namprd11.prod.outlook.com (2603:10b6:510:3c::21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4690.26; Tue, 16 Nov 2021 20:07:57 +0000 Received: from PH0PR11MB5000.namprd11.prod.outlook.com ([fe80::bc5f:31a7:10ad:443c]) by PH0PR11MB5000.namprd11.prod.outlook.com ([fe80::bc5f:31a7:10ad:443c%5]) with mapi id 15.20.4690.027; Tue, 16 Nov 2021 20:07:57 +0000 Message-ID: <1a3f1e7f-06bf-ef39-6819-339b0a8a29d4@intel.com> Date: Tue, 16 Nov 2021 20:07:49 +0000 Content-Language: en-US To: Tyler Retzlaff CC: Konstantin Ananyev , , "Ray Kinsella" , Thomas Monjalon , "David Marchand" References: <20211116002440.GA17166@linuxonhyperv3.guj3yctzbm1etfxqx2vob5hsef.xx.internal.cloudapp.net> <15f9b491-6c36-dc85-6348-d746196d4374@intel.com> <20211116175453.GA16051@linuxonhyperv3.guj3yctzbm1etfxqx2vob5hsef.xx.internal.cloudapp.net> From: Ferruh Yigit Subject: Re: ethdev: hide internal structures X-User: ferruhy In-Reply-To: <20211116175453.GA16051@linuxonhyperv3.guj3yctzbm1etfxqx2vob5hsef.xx.internal.cloudapp.net> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-ClientProxiedBy: DB6P193CA0016.EURP193.PROD.OUTLOOK.COM (2603:10a6:6:29::26) To PH0PR11MB5000.namprd11.prod.outlook.com (2603:10b6:510:41::19) MIME-Version: 1.0 Received: from [192.168.0.206] (37.228.236.146) by DB6P193CA0016.EURP193.PROD.OUTLOOK.COM (2603:10a6:6:29::26) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4690.15 via Frontend Transport; Tue, 16 Nov 2021 20:07:53 +0000 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: f5d0322c-8a6e-4d2b-859d-08d9a93cc878 X-MS-TrafficTypeDiagnostic: PH0PR11MB5061: X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:2000; X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: LJrHWHuwuVJ3WCeLRVDIs9KeUnvzGNCRDyoIn+9JGGpzlfy4caUiHnhk9ZIVo7fUZ/+K5qdPThpk9FRrhoXBZUgi3oJd1U25c2rwyBNQJ1bCzFPC9EObxHyFvW5j4IdHJvqSwka6gZ81zR5WD3MJW6COz17T8p34mHSqHHSYv5o4lQTIJUkT7B/NRzEtgnL0e18osGgJj/k6PiNqJk2D68XDWCn4nZsVWYf6VbarbPUHuHPolA1A81PiaTd+w7OqscHK0gakaX4PItuVti1XtbPz7RgpEq9iBLpmApbPKiTjVavl1Ppg5nu+hJ1anX+4KTmSMU2/IU4Tq6jrHXUvnFVQlvTqa6M5lxNS88eSX09UPAcEUvytJqcALDBHg+RIspOzVSmBYTFmQ0YHc8E+R7tkRJ3ShMlTAXMEMGmhVQWtaogasA+pPRmGuhHYaIdE5cNNKRA+DgDVb4MxySlpo8VW8LCGba/19m1MvlElhMgfiHfoxkPq9iOtfXvIM+j8Bsrl96BA2y65pXchi0lizubCCU+qlbImCX1giVWVVm+sz7wKNVCmqiDeDlo+R7tB/dyYOMxa/33rp3KGVWylTTPe90zujA0HRnfHy8VvL6660ZWHnH4T7HgppjBgYBn5zlRQeHcYRO6p/zeolDHzGb0Fa+Yc5rRnvoCXEgpjKspJlG4ipybMorJMpnnh/R/fVq7zYOcMDjm06Bh7fcZEPCQibyTd29E9FJuALx259UgK3xgVp+c9T7GDcNvzZ/r6eNBXE3MDek2VT8r7No4lmoEz5mJWh3LoS6VCzC7WGSe9ZYAwTL2J7tG/+mP//sCy8TN5xk1eZ4cpEqeBfAksCg== X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:PH0PR11MB5000.namprd11.prod.outlook.com; PTR:; CAT:NONE; SFS:(366004)(4326008)(8676002)(86362001)(6666004)(31696002)(66476007)(31686004)(8936002)(66946007)(44832011)(966005)(66556008)(508600001)(16576012)(83380400001)(186003)(36756003)(82960400001)(6486002)(54906003)(38100700002)(26005)(316002)(2616005)(6916009)(53546011)(2906002)(956004)(5660300002)(45980500001); DIR:OUT; SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?NjgvUTNqUFpDcGw5bjNReWVlWDBSWmJoaFJwYWNxSHFyVW1jdHlweHJtUG9k?= =?utf-8?B?S0NHU3hVNHI5YUhsejd6VURUek5KSERNSTdtS2lGaTBHVkRBS0dMZ3NHak9E?= =?utf-8?B?cjdvNG0xTWdPdTRaTVUxQVFUd1RDSFh1UlpGSml0bHk4SFlyQk9HYzRnQjNP?= =?utf-8?B?dTFud0V0dkcyWFV5ajIxQ3I4R09iSVAvN2prRXBoalgxSEFUd0xQMjNWRDYv?= =?utf-8?B?TnVUdGhYQjVnTkR2dnhtM2JJM25PQnhueUFoVG05ekVaWFhXdEMwLzNBMVg4?= =?utf-8?B?RkI3ak9TWVhQVG80UTMwUU9kSUNiSFdCZWNqWWE2LzRCQkVmS0k5NFo4cmQx?= =?utf-8?B?VjU4WlgrTTZrWFJXcjlpdFR2MStGMUk5MittSlVRQXZoQTdON2hEVWtrdjcr?= =?utf-8?B?Y3BtWWZwb081bzVPeGpoTUlaMC9iNnhiTUJmYzZJb0lkZlpOSTJtWjVocUZQ?= =?utf-8?B?WXMrbEJDRGRHcjZjS2gxS3ZjVWR4NXpFNVZxdXRyUEp1TXo3K0Exd2xyUlEv?= =?utf-8?B?NW9BN2p4MXFWdnZZaXpzdFgxQityVzFnTGE5d2hRKzVuKzNQNkdCTWM0enpH?= =?utf-8?B?MFdQcm5mVTdwR21Bd2Y0UDdJOFNXL2xTSXhKUUNyVVFlclBDaTJFbCtRLzQ2?= =?utf-8?B?bjVyUCtITHdXM09KYXk5cjBPY280cXFtd3Z6OFZsdzRnbnNPMjZ3S1JORnlo?= =?utf-8?B?ekFwOVpKeFdLdW9QcjU5VFA1MlBKMGI3anZxcnNPbEpBL2lyVmxmMUZFTksw?= =?utf-8?B?NDQ1OE1EL2lCd2EzVENVelFUQWp0QnJhSVdKcytGUWR0S0ZiMjZubzdlMzdV?= =?utf-8?B?cGNpRjZUK1BSc203UjFVcnhiTS8vVmlRT2cvbE96Tm5XQXpYN2pHSlVjd2RJ?= =?utf-8?B?dnVPZUM3OVhCdXBEQ1RyS0JQVCt2dnozTXdDbmdLYVNIWVgweFpQdWIyMEVT?= =?utf-8?B?OXR6RXZpK2RrbENSZ0c4SzNoYVhBd0Y4eVdyYXVOenlqQXlHc0RLKys5Ymx1?= =?utf-8?B?cGwzb1NlYzhIckNvSy94b1VVRVczNHVDWGtSS3lRbzQ1ZjF0Uy9qaXliTDRz?= =?utf-8?B?MndHbWNPRHk4bGNrTXhWSHdjN1dDVzY2SHdvZm9yMkxlOG8zS3dlVFNabjlR?= =?utf-8?B?L0VCMG9ReGpnb0w4elRsL1o3L2NIMTZyeFdzZnZ0RDN3cnpTQ2FMSDRXQjRr?= =?utf-8?B?N3N0VEZKVkRNZk9kcUc1WCthYThsQ0hLMG1BRkV1aE5HclBlRkJpRFZxRElM?= =?utf-8?B?N2FDQUc0YVVUcHBzbE9RaWQ1OGJMRGp3YVJlakxsZTN3NWlNZk5zRjNpL2VN?= =?utf-8?B?ZUQrNkFEZEdUTFZLSjRhd0ZiaUl5NDNrek5IQlcrazg2VDIyVklDZDFlTVY3?= =?utf-8?B?VnBpUHRQb0pGZ1pPMUc5TnF2NWdPbmt1ZnduVFBJcldBZ3JyWk1KdG04N0Yv?= =?utf-8?B?aVFzVXZ6eDZOc1BJREk0ckFURU1QRDV5Y042M2RZbnNnVlhmczBzaTBzUnpV?= =?utf-8?B?WkVNbmtlaC9XSjdqS1kzY0RJMDFTNkN0d21KQ0JkSXEyUWZEZUhjc2ZYMWhj?= =?utf-8?B?dE1lNmZoc0lPbHVqVU1RenVOSzR4UzU0OUc5eThhMUlCQ20rNndUQTFJTE1D?= =?utf-8?B?Z2liSXRta29RWkh2NzFidFFBV3p6RXJsZEJJOFAzU1h3aTRPQkZBdzVLOXRW?= =?utf-8?B?QjVhcVQ2Wi9tYUNCTmJNWHd5MlYxYWMvaXZNV3Zya3dBU3BSNzRkeFRxSExl?= =?utf-8?B?YzMwOTFzbVdBSUtjUHZNRlJiV1diN051Qkp6MXdjWGZ4UmRSZ1NMR2VEOTI4?= =?utf-8?B?VmRaWTc1SnczOVhPSFhSclhpOCtvZXY0M0ZQM0R6a3VISmxaMjRNS3FQMll1?= =?utf-8?B?Qlk0dlY0bVgyU0dEWkJET0lLdTluWjg1TDVIQ2Rncjh4OXFXbHNUNmtGL3V4?= =?utf-8?B?SG9nNDhWM3U1VDUydTU0MTZ5VHRSQ2VWZmRuVEdBdmJocGxCY012VzF5My9J?= =?utf-8?B?eldBb1Q2Z2dOMGV6Z2dQRURTTi82NFNQeFFVUG9JZWtYT2NGK1JxWGwyM0xs?= =?utf-8?B?VW11MGhXUVBSbkE5S09UNW5tK0NGcEUwKzcwSzVyUUkrbExJN1BQS0JKcUxV?= =?utf-8?B?K1dhaFJaaHhqbFgrVllpQXpkQlcyM1MzVGhQQjl0NkdvN3dXQWd5YUZnczhC?= =?utf-8?Q?RYH1y0k0xSVIasbMfvWx7lE=3D?= X-MS-Exchange-CrossTenant-Network-Message-Id: f5d0322c-8a6e-4d2b-859d-08d9a93cc878 X-MS-Exchange-CrossTenant-AuthSource: PH0PR11MB5000.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 16 Nov 2021 20:07:57.8001 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: VuoXplgZwo1Xpc7+W5RqaZQ1FB7geXTI1umnchP6mnu/3kBb/TKyl+4mfJ6S8XXaiUFRi5dVn4/5FsG7JSgtLg== X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH0PR11MB5061 X-OriginatorOrg: intel.com 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/16/2021 5:54 PM, Tyler Retzlaff wrote: > On Tue, Nov 16, 2021 at 09:32:15AM +0000, Ferruh Yigit wrote: >> >> Hi Tyler, >> >> It shouldn't be an API change, which API is changed? > > exported declarations that were consumed by the application were removed > from an installed header. anything making reference to rte_eth_devices[] > will no longer compile. > > any change that removes any identifier or macro visible to the application > from an installed header is an api break. > >> Existing binaries won't run and needs recompile, but shouldn't need to change >> the code. >> Unless application is accessing *internal* DPDK structs (which were exposed >> to application because of some technical issues that above commit fixes). > > the application was, but the access was to a symbol and identifier that > had not been previously marked __rte_internal or __rte_experimental and thus > assumed to be public. > > just to be clear i agree with the change making these internal but there > was virtually no warning. > > https://doc.dpdk.org/guides-19.11/contributing/abi_policy.html > > the exports and declarations need to be marked deprecated to give ample > time before being removed in accordance with the abi policy. > > i will ask that work be scheduled to identify the gap in the public api > surface that access to these structures was providing rather than > backing the change out. fortunately it is only schedule rather > than service impacting since the application hadn't been deployed yet. > > i thought someone was responsible for reviewing abi/api related changes > on the board to understand the implications of changes like this? > Sorry for the negative impact on your product, I can understand the frustration. The 'rte_eth_devices[]' was marked as '@internal' in the header file since 2012 [1], so it is not new, but it was not marked programmatically, only as comment in the header file. Expectation was applications to not directly use it. For long term ABI stability, this is a good step forward, although the impact was known, best time for these kind of change is the 21.11 release, otherwise change needs to wait (at least) one more year. This change has been discussed and accepted in the technical board [2], and a deprecation notice has been sent to mail list [3] for notification. Agree the announce was a little late than we normally do (although only a month late than what defined in process), this is accepted by the board to not miss the ABI break window (.11 release). As you will recognize, not only ethdev, but a few more device abstraction layer libraries had similar changes in this release. [1] f831c63cbe86 ("ethdev: minor changes") [2] https://mails.dpdk.org/archives/dev/2021-July/214662.html [3] https://patches.dpdk.org/project/dpdk/patch/20210826103500.2172550-1-ferruh.yigit@intel.com/