From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga04.intel.com (mga04.intel.com [192.55.52.120]) by dpdk.org (Postfix) with ESMTP id 3281E1094 for ; Tue, 17 Jan 2017 12:19:28 +0100 (CET) Received: from orsmga003.jf.intel.com ([10.7.209.27]) by fmsmga104.fm.intel.com with ESMTP; 17 Jan 2017 03:19:26 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.33,244,1477983600"; d="scan'208";a="923441087" Received: from irsmsx110.ger.corp.intel.com ([163.33.3.25]) by orsmga003.jf.intel.com with ESMTP; 17 Jan 2017 03:19:25 -0800 Received: from irsmsx103.ger.corp.intel.com ([169.254.3.77]) by irsmsx110.ger.corp.intel.com ([169.254.15.101]) with mapi id 14.03.0248.002; Tue, 17 Jan 2017 11:19:24 +0000 From: "Mcnamara, John" To: Jerin Jacob , "Horton, Remy" CC: "dev@dpdk.org" , "Pattan, Reshma" , Thomas Monjalon , "olivier.matz@6wind.com" Thread-Topic: [dpdk-dev] [PATCH v7 5/6] lib: added new library for latency stats Thread-Index: AQHScBSHpoacQ3W5Mky5piO2SUYXrqE8FHOAgABvRXA= Date: Tue, 17 Jan 2017 11:19:24 +0000 Message-ID: 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> In-Reply-To: <20170117042935.GA32676@localhost.localdomain> Accept-Language: en-GB, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-ctpclassification: CTP_IC x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiYTM4MGE5MzAtYjYzNC00NDJlLThhOTItMjRlNmRlMzI0Y2YxIiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX0lDIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE1LjkuNi42IiwiVHJ1c3RlZExhYmVsSGFzaCI6Im9JR1NPTjhpNUlcL1NtckV1eE5KNEFVd1JsQlJVazdtWm5ObFVuNlgwdkZRPSJ9 x-originating-ip: [163.33.239.181] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 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 11:19:28 -0000 > -----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 >=20 > 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 + >=20 > 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. >=20 > http://dpdk.org/dev/patchwork/patch/12878/ >=20 Hi Jerin, 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. 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. Perhaps we, as a community, should commit to the mbuf rework in 17.05 and make sure it gets done. John =20