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 B05C7A0A02; Thu, 25 Mar 2021 14:57:28 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 9ECE94067B; Thu, 25 Mar 2021 14:57:28 +0100 (CET) Received: from mga17.intel.com (mga17.intel.com [192.55.52.151]) by mails.dpdk.org (Postfix) with ESMTP id 6AADA40147 for ; Thu, 25 Mar 2021 14:57:26 +0100 (CET) IronPort-SDR: 1LaWUuG9Yk1I3O4kdgCF4Uxrb03FaL+lfpHha+BOwc4FGbvMylRRU4x/dWMOsGtfib4dsd1kyv XgwFMYMkufow== X-IronPort-AV: E=McAfee;i="6000,8403,9933"; a="170910536" X-IronPort-AV: E=Sophos;i="5.81,277,1610438400"; d="scan'208";a="170910536" Received: from fmsmga004.fm.intel.com ([10.253.24.48]) by fmsmga107.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 25 Mar 2021 06:57:25 -0700 IronPort-SDR: Sm+izSTX5hQCTEZkAinrwY89VV0tGAddk7nc2988IE2vKxIYlqlze31+LIytwLwwMwOjwCIA47 k7/iZY9iD/vQ== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.81,277,1610438400"; d="scan'208";a="436466489" Received: from silpixa00399126.ir.intel.com ([10.237.223.116]) by fmsmga004.fm.intel.com with ESMTP; 25 Mar 2021 06:57:24 -0700 From: Bruce Richardson To: dev@dpdk.org Cc: Bruce Richardson Date: Thu, 25 Mar 2021 13:57:03 +0000 Message-Id: <20210325135707.325536-1-bruce.richardson@intel.com> X-Mailer: git-send-email 2.27.0 In-Reply-To: <20210310172430.412405-1-bruce.richardson@intel.com> References: <20210310172430.412405-1-bruce.richardson@intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Subject: [dpdk-dev] [PATCH v2 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). V2: Rebased on latest main branch. Bruce Richardson (4): telemetry: use rte_log for logging telemetry: make the legacy registration function internal telemetry: rename 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 | 25 ------ lib/librte_telemetry/telemetry.c | 76 +++++++++---------- ...elemetry_legacy.h => telemetry_internal.h} | 41 +++++++++- lib/librte_telemetry/telemetry_legacy.c | 2 +- lib/librte_telemetry/version.map | 2 +- 9 files changed, 87 insertions(+), 90 deletions(-) rename lib/librte_telemetry/{rte_telemetry_legacy.h => telemetry_internal.h} (65%) -- 2.27.0