From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from NAM01-BN3-obe.outbound.protection.outlook.com (mail-bn3nam01on0051.outbound.protection.outlook.com [104.47.33.51]) by dpdk.org (Postfix) with ESMTP id 8000C2B83 for ; Tue, 17 Jan 2017 13:34:39 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=CAVIUMNETWORKS.onmicrosoft.com; s=selector1-cavium-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=iwM1bW4OEwdLG7p2mN8zRZap33Z4H9OZ8tSEIwijWxE=; b=iVLIWtYGCzxu9YvCrraFBduGDPip5+GZZJXdtsJFPIZpDStrZVufHsPiPQOe07pR7QJknEaOiav5oVsjXbxg6b8pkE8HalXolri0vYJ81KDmLKL6iNjGHL+bsNm+GXcauG3tSPcsmOtPs5VVkkwi6zlRxIyiI+5nAS3X7ZLD4KY= Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=Jerin.Jacob@cavium.com; Received: from localhost.localdomain (122.167.185.94) by BLUPR0701MB1713.namprd07.prod.outlook.com (10.163.85.14) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P384) id 15.1.845.12; Tue, 17 Jan 2017 12:34:34 +0000 Date: Tue, 17 Jan 2017 18:04:19 +0530 From: Jerin Jacob To: "Mcnamara, John" CC: "Horton, Remy" , "dev@dpdk.org" , "Pattan, Reshma" , Thomas Monjalon , "olivier.matz@6wind.com" Message-ID: <20170117123418.GA2611@localhost.localdomain> References: <1484583573-30163-1-git-send-email-remy.horton@intel.com> <1484583573-30163-6-git-send-email-remy.horton@intel.com> <20170117042935.GA32676@localhost.localdomain> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.7.1 (2016-10-04) X-Originating-IP: [122.167.185.94] X-ClientProxiedBy: MAXPR01CA0028.INDPRD01.PROD.OUTLOOK.COM (10.164.147.35) To BLUPR0701MB1713.namprd07.prod.outlook.com (10.163.85.14) X-MS-Office365-Filtering-Correlation-Id: c88eaefe-9f70-42b0-5945-08d43ed53300 X-Microsoft-Antispam: UriScan:; BCL:0; PCL:0; RULEID:(22001); SRVR:BLUPR0701MB1713; X-Microsoft-Exchange-Diagnostics: 1; BLUPR0701MB1713; 3:rPUymKus9eQ7p37WPYL/n3H6fodhOv/4NIi0RApnrBA2L0By44HmlhNDMykbynsFM6lAikHCae3pcpQOtU4qyla6aUVAW5ueUGa6T/dTVHmz7KLEqIJUK4Yi4eDQN8KzAB30TT5aQiTAPvPt2dJ/lt2CJX6kdC62xBG1Vgg4ZYoTqcvcJPUROZnft78Sf89wJGozK/RDBFXCjyUIjtuzqf0Ts58MMr1COG341OnJtQldjyZYjhJN3h1iQGczfvgvRGRlhcqoaLKU1zbLngaktw==; 25:oFSxnBCKza1p7W5BzEfZYitIH366rG+vBXriaMiGoAis3b+YpBL/9rApXVQXBZTndu9qsV2eWPWWPJ88l3uyHydHe6vmdxkpZAcIG1rcKkdmYvMl4eGLLCgneRxwxaR2QF3sfbvQCXlMzg0wC8rE9rT7Fq7aGN4Xb71GPKOBBoRE4segQ618wr4ygraZKDSV20GYoj2w0nSAGI3DERgZoLHYMDb94ExzCB98sVPqGJHnXKztyOXZLSNpZfJFRAwwQVPOt2yV+bDBVr4CkQJH6YnGPyHIOpknBSJg49qUZKElPKszuuEw5lt6dHOop30Uj4l+MGGKdlxlZnRr1tFyBOQj66wKO8owG2loEskYLcRXYJsKroKLRlJsAji1yV2AVlsuQCYJkNoZ09QAP/AqWLUJSRQiVTaCUD6P5MQM0xlMD1hxFQD1RSLMRxcUnARBKJHMKCPr2MrKauXee0Be0A== X-Microsoft-Exchange-Diagnostics: 1; BLUPR0701MB1713; 31:xndMuJGtvn5pO//3wgqRe4QiHkZu3B6e0BQZ/27jZG5IIGDZ8yMuzO6SE307jrIoTiUFCUKh6VSjAOY7y3e9Y9yLfH8B66OomDknUlUSskIrPar4mzLxTtEI0UwOfTp4unElQHqU9WAFuNKou0WjAg1J/3LrmBnkXuiLjIMhODeBx3CEy68JPW+8v2B7TRH8nLMrCCwNeHZggmzNcCZpZ4iJ4tn4crD8Pf3lOBO8xiIOL2JXqvccZ+gtiNlFqEmQuu5f2/UyN9le2yLg4vxbMA==; 20:nNxBgfG9Ko0IEBVbvjg4q8WJDQM1j//1/5CYsYqolJNCw2ECLrRhXxNpNR7Z9nx30+5ydTLzJyEihJ7JgnDBg3MlT43Y5RqEVAFnlWTu5FvFWB9rQ4KQEqQ5DaW03LT3w/9IcVTwja2/enzEencY/Ksor/zj4TLyUNmgqSorZC9/gTU3yM8YXSdmyl6fq5ahuQOb6cAtz2TpccI62l6xEi03hqz2t0Mx4CT4F/yy+BdOQRe8C1ai3l/ZWxMWpdzMKbmlqJLdy+ajO89admy5ChJ6GpKyRnJuYPjYWJ9y8QvBFGnk9wW+g9GJWjErjzH1scr+6pKBSeDfG3qPcS800721TGu0rkWTLDIzMtPX/tYxwgp+A+mq7uJnEc35QP0yvXn3RuWTCg1twtSWE4UiUhhtYC0lGuv61cmBF2Aflt8aaIQiBJNr42Z1id61gDUjxFP1W/EjyMp/B++TTcRUSKCp39+3U6bz8895uYzQh3tzopRwd3VEKCXRlW9keM8ACSePbI3L7H6zQN4M2gnl5NOK+F6obS5cVNFzg/vMcYhmi3SPbwNtfDnlzBTysd5pmtBL63xALUx5yKD5mHSZ7ayRJ53b0uH+hsufVSAApGg= X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:(100405760836317)(228905959029699); X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(6040375)(601004)(2401047)(5005006)(8121501046)(3002001)(10201501046)(6041248)(20161123560025)(20161123558021)(20161123555025)(20161123562025)(20161123564025)(6072148); SRVR:BLUPR0701MB1713; BCL:0; PCL:0; RULEID:; SRVR:BLUPR0701MB1713; X-Microsoft-Exchange-Diagnostics: 1; BLUPR0701MB1713; 4:Bnu8TjI3HT1nsSi+9gpQDiVqJc6ELwH+7nuvtjtbyFSxnhIEqnHyDQ6xijO1LKl4J60//HY9B67ZDmzjrEU0vKSyHsx30/kau4knqMArYoQzWqbYT8qJwfMvq/4UYI+JGE6YG5dRe0iR3f8qJIHXKeuBwcnJ/ZACr7kn24hyKhjLRc1W47zdqdPP61LFj9ba1GhZKxaei6PCHEWPyGiPnPCzCxZnO5Fi51oqNkzrGa0NxtP6Ht9zB73G02BXCOZE1Bn6tAJgQaFi8Bv8o7OoDifR7gVtWzndWuN+f4EUG4XDArmvCfY9s/5Ffojzgp7AvxTiaI2DbS94m2H8363WWBwXCsyTKVhk3yZ6xSkk1uTam0XAiKvVVnwxUQTlt5MQsDqVRyYmcq0CUQK1PHn9tkUqWUt3tKOdyebNveVo7IgzAD2xYOrcA/RBFjcBMmqT9YTREjENnsxebBV9q9Ab0zm7hJ4+KakMrFZrDzSRSfyIswTlG0hbgJZPO3V+m3aZpiNFKgxHXyALMWx/FNJPXxuLX6pf6V5KtAJ0wNNLar5ioSyDkc+J8aBdE/Pd7DrJVUe5gp4JPpMtPbjItPz2N7cJQEHrOj/bVKtuMqom7GqO2RvR8qRRM7+87DvCeQicb5Dj+tjbaQKqpifUTvaXrUiPvZvCqIyL7QivMzpXlhcrbuBtKrTVSVx2glyIAG+w X-Forefront-PRVS: 01901B3451 X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(4630300001)(6069001)(6009001)(7916002)(39450400003)(24454002)(199003)(377454003)(189002)(13464003)(305945005)(7736002)(8676002)(110136003)(81156014)(42186005)(97736004)(92566002)(83506001)(81166006)(6116002)(6666003)(50466002)(66066001)(30001)(4001350100001)(46406003)(5660300001)(6916009)(33656002)(2950100002)(189998001)(42882006)(97756001)(105586002)(54906002)(4326007)(6506006)(55016002)(50986999)(61506002)(3846002)(93886004)(25786008)(38730400001)(2906002)(47776003)(23726003)(9686003)(1076002)(76176999)(6306002)(68736007)(101416001)(106356001)(15395725005)(54356999)(229853002)(7099028)(18370500001); DIR:OUT; SFP:1101; SCL:1; SRVR:BLUPR0701MB1713; H:localhost.localdomain; FPR:; SPF:None; PTR:InfoNoRecords; A:1; MX:1; LANG:en; Received-SPF: None (protection.outlook.com: cavium.com does not designate permitted sender hosts) X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1; BLUPR0701MB1713; 23:x4K77DP1PI7QNChwlO9tDnQBcfy4Dxq0xFpk7oj?= =?us-ascii?Q?//lH8rIJ8kmMJoxFfn2R9wVpu6MPs6xNLTDrM54lzyU+uePmilNZ6dFKHNFc?= =?us-ascii?Q?Ve8vPwcFVp4JBDdjKXEfmkj1eGPIPz/kExV4Uzrt6UWu+2V5njf4ynFeN/OC?= =?us-ascii?Q?4H4O5dcGil59BvmP3Egbeo/Gs86beSNQQahFhDXW3nmXHgkkvYObVPapNY7K?= =?us-ascii?Q?6yBN3xR6g/2psV9otVXh8EqALfp4yIB6MwvMRC5RaEPIyYdASY745vPCATaF?= =?us-ascii?Q?15F4WuIh8obh8l1r3fkrwWIhIu0ODbR253wKJMLsOdDB0vDVYzLGL91Cp2a6?= =?us-ascii?Q?GOky2nFcT6NDPwdnbQD4dLym/NLSuPCKN4fdxD8DI9RRfMzxiCPiAfmfyUgB?= =?us-ascii?Q?O/Xd8QjEzXx5Cg+H16WTNOqWaJOvnVyBacmmmm3uQOUNUG5rQzo2kVEd/Roh?= =?us-ascii?Q?bA+YYIsJhQ9Ql6/y1IIy2lKOLjSNA3Bh6PFmbO0dev1Bc5A9FgU3KfS+n3H/?= =?us-ascii?Q?3OgXs402QPcrW9sV1iaEpeQh75jl7psMgbeyTWSoiKdLJWu1cwCUtZ04pgxJ?= =?us-ascii?Q?XcLlnV2bPcsanmsJ0DoaZ6Q7kuDVjXmLejsygNTRE962UUV/uZKtyiXRs3bf?= =?us-ascii?Q?3DP6ZLpXDcyUutYigoUf/EC7Hikim4Y5vsca7rYih+He0ETxW9WFFYDwnjaO?= =?us-ascii?Q?sUVKvRm+XXFiTWBw0aHkocp938rJx8+Vump9W+LzMeoNBWUBWLX6SjRNXPgy?= =?us-ascii?Q?CXpNxE+zTyzBykcY37zI777Xt969rfBB6A9VYhpoLzU+VkHCDgG2O4SIe79z?= =?us-ascii?Q?GGvOdWRvD/cBxsAwQU/z9uqEnAFESbROCmEvXTLcuJ4GJwXjYbAKp5jqzXx/?= =?us-ascii?Q?03ULQEg9FYIKRvXWw8quZ8pyCexE04w8GnRazqxyqipLSHJDDkydJC0BMzXk?= =?us-ascii?Q?uffgT1THmV1y6DH+Hzi+NY62sijkAXYYwd2O9fEmork2crV5ziR4Mf1d6Fgl?= =?us-ascii?Q?Nj2q2PPAF1+wu1SBZyNN9eATqish44jIdFy8KTXBoU2+q0zDWj3sq9+miIRm?= =?us-ascii?Q?eV9GnWFJj5ehPGVBXrqYkwsMW3BXYZeJE2JlapJAhd6s0RfjruG8PUAaVhYC?= =?us-ascii?Q?VxGEMwcNjC8cyx8sGp9qsH6cGVsqURP3RrnxC0kBGHbXpZPlA2MlTwEWDP3/?= =?us-ascii?Q?/b2h3ffutcD/EyvCcjpUfvxryuFpN3E1aWBEd4+KALoeKnT+KuDLNYBkUTHl?= =?us-ascii?Q?xf2287kyPZaNK1n9qxk7ew484sSerwGQZ5v/pacvNatCNRegazSY8WJ+ujY4?= =?us-ascii?Q?mMQRhM4Mcfm+1BbNPwGetYUkhjGsbE5zbmzq0tYmaRWVO0CYUaiJRavei2AT?= =?us-ascii?Q?0MrZz9EWEHp+JvD+QqHnT6RT3YHfUqRxZdC1WLGosbI4Rc4EzEBykrJT2P1S?= =?us-ascii?Q?N5sNVLL4dgw=3D=3D?= X-Microsoft-Exchange-Diagnostics: 1; BLUPR0701MB1713; 6:l2Xzr1VAERRo8LwILAy5diSAVG3CDFQ4bSebFERcqT6ZGAndbSvEQTKe0pX/l6jBxug4vxL22YHClmHMcAFDFcNncS+2gjoua5G++DJamV//uK9QpJHUbFt2lNFoltAiMh701UAtOYhZ5FxvI8HFPs5xA8zfar037CGp1MSLTKBk/e9e/LM4qOOLlOTFspaXCZSLDttnHUv+yI52rYzzKMCB77uYECUypO8u8tWEkVgdEMDWH8olOP0JDl0I/jOUZvvonfMEHzmKDmL8o1Av2uYddUXSv/46rXt5EtEjqFx1N3UfA3FzBg2N8rlIIAxOhPKFMGNAKk1Q6Go2A8K6UyhbD9W/HBkqVFerqQq9wya2sDCb17Jels/+ba3cQDpqfcDj1mUAnrPpA/67t2qPHm+ahcyWlGFXL6urVMo5SIg=; 5:sNzG8wNn3IqqZb/iT1dsEHzNKVhU9BmKqxfWKuu+ppI6A4JhjpQnWSUHWjEUNjPlMU67Yclho6A1v5L32/GIvZ2pI5DPrkYzwavImQX4aAYY2KJVAD6hJKbX2IK7R1F8xmh+swNo4fT3f+P+ummlaKL4Tvj9suQTD889ayXdF2I=; 24:7OeVRQAfTce5JUlqgXHoqpkveI+iLNl0zC/12gjfPveOgT4MTKQ6e2hfx4VlGUwSN1LyXTjvJmBifJ6yYWxqvK6KVueJiDSo6P5egq4D1UI= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-Microsoft-Exchange-Diagnostics: 1; BLUPR0701MB1713; 7:hc2K8r4kYfJUwSAKEzopHxyn7SRFV/42SkNS9aU+aAyNMiHsxXqolr7hrORMJs+aTT/OkHyPFmQn4z70Nyq4+/E3qX17N5QW9BU5nWkLlLv6Lb0J4dO/jNDbGGkjVLPpiIW+/0n+1btUB4nnlkG8aWZA6QptRESw88qEqVaoKZ3Gw7Ia30MOqgtLwUb1mwU6pT0aGS2zckfFLzOG7Hn7BIGmkvLHVGg+4dQGDVtBV/6dbxT2SnivxA2fdbtldxcw/p04pZZ/U2BSYW1QcWs+tYRxunCM/YRJt/DhRyRNknRVxsm4zVcO+0YwHDmMaoZtvCvfgAM3OkoonEKU1SwZoxA8n4vZ6XpQeO44fqomr+rKTawsf9hghwx7XXG1dBm74JkoSBp8NcyBzhtxCF1dictfNlevxr1uT8ZpSj7+uCf5fI0MiaKhX2Ce/6aRKJEG0eQ21fYtNOlNXSJXHBsBqg== X-OriginatorOrg: caviumnetworks.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 17 Jan 2017 12:34:34.5516 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: BLUPR0701MB1713 Subject: Re: [dpdk-dev] [PATCH v7 5/6] lib: added new library for latency stats X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 17 Jan 2017 12:34:39 -0000 On Tue, Jan 17, 2017 at 11:19:24AM +0000, Mcnamara, John wrote: > > -----Original Message----- > > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Jerin Jacob > > Sent: Tuesday, January 17, 2017 4:30 AM > > To: Horton, Remy > > Cc: dev@dpdk.org; Pattan, Reshma ; Thomas > > Monjalon > > Subject: Re: [dpdk-dev] [PATCH v7 5/6] lib: added new library for latency > > stats > > > > On Mon, Jan 16, 2017 at 04:19:32PM +0000, Remy Horton wrote: > > > From: Reshma Pattan > > > > > > Add a library designed to calculate latency statistics and report them > > > to the application when queried. The library measures minimum, average > > > and maximum latencies, and jitter in nano seconds. The current > > > implementation supports global latency stats, i.e. per application > > stats. > > > > > > Signed-off-by: Reshma Pattan > > > Signed-off-by: Remy Horton > > > --- > > > MAINTAINERS | 4 + > > > config/common_base | 5 + > > > doc/api/doxy-api-index.md | 1 + > > > doc/api/doxy-api.conf | 1 + > > > doc/guides/rel_notes/release_17_02.rst | 5 + > > > lib/Makefile | 1 + > > > lib/librte_latencystats/Makefile | 57 +++ > > > lib/librte_latencystats/rte_latencystats.c | 389 > > +++++++++++++++++++++ > > > lib/librte_latencystats/rte_latencystats.h | 146 ++++++++ > > > .../rte_latencystats_version.map | 10 + > > > lib/librte_mbuf/rte_mbuf.h | 3 + > > > > It is a value added feature for DPDK. But what is the plan for > > incorporating the mbuf change? I have 8 month old mbuf change for ARM for > > natural alignment. If we are accepting any mbuf change then we need to > > include outstanding mbuf changes to avoid future ABI breakage. > > > > http://dpdk.org/dev/patchwork/patch/12878/ > > > > Hi Jerin, Hi John, > > As far as I know the plan was to reach some sort of consensus on the mbuf > structure at the DPDK Userspace 2016, during and after Olivier's > presentation and then to make those changes during 17.02. > > However, I believe Olivier had other work commitments in this release and > wasn't able to work on the mbuf changes. > > The above mbuf change (and addition at the end of the struct) should > have gone into that mbuf rework, along with your changes. > > However, since the mbuf rework didn't happen we need to add the field in > this release. So we don't care the mbuf ABI breakage in the next release. This wasn't the message I got earlier for ARM's mbuf change. http://dpdk.org/dev/patchwork/patch/12878/ > > I guess the difference between the above change and your change is that > the latter is more complex and potentially affect performance, and as such > makes more sense as part of a rework. The mbuf natural alignment is a not complex change, it just moving the field and it does not have any performance impact on IA nor nobody reported any performance regression on IA. There is nothing against you or this feature. The only part concerns me that some set of patches can always override any rule and include in the release (even as marking as EXPERIMENTAL) because of its important for some set of consumers. Another set has to wait in the queue because its not important for some people. For me, it is not a sign of vendor neutral open source project. Jerin > > Perhaps we, as a community, should commit to the mbuf rework in 17.05 > and make sure it gets done. > > John > > > > > >