From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Mokhtar, Amr" Subject: Re: [PATCH v2] baseband/turbo_sw: offload cost measurement test Date: Tue, 24 Apr 2018 19:09:09 +0000 Message-ID: <3D3765A8CDB52A4C8B410430AA19CB236ECBD232@IRSMSX104.ger.corp.intel.com> References: <20180404140602.9344-2-kamilx.chalupnik@intel.com> <20180417142724.27748-1-kamilx.chalupnik@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Cc: "Chalupnik, KamilX" To: "De Lara Guarch, Pablo" , "Chalupnik, KamilX" , "dev@dpdk.org" Return-path: Received: from mga17.intel.com (mga17.intel.com [192.55.52.151]) by dpdk.org (Postfix) with ESMTP id 693C137A6 for ; Tue, 24 Apr 2018 21:09:13 +0200 (CEST) In-Reply-To: Content-Language: en-US List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" > -----Original Message----- > From: De Lara Guarch, Pablo > Sent: Tuesday 24 April 2018 18:45 > To: Chalupnik, KamilX ; dev@dpdk.org > Cc: Mokhtar, Amr ; Chalupnik, KamilX > > Subject: RE: [dpdk-dev] [PATCH v2] baseband/turbo_sw: offload cost > measurement test >=20 >=20 >=20 > > -----Original Message----- > > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of KamilX Chalupnik > > Sent: Tuesday, April 17, 2018 3:27 PM > > To: dev@dpdk.org > > Cc: Mokhtar, Amr ; Chalupnik, KamilX > > > > Subject: [dpdk-dev] [PATCH v2] baseband/turbo_sw: offload cost > measurement > > test > > > > New test created to measure offload cost. > > Changes were introduced in API, turbo software driver and test > application. > > >=20 > Shouldn't this be generic to bbdev/baseband drivers in general and not ju= st > turbo? >=20 Yes, it is generic. But the only driver we have right now is the turbo_sw driver. Future driver= s will have a similar support. > > Signed-off-by: KamilX Chalupnik >=20 > ... >=20 > > --- a/lib/librte_bbdev/rte_bbdev.h > > +++ b/lib/librte_bbdev/rte_bbdev.h > > @@ -239,6 +239,10 @@ struct rte_bbdev_stats { > > uint64_t enqueue_err_count; > > /** Total error count on operations dequeued */ > > uint64_t dequeue_err_count; > > +#ifdef RTE_TEST_BBDEV > > + /** It stores turbo decoder/encoder working time. */ > > + uint64_t turbo_perf_time; > > +#endif >=20 > I don't think it is a good idea to use RTE_TEST_BBDEV here. > This macro is used to enable/disable the compilation of the bbdev test ap= p, > so I think it should not be used in the API/PMDs. >=20 > Also, this looks too specific for the Turbo SW PMD to be exposed > as a generic statistic. Well, it should be generic. Probably 'turbo' is a bad comment and name. It's intention is to feedback execution time/cycles back to test app in ord= er to collect the offload cost of the bbdev driver. What is meant by the offload cost is the cycles consumed from the moment of enqueue till the moment the request is put on the acceleration engine inbound sw ring (software) or MMIO operation (hardware). >=20 > > }; > > > > /** > > -- > > 2.5.5 > >