From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Van Haaren, Harry" Subject: Re: [PATCH v4 03/13] telemetry: initial telemetry infrastructure Date: Tue, 16 Oct 2018 00:45:31 +0000 Message-ID: References: <20181010105134.48079-1-kevin.laatz@intel.com> <20181011165837.81030-1-kevin.laatz@intel.com> <20181011165837.81030-4-kevin.laatz@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Cc: "stephen@networkplumber.org" , "gaetan.rivet@6wind.com" , "shreyansh.jain@nxp.com" , "thomas@monjalon.net" , "mattias.ronnblom@ericsson.com" , "Richardson, Bruce" , Ciara Power , Brian Archbold To: "Laatz, Kevin" , "dev@dpdk.org" Return-path: Received: from mga05.intel.com (mga05.intel.com [192.55.52.43]) by dpdk.org (Postfix) with ESMTP id F139B2BF5 for ; Tue, 16 Oct 2018 02:45:35 +0200 (CEST) In-Reply-To: <20181011165837.81030-4-kevin.laatz@intel.com> 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: Laatz, Kevin > Sent: Thursday, October 11, 2018 9:58 AM > To: dev@dpdk.org > Cc: Van Haaren, Harry ; > stephen@networkplumber.org; gaetan.rivet@6wind.com; shreyansh.jain@nxp.co= m; > thomas@monjalon.net; mattias.ronnblom@ericsson.com; Richardson, Bruce > ; Ciara Power ; Brian > Archbold ; Laatz, Kevin > Subject: [PATCH v4 03/13] telemetry: initial telemetry infrastructure >=20 > From: Ciara Power >=20 > This patch adds the infrastructure and initial code for the telemetry > library. >=20 > The telemetry init is registered with eal_init(). We can then check to se= e > if --telemetry was passed as an eal flag. If --telemetry was parsed, then > we call telemetry init at the end of eal init. >=20 > Control threads are used to get CPU cycles for telemetry, which are > configured in this patch also. >=20 > Signed-off-by: Ciara Power > Signed-off-by: Brian Archbold > Signed-off-by: Kevin Laatz > --- I think we need to set the Makefile default to =3DN, and distros/users can = enable if they have Jansson available. With that update: Acked-by: Harry van Haaren