From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rajagopalan Sivaramakrishnan Subject: Re: [PATCH v3] pipeline: add statistics for librte_pipeline Date: Thu, 28 May 2015 19:50:17 +0000 Message-ID: References: <3EB4FA525960D640B5BDFFD6A3D891263236E6CA@IRSMSX108.ger.corp.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable To: "Dumitrescu, Cristian" , "dev@dpdk.org" Return-path: Received: from na01-bl2-obe.outbound.protection.outlook.com (mail-bl2on0116.outbound.protection.outlook.com [65.55.169.116]) by dpdk.org (Postfix) with ESMTP id 228FEC390 for ; Thu, 28 May 2015 21:50:20 +0200 (CEST) In-Reply-To: <3EB4FA525960D640B5BDFFD6A3D891263236E6CA@IRSMSX108.ger.corp.intel.com> Content-Language: en-US Content-ID: <6FA727828278F7408EA952FBF34819E2@namprd05.prod.outlook.com> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" My first preference would be to enable stats always. However, if the majority feels that it should be optional, your preference of 3, 2, 1 seems fine to me. I hope the same decision will apply to port/table/other stats. Raja On 5/28/15, 12:26 PM, "Dumitrescu, Cristian" wrote: >Hi Raja, > >Thanks for your input. > >I think we have the following options identified so far for stats >collection configuration: > >1. Stats configuration through the RTE_LOG_LEVEL >2. Single configuration flag global for all DPDK libraries >3. Single configuration flag per DPDK library > >It would be good if Thomas and Stephen, as well as others, would reply >with their preference order. > >My personal preference order is: 3., 2., 1., but I can work with any of >the above that is identified by the majority of the replies. My goal >right now is reaching a conclusion on this item as soon as we can. > >Regards, >Cristian > > > >> -----Original Message----- >> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Rajagopalan >> Sivaramakrishnan >> Sent: Wednesday, May 27, 2015 11:45 PM >> To: dev@dpdk.org >> Subject: Re: [dpdk-dev] [PATCH v3] pipeline: add statistics for >>librte_pipeline >>=20 >>=20 >> > > You also reiterate that you would like to have the stats always >>enabled. >> You >> > can definitely do this, it is one of the available choices, but why >>not also >> > accommodate the users that want to pick the opposite choice? Why force >> > apps to spend cycles on stats if the app either does not want these >> counters >> > (library counters not relevant for that app, maybe the app is only >> interested >> > in maintaining some other stats that it implements itself) or do not >>want >> > them anymore (maybe they only needed them during debug phase), etc? >> > Jay asked this question, and I did my best in my reply to describe our >> > motivation (http://www.dpdk.org/ml/archives/dev/2015- >> May/017992.html). >> > Maybe you missed that post, it would be good to get your reply on >>this one >> > too. >> > >> > I want to see DPDK get out of the config madness. >> > This is real code, not an Intel benchmark special. >>=20 >>=20 >> I agree that statistics will definitely be required in most real-world >>production >> environments and the overhead >> from per-core stats gathering will be minimal if the data structures >>are such >> that CPU cache thrashing is avoided. >> However, if there are scenarios where it is desirable to turn stats >>off, I think >> we can live with a config option. >> I am not comfortable with using the log level to enable/disable >>statistics as >> they are not really related. A >> separate config option for stats collection seems like a reasonable >> compromise. >>=20 >> Raja