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 C68C7A0545; Thu, 15 Dec 2022 14:58:14 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id E8BF340684; Thu, 15 Dec 2022 14:58:13 +0100 (CET) Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by mails.dpdk.org (Postfix) with ESMTP id DF37140223 for ; Thu, 15 Dec 2022 14:58:11 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1671112692; x=1702648692; h=date:from:to:cc:subject:message-id:references: content-transfer-encoding:in-reply-to:mime-version; bh=lwwQZMhzNai1PtLrdMNAlZL0agg+AJqdTGKqNcHTiqg=; b=KzjhR3U59LfrzcQTSTh7R6nwX/U8LRpMzQT//VbI4XFsjcl/vobu8zdR Ky+soggvfgdrQ3zjQPCJMVnOPpZiaj/ut4RY/qhHKX6PgftbvShRPoodI 1We6ehIqcUxu5CLsOuZiEtk7nm8XaDvDmHLLPMRd9h0sER0qDIYcHYtVX mVoVhSbFP+BiDRApLjeC90yYzgeEDgBWiXVOlgpvPwkxP9WQuBGnsq102 worgKmh4E/2uEzAgXtXUnQZRU+pGV1M/IRzggUuTdN7o7LOEAxfv61kWs 5eU697dpN4wrFQ7AZ65BSGAcKI0mHPg6M3xjkjchZseJ7lHxECUZexU9k A==; X-IronPort-AV: E=McAfee;i="6500,9779,10561"; a="345766847" X-IronPort-AV: E=Sophos;i="5.96,247,1665471600"; d="scan'208";a="345766847" Received: from fmsmga002.fm.intel.com ([10.253.24.26]) by fmsmga101.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 15 Dec 2022 05:58:10 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6500,9779,10561"; a="756345457" X-IronPort-AV: E=Sophos;i="5.96,247,1665471600"; d="scan'208";a="756345457" Received: from orsmsx603.amr.corp.intel.com ([10.22.229.16]) by fmsmga002.fm.intel.com with ESMTP; 15 Dec 2022 05:58:10 -0800 Received: from orsmsx610.amr.corp.intel.com (10.22.229.23) by ORSMSX603.amr.corp.intel.com (10.22.229.16) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.16; Thu, 15 Dec 2022 05:58:09 -0800 Received: from orsmsx610.amr.corp.intel.com (10.22.229.23) by ORSMSX610.amr.corp.intel.com (10.22.229.23) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.16; Thu, 15 Dec 2022 05:58:09 -0800 Received: from ORSEDG601.ED.cps.intel.com (10.7.248.6) by orsmsx610.amr.corp.intel.com (10.22.229.23) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.16 via Frontend Transport; Thu, 15 Dec 2022 05:58:09 -0800 Received: from NAM10-BN7-obe.outbound.protection.outlook.com (104.47.70.104) by edgegateway.intel.com (134.134.137.102) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2507.16; Thu, 15 Dec 2022 05:58:09 -0800 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=mkgsbHf5P9f5c1djfWXkr7UFEhC5CSUVAdYhO5dF3cIXTcsqHj0yzeLplg4qIL8sCEIKI1nYgDHLD5fTFoe8ZMaaczbLQiPXCWb1K/DL+en01Do/Pn+s+8vGNqUqLQctSTwlsdHB4ZSvtl6In5Zi5McldvakhpiUpgLgy2vtb516+BZGkwLLFkUPfY8ub1SfhSvyU/Jxmx8sT8Vx0zP2X0xcaZBdFZpfMM6QbKoVtNQZSHC6pAw8rfMFqqcLzpl27P4BD0j0FdrQhmFzeemYgSkLpSo2fRlhiuxf4MwfhR3+17r7HWVUB395ndE8lSX74Cha8t7CSIxUJFNYVbSOOw== 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=YEviCqQh4Yix4Z7gSXWKKDptYkSwlrDcDWBFrkNLyqI=; b=VgYT6MHY9syZ7tTB5vzBJx5b5Rk3aU5BhzHcJoyzSu48QBHuIMEJHFbaOJU0EsiQVJacTlGLrocfYaZG0mlmvhsGy3tPdW9ra6MpAPJcPQVXEU7mEBp+K2bbgGrStXvBBUtKGoCrjO9D2yZYO2vwB38IxCna0xklm3G8Hdv6ooIlNLIvmoGQZlYCAAdpKIQUJmAdto+PcLwseX9ZqhZpOLSXjym/peUw2VkE6OiE4Of1STOM47sSIgW4D0Y1+rMM/dpGpbTc2ozaKzLKDADrgIVIvuoz1DxAuRdy2aHiWx8mp4Z6LTWaZkrC4JYy/wF246kBmtgx5jUlCrgbZQ6ATw== 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 MWHPR11MB1629.namprd11.prod.outlook.com (2603:10b6:301:d::21) by CY8PR11MB7827.namprd11.prod.outlook.com (2603:10b6:930:77::9) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5924.11; Thu, 15 Dec 2022 13:58:07 +0000 Received: from MWHPR11MB1629.namprd11.prod.outlook.com ([fe80::18bd:edae:ad31:a228]) by MWHPR11MB1629.namprd11.prod.outlook.com ([fe80::18bd:edae:ad31:a228%3]) with mapi id 15.20.5924.011; Thu, 15 Dec 2022 13:58:07 +0000 Date: Thu, 15 Dec 2022 13:58:01 +0000 From: Bruce Richardson To: Thomas Monjalon CC: Tyler Retzlaff , , Morten =?iso-8859-1?Q?Br=F8rup?= , Subject: Re: [RFC PATCH 2/7] telemetry: add uint type as alias for u64 Message-ID: References: <20221213182730.97065-1-bruce.richardson@intel.com> <20221214173845.GC31935@linuxonhyperv3.guj3yctzbm1etfxqx2vob5hsef.xx.internal.cloudapp.net> <2122310.KiezcSG77Q@thomas> Content-Type: text/plain; charset="iso-8859-1" Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <2122310.KiezcSG77Q@thomas> X-ClientProxiedBy: LO4P123CA0109.GBRP123.PROD.OUTLOOK.COM (2603:10a6:600:192::6) To MWHPR11MB1629.namprd11.prod.outlook.com (2603:10b6:301:d::21) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: MWHPR11MB1629:EE_|CY8PR11MB7827:EE_ X-MS-Office365-Filtering-Correlation-Id: 4c55ee20-8291-4737-6f9e-08dadea464a6 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: 9XAEMt69dR1MbdUCObBUMwSp59KRjlGJHPcG8oItq2z8aQX0x1BS8KivY7MUfcIac8jMU1tcoGngxpqNH+mG4M9rUyvYayrNfEzjw6aP4dr7b1tVAYnW5Ov+GmG9rI1LB8pFgjnHgEXpIt2xVgk+0tMy/J1yJHVaw0a2ViJQATfHoUo/BugwIHh3DmUs+TotyjH7Oo50fseYt2Bm3RHLQwH5l73ECacVTamjVDFAhpmfHQpzu8vMrGpbLC72QmZlEdgtW13vo22D741ATqDxRfoA0+BbGIZ/UBX4hGS9yt2QMH+ijEcnvJSOguQJTxrc6mrEJK6W5cjSOCRBDac+i0Wla724v+e/hcRzI5Nivbof8HwZmV6CxwGG1Xs/HB8rPtIPq+UVrhz9EUnj5K8cb7+Zr8328aBOrg8dnZOssxAIiI+1WB5C/GOPYgSQE/hsmjLAXL+NUZifdxNn6rnQBAGRHRgcASiMS6H2tqv5UD42E1saLOeIqu4+lBs4y1LRtZ+nELm3fpOTj+Gs23N+ne6hc0w71knZ5nV7STWYH2eBgy/9h2wivIWWld4XLTmMVs+zeoEUboOyoJfze2O4ggwykZUE5X4q4anYQ7BmNSNdzuiVIBim9COYjhRSJxK9CQrn2mT4mkZeRverHkSnsA== X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:MWHPR11MB1629.namprd11.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230022)(346002)(366004)(396003)(376002)(136003)(39860400002)(451199015)(82960400001)(38100700002)(6506007)(66899015)(316002)(6666004)(6486002)(86362001)(478600001)(186003)(6512007)(26005)(66946007)(54906003)(6916009)(66476007)(2906002)(4326008)(66556008)(66574015)(41300700001)(8676002)(83380400001)(44832011)(8936002)(5660300002); DIR:OUT; SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?iso-8859-1?Q?quL1JAWv54wC+n9/wZb7aaMGPgB7dI/m+Uwakaqb4wdz6XiBnRIp7wUwnZ?= =?iso-8859-1?Q?rTWFIksf0++VIKVoB9iBcBGoEIDyw7w3hMJVRiplTd3QkpwFB+8SUJLZ5A?= =?iso-8859-1?Q?tue6n9GLl8ehWstz9b23NHEsiWOYruDFwltUrjnhPuRhEakAxhuAwhqPBI?= =?iso-8859-1?Q?/qAUXMyq1QghWPDOK7JI+Sb7UN9TY83YHo1v2L0yma3e4an5tYvGEhQhGq?= =?iso-8859-1?Q?MuvMNNMd0KlWxd2QFvCfirv8Jvgljr6zp1ZQDiqMd268kZb33XnQz5/1lL?= =?iso-8859-1?Q?ti42blxYvWGnuoFDdB1KNHTMpspNJWCiEGRYkilYHTX3fMbbLcJxo/KBfX?= =?iso-8859-1?Q?opg412e21vs4xMUec7+ZWMPagV5P58jxvZIt1xO90DEQwi+rypR642yZ/x?= =?iso-8859-1?Q?TPbw6RiWkzl70PlNwAg7PvJnI5/D6P6akDkpZnoihTSLwFsqoVBxZPLSH9?= =?iso-8859-1?Q?PR/fOAtvU3qjmfK/LD7c9m3eC4OCfiCPJ0qtrHyxWsMr0O1eQ2XxKmrvAG?= =?iso-8859-1?Q?rouzJR/faNELWLIRJHzpY2ZonH2NvIKeG6FgPp+NOaQ3InYj5nCj5SYhs/?= =?iso-8859-1?Q?EQg0/KyLRD5tJWJyPQqylWBto+1Bz/I5QvsHDV38l0Zy4EqbIwS2fmVhgw?= =?iso-8859-1?Q?RkVJaenU3FHW5pKdV9U3n3a5PuylupI8WTOIs2MSeIJkGOAComb15ju3Co?= =?iso-8859-1?Q?MC6vcV3TuUPvE5RDqJdirRuPtQKt1O0jNCjWCL9SfHmh0hQUd50NcS77JC?= =?iso-8859-1?Q?FOpFc523DIgmKlYd36B96MepjT+jrGXSqq+eWtjQAMiokjSZDuVCxwEFrK?= =?iso-8859-1?Q?l0AhYsaov53ieI4GQ8PQnH/CYy7FX8vI68HHlBS/wQTB1TKrRCci8+Wpmm?= =?iso-8859-1?Q?JpbClM7klTCOwStvW21TLubzmDiv0Go+wvlIYFio95kQ2g+X2cEaw/l6X3?= =?iso-8859-1?Q?7a48embDR0TgSHku34NDrIXnLJhBQVBqo1RpSJEO4tabFrlIerdGdlZ5yS?= =?iso-8859-1?Q?OnWJsgwHO/6zbFLNLvEmDVJWjz4uxecAfWHECWQEhcxx9kA94sNRo6hLLr?= =?iso-8859-1?Q?i08gKGT7paLiYE6O0+otR9oY9i8XfVx7QM8OJI+OjaIb+WEePpMMc3tLEu?= =?iso-8859-1?Q?LDgrBj/ietr5PBOQFJ/6B2uV3+89eKuv8ytruCJ42dFWL4KSxEbK6YCmC6?= =?iso-8859-1?Q?S7tF6F7NYPKTvTsa1HzEtRppLmdClq0tW30R3fcqx18WZI6+v2QjnxRgOO?= =?iso-8859-1?Q?JSJgGfcU9GYqzr9B5zD3nodHaKnoQD8jHTrJIfzijbMbhScZrzTwcJiyBh?= =?iso-8859-1?Q?wlOTKrjT3CfdhqWXVdwBxWeAsg3MUYdW83VH8V7nnJMgE8SC9N1O+AfFHx?= =?iso-8859-1?Q?VN+CRGbUY2ep/WU8woR6NBANTROB9gMZ/lmtWe/USRFUbiwgjhPC+KHjeB?= =?iso-8859-1?Q?tfDgCNeKhUXXcnaHhSmQQ2zvQ1TMkNq7IXpeSs77pvAznCtjdCcyHBoXVG?= =?iso-8859-1?Q?H8SoDtODV95q6tw12vndykI4zkvjfQrcgwW4J/wgJjpNu3JVrQG55GOw8F?= =?iso-8859-1?Q?XPJf8QNYaHQ/P5KkrNzTh+s4TnmGytcy9arVwwQBLHsQeU6efq1U+oj7LQ?= =?iso-8859-1?Q?qCKKglujLbyeObvkOsDcnDdUNvPnYTLc+IBrJNIEcqn/pQ9uDrBiCH4w?= =?iso-8859-1?Q?=3D=3D?= X-MS-Exchange-CrossTenant-Network-Message-Id: 4c55ee20-8291-4737-6f9e-08dadea464a6 X-MS-Exchange-CrossTenant-AuthSource: MWHPR11MB1629.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 15 Dec 2022 13:58:07.3790 (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: uAa1yYgJmKgMtRJu2I7WbSN13zOQh4Mus5F2KcH07bqsTvsdd+3o9E0X8bb0A/WSx8468jpWaBWVEj8zf/5DP3pGF7SeDXZcDhrYMjgN/0Q= X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY8PR11MB7827 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 Thu, Dec 15, 2022 at 02:36:51PM +0100, Thomas Monjalon wrote: > 15/12/2022 10:44, Bruce Richardson: > > On Wed, Dec 14, 2022 at 09:38:45AM -0800, Tyler Retzlaff wrote: > > > On Tue, Dec 13, 2022 at 06:27:25PM +0000, Bruce Richardson wrote: > > > > For future standardization on the "uint" name for unsigned values rather > > > > than the existing "u64" one, we can for now: > > > > * rename all internal values to use uint rather than u64 > > > > * add new function names to alias the existing u64 ones > > > > > > > > Suggested-by: Morten Brørup > > > > Signed-off-by: Bruce Richardson > > > > --- > > > > lib/telemetry/rte_telemetry.h | 36 ++++++++++++++++++++++++++++++++++ > > > > lib/telemetry/telemetry.c | 16 +++++++-------- > > > > lib/telemetry/telemetry_data.c | 28 ++++++++++++++++++-------- > > > > lib/telemetry/telemetry_data.h | 4 ++-- > > > > lib/telemetry/version.map | 7 +++++++ > > > > 5 files changed, 73 insertions(+), 18 deletions(-) > > > > > > > > diff --git a/lib/telemetry/rte_telemetry.h b/lib/telemetry/rte_telemetry.h > > > > index c2ad65effe..60877dae0a 100644 > > > > --- a/lib/telemetry/rte_telemetry.h > > > > +++ b/lib/telemetry/rte_telemetry.h > > > > @@ -8,6 +8,8 @@ > > > > #ifndef _RTE_TELEMETRY_H_ > > > > #define _RTE_TELEMETRY_H_ > > > > > > > > +#include > > > > + > > > > #ifdef __cplusplus > > > > extern "C" { > > > > #endif > > > > @@ -121,6 +123,22 @@ int > > > > rte_tel_data_add_array_int(struct rte_tel_data *d, int x); > > > > > > > > /** > > > > > > when adding __rte_experimental api i have been asked to add the > > > following boilerplate documentation block. i'm not pushing it, just > > > recalling it is what i get asked for, so in case it's something we do? > > > see lib/eal/include/rte_thread.h as an example > > > > > > > > > ``` > > > * @warning > > > * @b EXPERIMENTAL: this API may change without prior notice. > > > ``` > > > > > > > Ok, thanks for the notice. > > > > Actually, related to this, since we are adding these functions as aliases > > for existing stable functions, I would like to see these being added not as > > experimental. The reason for that, is that while they are experimental, we > > cannot feasibly mark the old function names as deprecated. :-( > > > > Adding Thomas and David on CC for their thoughts. > > Is it related to telemetry? > > In general, yes we cannot deprecate something if there is no stable replacement. > The recommended step is to introduce a new experimental API > and deprecate the old one when the new API is stable. > Yes, understood. What we are really trying to do here is to rename an API, by process of adding the new API and then marking the old one as deprecated. The small issue is that adding the new one it is by default experimental, meaning we need to wait for deprecating old one. Ideally, as soon as the new API is added, we would like to point people to use that, but can't really do so while it is experimental. --- By way of explicit detail, Morten pointed out the inconsistency in the telemetry APIs and types: * we have add_*_int, which takes a 32-bit signed value * we have add_*_u64 which takes 64-bit unsigned (as name suggests). The ideal end-state is to always use 64-bit values (since there is no space saving from 32-bit as a union is used), and just name everything as "int" or "uint" for signed/unsigned. The two big steps here are: * expanding type of the "int" functions to take 64-bit parameters - this is ABI change but not API one, since existing code will happily promote values on compile. Therefore, we just use ABI versioning to have a 32-bit version for older linked binaries. * the rename of the rte_tel_data_add_array_u64 and rte_tel_data_add_dict_u64 to *_uint variants. Though keeping compatibility is easier, as we can just add new functions, the overall process is slower since the new functions technically should be added as experimental - hence the email. For the case of function renaming, do we still need to have the "renamed" versions as experimental initially? Given where we are in the overall DPDK releases cycle, it's not a major issue either way, I just would like some clarity. My main concern with having it spread over a couple of releases, is that it's more likely a step will be missed/forgotten somewhere along the way! /Bruce