From patchwork Fri Feb 5 19:39:33 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Bruce Richardson X-Patchwork-Id: 87805 X-Patchwork-Delegate: thomas@monjalon.net Return-Path: X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 0E2AAA0524; Fri, 5 Feb 2021 20:40:00 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id BE46640682; Fri, 5 Feb 2021 20:39:59 +0100 (CET) Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) by mails.dpdk.org (Postfix) with ESMTP id 2CA404067B; Fri, 5 Feb 2021 20:39:57 +0100 (CET) IronPort-SDR: MMtJqC1W4pkEEcD+1hzS0v560a33WidSJqSMDIodZpB/s0P4T1kKQ7AJ57qQOLYRhd+dxFR6i7 WZoC0jdJ02fw== X-IronPort-AV: E=McAfee;i="6000,8403,9886"; a="177974767" X-IronPort-AV: E=Sophos;i="5.81,156,1610438400"; d="scan'208";a="177974767" Received: from orsmga004.jf.intel.com ([10.7.209.38]) by fmsmga102.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 05 Feb 2021 11:39:50 -0800 IronPort-SDR: 4CFH0KUjeuA8NAnqkqvP6FJ9D4i4c7fxaWKcfD3At5lrwfx2V1oK5bLG9gTweOfgpBbVPJ6ISv +7pB4FXpm7VA== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.81,156,1610438400"; d="scan'208";a="508647053" Received: from silpixa00399126.ir.intel.com ([10.237.222.4]) by orsmga004.jf.intel.com with ESMTP; 05 Feb 2021 11:39:48 -0800 From: Bruce Richardson To: dev@dpdk.org Cc: thomas@monjalon.net, david.marchand@redhat.com, Bruce Richardson , stable@dpdk.org, Ray Kinsella , Neil Horman , Kevin Laatz Date: Fri, 5 Feb 2021 19:39:33 +0000 Message-Id: <20210205193933.410011-1-bruce.richardson@intel.com> X-Mailer: git-send-email 2.27.0 MIME-Version: 1.0 Subject: [dpdk-dev] [PATCH] eal: fix querying DPDK version at runtime 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" For using a DPDK application, such as OVS, which is dynamically linked, the DPDK version in use should always report the actual version, not the version used at build time. This incorrect behaviour can be seen by building OVS against one version of DPDK and running it against a later one. Using "ovs-vsctl list Open_vSwitch" to query basic info, the dpdk_version returned will be the build version not the currently running one - which can be verified using the DPDK telemetry library client. $ sudo ovs-vsctl list Open_vSwitch | grep dpdk_version dpdk_version : "DPDK 20.11.0-rc4" $ echo quit | sudo dpdk-telemetry.py Connecting to /var/run/dpdk/rte/dpdk_telemetry.v2 {"version": "DPDK 21.02.0-rc2", "pid": 405659, "max_output_len": 16384} --> To fix this, we need to convert the rte_version() function, and any other necessary parts of the rte_version.h, to be actual functions in EAL, not just inlines/macros. The only complication in doing so is that telemetry library cannot call rte_version() directly, and instead needs the version string passed in on init. Fixes: af75078fece3 ("first public release") Cc: stable@dpdk.org Signed-off-by: Bruce Richardson --- While new functions are technically supposed to be experimental, I don't believe it should apply in this case --- lib/librte_eal/common/meson.build | 2 + lib/librte_eal/common/rte_version.c | 46 ++++++++++++++++++++ lib/librte_eal/freebsd/eal.c | 1 + lib/librte_eal/include/rte_version.h | 63 ++++++++++++++++------------ lib/librte_eal/linux/eal.c | 1 + lib/librte_eal/version.map | 7 ++++ lib/librte_telemetry/rte_telemetry.h | 2 +- lib/librte_telemetry/telemetry.c | 12 +++--- 8 files changed, 101 insertions(+), 33 deletions(-) create mode 100644 lib/librte_eal/common/rte_version.c -- 2.27.0 diff --git a/lib/librte_eal/common/meson.build b/lib/librte_eal/common/meson.build index 39d4272976..70bd854fec 100644 --- a/lib/librte_eal/common/meson.build +++ b/lib/librte_eal/common/meson.build @@ -36,6 +36,7 @@ if is_windows 'rte_random.c', 'rte_reciprocal.c', 'rte_service.c', + 'rte_version.c', ) subdir_done() endif @@ -79,6 +80,7 @@ sources += files( 'rte_random.c', 'rte_reciprocal.c', 'rte_service.c', + 'rte_version.c', ) if is_linux diff --git a/lib/librte_eal/common/rte_version.c b/lib/librte_eal/common/rte_version.c new file mode 100644 index 0000000000..4ae5d66c89 --- /dev/null +++ b/lib/librte_eal/common/rte_version.c @@ -0,0 +1,46 @@ +/* SPDX-License-Identifier: BSD-3-Clause + * Copyright(c) 2021 Intel Corporation + */ + +#include + +const char * +rte_version_prefix(void) { return RTE_VER_PREFIX; } + +unsigned int +rte_version_year(void) { return RTE_VER_YEAR; } + +unsigned int +rte_version_month(void) { return RTE_VER_MONTH; } + +unsigned int +rte_version_minor(void) { return RTE_VER_MINOR; } + +const char * +rte_version_suffix(void) { return RTE_VER_SUFFIX; } + +unsigned int +rte_version_release(void) { return RTE_VER_RELEASE; } + +const char * +rte_version(void) +{ + static char version[32]; + if (version[0] != 0) + return version; + if (strlen(RTE_VER_SUFFIX) == 0) + snprintf(version, sizeof(version), "%s %d.%02d.%d", + RTE_VER_PREFIX, + RTE_VER_YEAR, + RTE_VER_MONTH, + RTE_VER_MINOR); + else + snprintf(version, sizeof(version), "%s %d.%02d.%d%s%d", + RTE_VER_PREFIX, + RTE_VER_YEAR, + RTE_VER_MONTH, + RTE_VER_MINOR, + RTE_VER_SUFFIX, + RTE_VER_RELEASE); + return version; +} diff --git a/lib/librte_eal/freebsd/eal.c b/lib/librte_eal/freebsd/eal.c index 51478358c7..3f4dde6f85 100644 --- a/lib/librte_eal/freebsd/eal.c +++ b/lib/librte_eal/freebsd/eal.c @@ -943,6 +943,7 @@ rte_eal_init(int argc, char **argv) if (!internal_conf->no_telemetry) { const char *error_str = NULL; if (rte_telemetry_init(rte_eal_get_runtime_dir(), + rte_version(), &internal_conf->ctrl_cpuset, &error_str) != 0) { rte_eal_init_alert(error_str); diff --git a/lib/librte_eal/include/rte_version.h b/lib/librte_eal/include/rte_version.h index f7a3a1ebcf..2f3f727b46 100644 --- a/lib/librte_eal/include/rte_version.h +++ b/lib/librte_eal/include/rte_version.h @@ -28,38 +28,47 @@ extern "C" { * All version numbers in one to compare with RTE_VERSION_NUM() */ #define RTE_VERSION RTE_VERSION_NUM( \ - RTE_VER_YEAR, \ - RTE_VER_MONTH, \ - RTE_VER_MINOR, \ - RTE_VER_RELEASE) + rte_version_year(), \ + rte_version_month(), \ + rte_version_minor(), \ + rte_version_release()) + +/** + * Function to return DPDK version prefix string + */ +const char *rte_version_prefix(void); + +/** + * Function to return DPDK version year + */ +unsigned int rte_version_year(void); + +/** + * Function to return DPDK version month + */ +unsigned int rte_version_month(void); + +/** + * Function to return DPDK minor version number + */ +unsigned int rte_version_minor(void); + +/** + * Function to return DPDK version suffix for any release candidates + */ +const char *rte_version_suffix(void); + +/** + * Function to return DPDK version release candidate value + */ +unsigned int rte_version_release(void); /** * Function returning version string * @return - * string + * DPDK version string */ -static inline const char * -rte_version(void) -{ - static char version[32]; - if (version[0] != 0) - return version; - if (strlen(RTE_VER_SUFFIX) == 0) - snprintf(version, sizeof(version), "%s %d.%02d.%d", - RTE_VER_PREFIX, - RTE_VER_YEAR, - RTE_VER_MONTH, - RTE_VER_MINOR); - else - snprintf(version, sizeof(version), "%s %d.%02d.%d%s%d", - RTE_VER_PREFIX, - RTE_VER_YEAR, - RTE_VER_MONTH, - RTE_VER_MINOR, - RTE_VER_SUFFIX, - RTE_VER_RELEASE); - return version; -} +const char *rte_version(void); #ifdef __cplusplus } diff --git a/lib/librte_eal/linux/eal.c b/lib/librte_eal/linux/eal.c index 32b48c3de9..3492926117 100644 --- a/lib/librte_eal/linux/eal.c +++ b/lib/librte_eal/linux/eal.c @@ -1316,6 +1316,7 @@ rte_eal_init(int argc, char **argv) if (!internal_conf->no_telemetry) { const char *error_str = NULL; if (rte_telemetry_init(rte_eal_get_runtime_dir(), + rte_version(), &internal_conf->ctrl_cpuset, &error_str) != 0) { rte_eal_init_alert(error_str); diff --git a/lib/librte_eal/version.map b/lib/librte_eal/version.map index fce90a112f..756c60ed1d 100644 --- a/lib/librte_eal/version.map +++ b/lib/librte_eal/version.map @@ -199,6 +199,13 @@ DPDK_21 { rte_uuid_is_null; rte_uuid_parse; rte_uuid_unparse; + rte_version; + rte_version_minor; + rte_version_month; + rte_version_prefix; + rte_version_release; + rte_version_suffix; + rte_version_year; rte_vfio_clear_group; rte_vfio_container_create; rte_vfio_container_destroy; diff --git a/lib/librte_telemetry/rte_telemetry.h b/lib/librte_telemetry/rte_telemetry.h index 76172222c9..f6c3992a9c 100644 --- a/lib/librte_telemetry/rte_telemetry.h +++ b/lib/librte_telemetry/rte_telemetry.h @@ -311,7 +311,7 @@ rte_telemetry_register_cmd(const char *cmd, telemetry_cb fn, const char *help); */ __rte_experimental int -rte_telemetry_init(const char *runtime_dir, rte_cpuset_t *cpuset, +rte_telemetry_init(const char *runtime_dir, const char *rte_version, rte_cpuset_t *cpuset, const char **err_str); /** diff --git a/lib/librte_telemetry/telemetry.c b/lib/librte_telemetry/telemetry.c index b142729da4..14b4ff5ea7 100644 --- a/lib/librte_telemetry/telemetry.c +++ b/lib/librte_telemetry/telemetry.c @@ -15,7 +15,6 @@ #include #include #include -#include #include "rte_telemetry.h" #include "telemetry_json.h" @@ -48,6 +47,8 @@ struct socket { static struct socket v2_socket; /* socket for v2 telemetry */ static struct socket v1_socket; /* socket for v1 telemetry */ #endif /* !RTE_EXEC_ENV_WINDOWS */ + +static const char *telemetry_version; /* save rte_version */ static char telemetry_log_error[1024]; /* Will contain error on init failure */ /* list of command callbacks, with one command registered by default */ static struct cmd_callback callbacks[TELEMETRY_MAX_CALLBACKS]; @@ -105,7 +106,7 @@ json_info(const char *cmd __rte_unused, const char *params __rte_unused, struct rte_tel_data *d) { rte_tel_data_start_dict(d); - rte_tel_data_add_dict_string(d, "version", rte_version()); + rte_tel_data_add_dict_string(d, "version", telemetry_version); rte_tel_data_add_dict_int(d, "pid", getpid()); rte_tel_data_add_dict_int(d, "max_output_len", MAX_OUTPUT_LEN); return 0; @@ -303,7 +304,7 @@ client_handler(void *sock_id) char info_str[1024]; snprintf(info_str, sizeof(info_str), "{\"version\":\"%s\",\"pid\":%d,\"max_output_len\":%d}", - rte_version(), getpid(), MAX_OUTPUT_LEN); + telemetry_version, getpid(), MAX_OUTPUT_LEN); if (write(s, info_str, strlen(info_str)) < 0) { close(s); return NULL; @@ -481,9 +482,10 @@ telemetry_v2_init(const char *runtime_dir, rte_cpuset_t *cpuset) #endif /* !RTE_EXEC_ENV_WINDOWS */ int32_t -rte_telemetry_init(const char *runtime_dir, rte_cpuset_t *cpuset, - const char **err_str) +rte_telemetry_init(const char *runtime_dir, const char *rte_version, + rte_cpuset_t *cpuset, const char **err_str) { + telemetry_version = rte_version; #ifndef RTE_EXEC_ENV_WINDOWS if (telemetry_v2_init(runtime_dir, cpuset) != 0) { *err_str = telemetry_log_error;