From mboxrd@z Thu Jan 1 00:00:00 1970 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) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga002.jf.intel.com ([10.7.209.21]) by fmsmga107.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 24 Apr 2018 12:09:12 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.49,324,1520924400"; d="scan'208";a="53320799" Received: from irsmsx106.ger.corp.intel.com ([163.33.3.31]) by orsmga002.jf.intel.com with ESMTP; 24 Apr 2018 12:09:11 -0700 Received: from irsmsx104.ger.corp.intel.com ([169.254.5.171]) by IRSMSX106.ger.corp.intel.com ([169.254.8.37]) with mapi id 14.03.0319.002; Tue, 24 Apr 2018 20:09:10 +0100 From: "Mokhtar, Amr" To: "De Lara Guarch, Pablo" , "Chalupnik, KamilX" , "dev@dpdk.org" CC: "Chalupnik, KamilX" Thread-Topic: [dpdk-dev] [PATCH v2] baseband/turbo_sw: offload cost measurement test Thread-Index: AQHT2/PyY2bGE+95rUWEhaGdz5wzK6QQQ9/w 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> In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiNWFkZWI1NzItYzk2ZC00ZDAzLWEzYWEtZWE3YjJjM2UwZDAzIiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX05UIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE3LjIuNS4xOCIsIlRydXN0ZWRMYWJlbEhhc2giOiI3dHN1WEU2Z0xnb2V2cVdvMGhKVEloblpsclVjYjlkN3YySVVtRDZ1ZHpsek9MQjE3YXo4Y1AwYVwvUTdjQzg1ZyJ9 x-ctpclassification: CTP_NT dlp-product: dlpe-windows dlp-version: 11.0.200.100 dlp-reaction: no-action x-originating-ip: [163.33.239.182] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Subject: Re: [dpdk-dev] [PATCH v2] baseband/turbo_sw: offload cost measurement test 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, 24 Apr 2018 19:09:14 -0000 > -----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 > >