From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id F401CA056A; Wed, 10 Mar 2021 18:24:40 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id CA7F322A348; Wed, 10 Mar 2021 18:24:40 +0100 (CET) Received: from mga04.intel.com (mga04.intel.com [192.55.52.120]) by mails.dpdk.org (Postfix) with ESMTP id 19A1D40F35 for ; Wed, 10 Mar 2021 18:24:38 +0100 (CET) IronPort-SDR: EN+N8NpNeVjWrg2pY7f4NB6Ao4JF1Ubuo4g+xexI349KyjHfv6YoIkIFJqNE8cIAp7JttnMXwm iKot+Mu25pEQ== X-IronPort-AV: E=McAfee;i="6000,8403,9919"; a="186141272" X-IronPort-AV: E=Sophos;i="5.81,237,1610438400"; d="scan'208";a="186141272" Received: from orsmga008.jf.intel.com ([10.7.209.65]) by fmsmga104.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 10 Mar 2021 09:24:36 -0800 IronPort-SDR: PcCGVE2pBiQDmQcPkdaVP/SiTEXOGHQtYMGiXlBbDJgA1kn8KOPrMtx2vgRbQbsHk1tGwOUcoD xu6PmHR3hPUg== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.81,237,1610438400"; d="scan'208";a="410271617" Received: from silpixa00399126.ir.intel.com ([10.237.223.188]) by orsmga008.jf.intel.com with ESMTP; 10 Mar 2021 09:24:35 -0800 From: Bruce Richardson To: dev@dpdk.org Cc: Bruce Richardson Date: Wed, 10 Mar 2021 17:24:26 +0000 Message-Id: <20210310172430.412405-1-bruce.richardson@intel.com> X-Mailer: git-send-email 2.27.0 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Subject: [dpdk-dev] [PATCH 0/4] telemetry logging improvements and cleanup X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" This set adds support for using the regular rte_log functions from the telemetry library; avoiding circular dependencies by having EAL register the telemetry library itself and then passing the required handles to that library as part of the telemetry_init call. Beyond this change, the other three patches are cleanups to ensure that all internal functions are clearly separate from the public APIs. (Patches 3 & 4 may be merged into a single one on apply, for I've kept them separate for now for clarity). Bruce Richardson (4): telemetry: use rte_log for logging telemetry: make the legacy registration function internal telemetry: create internal-only header file telemetry: move init function to internal header doc/guides/rel_notes/release_21_05.rst | 5 ++ lib/librte_eal/freebsd/eal.c | 12 +-- lib/librte_eal/linux/eal.c | 12 +-- lib/librte_metrics/rte_metrics_telemetry.c | 2 +- lib/librte_telemetry/rte_telemetry.h | 22 ------ lib/librte_telemetry/telemetry.c | 74 +++++++++---------- ...elemetry_legacy.h => telemetry_internal.h} | 37 +++++++++- lib/librte_telemetry/telemetry_legacy.c | 2 +- lib/librte_telemetry/version.map | 2 +- 9 files changed, 82 insertions(+), 86 deletions(-) rename lib/librte_telemetry/{rte_telemetry_legacy.h => telemetry_internal.h} (67%) -- 2.27.0