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 B39DA4591E; Fri, 6 Sep 2024 15:07:52 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 4870A4025D; Fri, 6 Sep 2024 15:07:52 +0200 (CEST) Received: from mgamail.intel.com (mgamail.intel.com [198.175.65.9]) by mails.dpdk.org (Postfix) with ESMTP id EEF94400D5 for ; Fri, 6 Sep 2024 15:07:50 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1725628071; x=1757164071; h=date:from:to:cc:subject:message-id:references: content-transfer-encoding:in-reply-to:mime-version; bh=5sc/qwy9SQVB1azFlFzeLi8/QTYIVG/OV1LWCQyGpaE=; b=c7A/vdCbyWg5jTSFevpG7/ndD3/Bd4Hp3qaa1XZM5JXHCu1T5QnWpVYl fKdPP+YDVoId8zW/ZUu9cJmihmcWZ3bjv3SIrsr6Hm897Xvt7+ROfii6s WsFk+T2bgburQbhTChIVnSbO9OtNDygAcN5Z8M0ZXtZS/PY4HtB2blEYI sKLdtLv6Dt4FORR4+yeDyYLV4iEyy53RnIVvjhR7U+nhcB3NOH3eJCk0F 3C8Ch9MbkzX4SV3tzimqN/UCanEfW/oi+ImbQl4fMvYg0C/OILg40Ahg4 BWHA6wLYTydnAxUDz0HLORuTZE8nH4GgjQnZ02gKnSqX3oZVwBqitUpbM A==; X-CSE-ConnectionGUID: DFKjQ3vgSo2o+F24XGNUtQ== X-CSE-MsgGUID: 1412khspSx6WgJ/d6PDsiA== X-IronPort-AV: E=McAfee;i="6700,10204,11187"; a="46913376" X-IronPort-AV: E=Sophos;i="6.10,207,1719903600"; d="scan'208";a="46913376" Received: from fmviesa009.fm.intel.com ([10.60.135.149]) by orvoesa101.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 06 Sep 2024 06:07:50 -0700 X-CSE-ConnectionGUID: wqAQbNPiT1uBehCQAA7XMg== X-CSE-MsgGUID: pGN5fR2eSlar2mM8lQcX0g== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.10,207,1719903600"; d="scan'208";a="65943707" Received: from fmsmsx602.amr.corp.intel.com ([10.18.126.82]) by fmviesa009.fm.intel.com with ESMTP/TLS/AES256-GCM-SHA384; 06 Sep 2024 06:07:49 -0700 Received: from fmsmsx612.amr.corp.intel.com (10.18.126.92) by fmsmsx602.amr.corp.intel.com (10.18.126.82) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.39; Fri, 6 Sep 2024 06:07:48 -0700 Received: from fmsmsx611.amr.corp.intel.com (10.18.126.91) by fmsmsx612.amr.corp.intel.com (10.18.126.92) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.39; Fri, 6 Sep 2024 06:07:48 -0700 Received: from FMSEDG603.ED.cps.intel.com (10.1.192.133) by fmsmsx611.amr.corp.intel.com (10.18.126.91) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.39 via Frontend Transport; Fri, 6 Sep 2024 06:07:48 -0700 Received: from NAM10-MW2-obe.outbound.protection.outlook.com (104.47.55.47) by edgegateway.intel.com (192.55.55.68) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2507.39; Fri, 6 Sep 2024 06:07:48 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=Orcf0q4MoQem7rkfnfNpUCAo8eyVzKyM5XFEH3Zu7iU8FWD02X8kOxceq+8+cVAcMUfAVGEyONVTLNxRn5D+tEUo2G2/bWHUcfhKvEZquvxIlh83/5jeYLzouhg3xkaWGSMBuZCC3FJ/J2t/CIYPp+iFrHlAqog0AKk77pTtTbJKPO6jVOFobPbWsDJvdXjyyD863bWs1XSpSWlvwvoPQlhLq0w5iu+TaT+2sAwoTJjtwu0yDGZeE6XJ6QWQ77smMHf5J1FCdJOXLFlI9spkhx2J10z1Aj/nFMeZ/MhWw4iDZt+dLiWOpTfdzsVTTOZiUU6Jgj75Nht6A/dWk+uh3w== 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=DJBBDYOpyV83YDJrdNtrJeRlcEkA6+SJzdzpskGJRuU=; b=O+mMjkF/HD6j0FrPHbEcB0u0uCsbva40HDFb8+u9Fr5ePgPqcP8OkYNLURlzi9rXnHFtA+JRtryGeEmijPJhGFspsfCJySGD1SDzuPJZsW9ilEuEEA2or59I63ANlc6rVszmhApmv6zD29yfT/8vw0VNN9HLvz/gnMH606xcNco5hugkdjiHtUxVBwVtcwuKBNF3Yy8rhjkgt0KTEAnY7/FQIGm8H2GKKx2urmQgigxNXreCNnXyTeS93an+uOc9KU5xNxsxBrs6IP/MCVJp9gqugxIhme1tiWuSF33ZIUcXOa2VyazcMvVxHIC7BeLoiK11oNoxWSRdZrELM4krhw== 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 Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=intel.com; Received: from DS0PR11MB7309.namprd11.prod.outlook.com (2603:10b6:8:13e::17) by IA1PR11MB8149.namprd11.prod.outlook.com (2603:10b6:208:447::7) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7918.27; Fri, 6 Sep 2024 13:07:44 +0000 Received: from DS0PR11MB7309.namprd11.prod.outlook.com ([fe80::f120:cc1f:d78d:ae9b]) by DS0PR11MB7309.namprd11.prod.outlook.com ([fe80::f120:cc1f:d78d:ae9b%2]) with mapi id 15.20.7918.024; Fri, 6 Sep 2024 13:07:44 +0000 Date: Fri, 6 Sep 2024 14:07:38 +0100 From: Bruce Richardson To: Morten =?iso-8859-1?Q?Br=F8rup?= CC: "Burakov, Anatoly" , Subject: Re: [RFC PATCH v1 0/5] Adjust wording for NUMA vs. socket ID in DPDK Message-ID: References: <98CBD80474FA8B44BF855DF32C47DC35E9F6B6@smartserver.smartshare.dk> <84df3e19-f6cb-4a32-9360-a4384ce4c2ac@intel.com> <98CBD80474FA8B44BF855DF32C47DC35E9F6B7@smartserver.smartshare.dk> Content-Type: text/plain; charset="iso-8859-1" Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <98CBD80474FA8B44BF855DF32C47DC35E9F6B7@smartserver.smartshare.dk> X-ClientProxiedBy: DU2PR04CA0358.eurprd04.prod.outlook.com (2603:10a6:10:2b4::31) To DS0PR11MB7309.namprd11.prod.outlook.com (2603:10b6:8:13e::17) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: DS0PR11MB7309:EE_|IA1PR11MB8149:EE_ X-MS-Office365-Filtering-Correlation-Id: 32810b7d-5c74-4979-5285-08dcce74e599 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|376014|366016|1800799024; X-Microsoft-Antispam-Message-Info: =?iso-8859-1?Q?zzNX+DRyCaHZDuaJvHzW17Mi8MQvoxMDowjdilXZ3C4HCa2DbjHQo161W3?= =?iso-8859-1?Q?8flqZ2ShPkBlZHR2yi1ziQphGoVYjScPaz35Yy6sPTmVxb1K3qY7KkT11I?= =?iso-8859-1?Q?pnAXIGerNICCIJSi6XrBK7YxczLNHRT/oEZptl8kViaWhFgoEDDe9eUi1U?= =?iso-8859-1?Q?oP8mY6zjrtkLBR2itKiuMYzrOzdC/aayrm0m/BjNF9EXLCgwClew5z3fZj?= =?iso-8859-1?Q?djau8/Oo3gSYjuhq6xStOX24bPJaL/yghq8eiTaWuNCTGyAs6aOWkyyu0r?= =?iso-8859-1?Q?hh1gcm2cMVuYsiVZ6RqZaLYKzPZxHRstN7hpAbL3C5/0mR5Xg0EGzFc+mt?= =?iso-8859-1?Q?j9Uy6uUVSSgJO4TnFVJFVW34jBFLbuR3q4F4QlYN/kvxBpiehhJh8iy2Sj?= =?iso-8859-1?Q?0RHRK8n3i7M7v4v+DHzq0MfI23hUOmlgp2zYB5rrYWK3ytCjiS62JBZ7Db?= =?iso-8859-1?Q?nmr6NX+RhRNujdmvUjmVFQYNL7tt8hHitlL2T8WLL7Ql6DnvkJC6W6j6H9?= =?iso-8859-1?Q?KRB/qxdVpPnyRgiy/Xo6s4jplqLFM2mthifkttGssYYgDYPN59QPLBcRAW?= =?iso-8859-1?Q?ubkoct805ujPDIctGsMGZxlqajejYr+LVxk/ONCl07JkRU9JgJT5lmE6ug?= =?iso-8859-1?Q?F5/tYU6HOB3TYJrwLbIXH8MIbYHbDpAZbZ6X4LB89nSAsMSpa1qhSlqHhx?= =?iso-8859-1?Q?0DSbDw1TQwuZzVGJs5Bhs4P0/nAOZVUwU+nl6n/ligrpEdif6ubRUsVhjj?= =?iso-8859-1?Q?vRvEiC9kNQQ0tounNeTbmPrr4ld0uACT7UrUf2+zln5ZF5BF9KcoHwz8Ea?= =?iso-8859-1?Q?JMXFY95vc+xlQhxfp8T6AhqMqUNoX6xoOB26xypaA1fL3Kd+jn7jwDQsfn?= =?iso-8859-1?Q?kx0OsCAWlR6N5UCstRigD3KJYo/t0Ym/vy8x+nHPNeFC3IrLlrw3Amf7C3?= =?iso-8859-1?Q?+fnWh1mxBbdqDVHxO6UP/ehWEs206iUKOVIxy4YLsavskZ9+FK/jNzJxvv?= =?iso-8859-1?Q?ljmjLyMX2cPbnvKYSVQp1oIyk4yKrnr76//YXz9I824Qatc1FOJeaS0Hgx?= =?iso-8859-1?Q?yI8K3ru2dgdZUie4kKMZKMtIcVEShkle61xQua7dhgPmqeQMN0sPI/o1Tv?= =?iso-8859-1?Q?Zho2KdtfbLQ3EwTojTaHlxx0X+ibqzLEG7Vf0CrlMBAt2reqYtmGcSfLJc?= =?iso-8859-1?Q?doRY0W1jnt0oVawAqdG3rrQ+Ioi0j7plaOBuZkh47aVMvnbtHD6JUoH/dy?= =?iso-8859-1?Q?fKcnrc4SGYRO6l/dg5BVlEV6DRIOuQRf5h0Xb7jhO4VoHjwSW2uwUaS4Ii?= =?iso-8859-1?Q?R8JM/nMfEB9KoySee5I8ozT/hgi7O+bdnNABtzVd6lQqSlnJ6DTYDU62O+?= =?iso-8859-1?Q?Z7lOp8pe4VXIIOr/yomMPkkvBuYCoGbrZXWxvclZth12jDTGr5t7s=3D?= X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:DS0PR11MB7309.namprd11.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230040)(376014)(366016)(1800799024); DIR:OUT; SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?iso-8859-1?Q?tB/Hkbw3xjVCKQo8YNmFMqjO9pOOqrDyy3S8IUUzysBiwbePp7x2uFkXpG?= =?iso-8859-1?Q?TvbUWc0Ml80hcgnoqwjihyG7Zh4aov9cmu+q3lxor0oBGVO7uHbSk+LzVk?= =?iso-8859-1?Q?C+1w+X1LhUOk3x4b2T1fIDGEuhJmmq6GV6t07O8MZ0H4/nExYBlN7YEeN0?= =?iso-8859-1?Q?T2S7Od52K4BIthwhPziwnZPFXVFVI4ASxixwza70RDrjN7sO19hZR08PTr?= =?iso-8859-1?Q?mw9QxOlT57kcNOHdEt9Ip3oU8+AhXEDwLsR0pnlDNf7t6IZ1NOjdJTrVsr?= =?iso-8859-1?Q?daBts2uTflBQG9SLqmAtAlhKQtUe+1mmSm1S7IUxBllwNqsqc3Yfb5mCIP?= =?iso-8859-1?Q?8qf5ZwFxtqj527ZXCMncLOxn+zI5e91bbztAg9sOKH2dDFQYwwyrp7s0kJ?= =?iso-8859-1?Q?kfNwDYxIXFyl8pRjPAzGgQyxfObdOYSD6ObU8bgEn0MCuoSB7HD3edQTvb?= =?iso-8859-1?Q?9UmX5xDCn/4szAFgsazjdIPyMRZUlOZXd6oUROAcif+sGzaO5zzY6Eq7SK?= =?iso-8859-1?Q?sK56bhzkFldIciXsMaEfqwXsqDdgr5zQjRtwFsDfBujhtI8xnbbwwnW47p?= =?iso-8859-1?Q?d49/4OCsguCKOI+2wB5vEFO/JqShd4Eywz+PhiFLBYEtv3yN+whFTGFgjz?= =?iso-8859-1?Q?tdUbDR63KWFpcj4wlBu5J3b2EPHdqN3CqUHY1rFgqfdqqwcfb2BWXsp3dD?= =?iso-8859-1?Q?5TqioN6IPMVrVh28nrugmlIuSthjjO+5BWHtw45ZCfU1o5YhszTsmEqFbh?= =?iso-8859-1?Q?X+fu5/+ur6zxPJ7umvDnwe61H6vOnZJVsQRy6doeESUfhqoeAvf3beID6K?= =?iso-8859-1?Q?mwdWerZbx665QXsui1tw5oAqCUlFe7NZ0POz/SxQJqBVyPZALMQsJgo0qy?= =?iso-8859-1?Q?Y1sglZctvjbs4gJnmGnPo/X3UhWC5DwkXoJmWGD1oo/axUHMfETqDI0SGR?= =?iso-8859-1?Q?uD/DI9L7Qpz6ihcnsji0PZa6HKajHLhFl6ysXpulDw/XPEM+5jxQZGkNur?= =?iso-8859-1?Q?M/o3/nS6CwzTY84ix7dl+qO/EgOP2skG227TqsNIV21T97p4VBPJcuEDvo?= =?iso-8859-1?Q?ofuQtqXEl+vqGNY1MKtTVZNbtlLslm4VjQIDODN1d76PEyVQaWCG4VaBcl?= =?iso-8859-1?Q?LwFjFhW/gjN0xs3JdOZSUzoh/nI9KJojrHLtpoetmpNecupnuuNTlGw1I0?= =?iso-8859-1?Q?0WxdDKEE94YyHG9ulsN82OGxoO5/JARUlqihkwLutSbRkOvoK3AW5iDCRr?= =?iso-8859-1?Q?rtSl9+olLhDI294qzIsVRgkE/arBnQDo1Flh3LuMt/n2soIehJOKZ+WZL2?= =?iso-8859-1?Q?L9w27fgy2kifA9rePHg6HZPnmdKYnCB+qSefTqnWU6eZPabuFt+ZQ9js0L?= =?iso-8859-1?Q?Wz5zo5TCf+IWL9YjnAms+sMR1rxDi8cQ7QWmFVap3IS+MhWa87WWTIoTe7?= =?iso-8859-1?Q?O1vzWhZ2ssMphOFiHoyC4bX1RVt8LCytGUxN9ucrHl5C5gEZMK402AORpG?= =?iso-8859-1?Q?yz524rQfHyyuvSs5nhjOKykd2MKYyqBgbuyebgMQjDYEc6AG7CR2OCyvhu?= =?iso-8859-1?Q?7SVtOvvTBDNF+kZlrevCDUxqMB0K6E7aTC9hP8z7dU3NlkgZQpGakmfMfR?= =?iso-8859-1?Q?R3dG+fnVxTjiw5MqC7rNRjiqX9tN5mAnX1x/0SHqMii1h3SelzQNQS7g?= =?iso-8859-1?Q?=3D=3D?= X-MS-Exchange-CrossTenant-Network-Message-Id: 32810b7d-5c74-4979-5285-08dcce74e599 X-MS-Exchange-CrossTenant-AuthSource: DS0PR11MB7309.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 06 Sep 2024 13:07:44.4720 (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: bcyLUFI84z4MdihtJpUWL9eDT4jv/RjV11sx9a04YzdBXb2ev1lksVyVYol904v3KauxdeLyKb1B1AuOH1R6EhZP94qmc5SqP2TEbc1j1pI= X-MS-Exchange-Transport-CrossTenantHeadersStamped: IA1PR11MB8149 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 Fri, Sep 06, 2024 at 03:02:53PM +0200, Morten Brørup wrote: > > From: Burakov, Anatoly [mailto:anatoly.burakov@intel.com] > > Sent: Friday, 6 September 2024 14.46 > > > > On 9/6/2024 2:37 PM, Morten Brørup wrote: > > >> From: Anatoly Burakov [mailto:anatoly.burakov@intel.com] > > >> Sent: Friday, 6 September 2024 13.47 > > >> To: dev@dpdk.org > > >> Subject: [RFC PATCH v1 0/5] Adjust wording for NUMA vs. socket ID in DPDK > > >> > > >> While initially, DPDK has used the term "socket ID" to refer to physical > > >> package > > >> ID, the last time DPDK read "physical_package_id" for socket ID was ~9 > > years > > >> ago, so it's been a while since we've actually switched over to using the > > term > > >> "socket" to mean "NUMA node". > > >> > > >> This wasn't a problem before, as most systems had one NUMA node per > > physical > > >> socket. However, in the last few years, more and more systems have multiple > > >> NUMA > > >> nodes per physical CPU socket. Since DPDK used NUMA nodes already, the > > >> transition was pretty seamless, however now we're faced with a situation > > when > > >> most of our documentation still uses outdated terms, and our API is ripe > > with > > >> references to "sockets" when in actuality we mean "NUMA nodes". This could > > be > > >> a > > >> source of confusion. > > >> > > >> While completely renaming all of our API's would be a huge effort, will > > take a > > >> long time and arguably wouldn't even be worth the API breakages (given that > > >> this > > >> mismatch between terminology and reality is implicitly understood by most > > >> people > > >> working on DPDK, and so this isn't so much of a problem in practice), we > > can > > >> do > > >> some tweaks around the edges and at least document this unfortunate > > reality. > > >> > > >> This patchset suggests the following changes: > > >> > > >> - Update rte_socket/rte_lcore documentation to refer to NUMA nodes rather > > than > > >> sockets - Rename internal structures' fields to better reflect this > > intention > > >> - > > >> Rename --socket-mem/--socket-limit flags to refer to NUMA rather than > > sockets > > >> - > > >> Add internal API to get physical package ID [1] > > >> > > >> The documentation is updated to refer to new EAL flags, but is otherwise > > left > > >> untouched, and instead the entry in "glossary" is amended to indicate that > > >> when > > >> DPDK documentation refers to "sockets", it actually means "NUMA ID's". As > > next > > >> steps, we could rename all API parameters to refer to NUMA ID rather than > > >> socket > > >> ID - this would not break neither API nor ABI, and instead would be a > > >> documentation change in practice. > > >> > > >> [1] This could be used to group lcores by physical package, see e.g. > > >> discussion > > >> under this patch: > > >> https://patches.dpdk.org/project/dpdk/cover/20240827151014.201-1- > > >> vipin.varghese@amd.com/ > > > > > > Thank you for cleaning this up, Anatoly. > > > > > > I would prefer to take one more step and also rename functions and > > parameters, e.g. rte_socket_id() -> rte_numa_id(). > > > > > > For backwards compatibility, macros/functions with the old names can be > > added. > > > > > > > I don't think we can do such changes without deprecation notices, but > > it's a good candidate for next release. > > Perhaps we can keep ABI compatibility by adding wrapper functions with the old names/parameters, which simply call the same functions with the new names/parameters. > > The Devil is in the details, and I haven't looked deeply into this. So take with a grain of salt. > > > > > I have thought about including parameter renames in this patchset, but > > for now I decided against doing so. I can certainly include this in the > > next revision if that's something community is willing to accept. > > I agree with your decision on this. Renaming the parameters without renaming the functions could be confusing. > I actually wonder if that is true. If we are simply renaming the parameters without: a) changing their types b) changing the function behaviour then it is neither an API nor an ABI break. If we were to do so, it would be like changing a comment, since the actual parameter name is purely a convenience to hint to the user what the value being passed actually does. That only applies for function parameters though. For any defines or macros that need renaming, then we are into API break territory and we would want backward compatible versions of same. /Bruce