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 BF41B45AD5; Mon, 7 Oct 2024 13:07:35 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 55112402EA; Mon, 7 Oct 2024 13:07:35 +0200 (CEST) Received: from mgamail.intel.com (mgamail.intel.com [192.198.163.17]) by mails.dpdk.org (Postfix) with ESMTP id C681A4026C for ; Mon, 7 Oct 2024 13:07:33 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1728299254; x=1759835254; h=from:to:cc:subject:date:message-id:mime-version: content-transfer-encoding; bh=wRdlpAX/vWAm4cU2IB/MI+q4Y26nCSmd5r9BKp2N+fI=; b=aNWVrv7e2FZTZxEtmkEBhlhjyTS6rec7mfmPqnJKsFT34VxEPAELM+G/ 2wRUFBtrRYexGpcmdxRqNz+ohYGsbyncNMgu6pSiK8dtrl4KdhZCvBU9l tSTEyNmck/GMrma2/ONUX9a/ACPmuN8FPglzahmQg8tM07CyJ/Diy5U1w NvvCkQQP1MWLaqSEBXiqImR8u4qtDu1oT8j+FWIcHfOcNQYmxmy4/SFvr EeYI7bZlir0DR+9H7+S4EO+VKoBaCBt7+0bQgQKfvxNgfiyOrLFXG+aB6 PlIo8Dk0JHsuAEAqhvUX/GJwgA1LaI1kZyEomvrBurJfITT51rxPYg13s g==; X-CSE-ConnectionGUID: kQ6dtU0fSuOJ44H8Hs7mEQ== X-CSE-MsgGUID: 09i6mjOeSdywidz68RRmYg== X-IronPort-AV: E=McAfee;i="6700,10204,11217"; a="27327759" X-IronPort-AV: E=Sophos;i="6.11,184,1725346800"; d="scan'208";a="27327759" Received: from orviesa008.jf.intel.com ([10.64.159.148]) by fmvoesa111.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 07 Oct 2024 04:07:33 -0700 X-CSE-ConnectionGUID: F2sB4bn4S0+MQT0coZ0oOA== X-CSE-MsgGUID: 2GEZ9WurR0ylgqte7LF84w== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.11,184,1725346800"; d="scan'208";a="76273782" Received: from unknown (HELO silpixa00401385.ir.intel.com) ([10.237.214.25]) by orviesa008.jf.intel.com with ESMTP; 07 Oct 2024 04:07:32 -0700 From: Bruce Richardson To: dev@dpdk.org Cc: david.marchand@redhat.com, Bruce Richardson Subject: [PATCH] eal/x86: cache queried CPU flags Date: Mon, 7 Oct 2024 12:06:43 +0100 Message-ID: <20241007110725.377550-1-bruce.richardson@intel.com> X-Mailer: git-send-email 2.43.0 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit 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 Rather than re-querying the HW each time a CPU flag is requested, we can just save the return value in the flags array. This should speed up repeated querying of CPU flags, and provides a workaround for a reported issue where errors are seen with constant querying of the AVX-512 CPU flag from a non-AVX VM. Bugzilla Id: 1501 Signed-off-by: Bruce Richardson --- lib/eal/x86/rte_cpuflags.c | 20 +++++++++++++++----- 1 file changed, 15 insertions(+), 5 deletions(-) diff --git a/lib/eal/x86/rte_cpuflags.c b/lib/eal/x86/rte_cpuflags.c index 26163ab746..62e782fb4b 100644 --- a/lib/eal/x86/rte_cpuflags.c +++ b/lib/eal/x86/rte_cpuflags.c @@ -8,6 +8,7 @@ #include #include #include +#include #include "rte_cpuid.h" @@ -21,12 +22,14 @@ struct feature_entry { uint32_t bit; /**< cpuid register bit */ #define CPU_FLAG_NAME_MAX_LEN 64 char name[CPU_FLAG_NAME_MAX_LEN]; /**< String for printing */ + bool has_value; + bool value; }; #define FEAT_DEF(name, leaf, subleaf, reg, bit) \ [RTE_CPUFLAG_##name] = {leaf, subleaf, reg, bit, #name }, -const struct feature_entry rte_cpu_feature_table[] = { +struct feature_entry rte_cpu_feature_table[] = { FEAT_DEF(SSE3, 0x00000001, 0, RTE_REG_ECX, 0) FEAT_DEF(PCLMULQDQ, 0x00000001, 0, RTE_REG_ECX, 1) FEAT_DEF(DTES64, 0x00000001, 0, RTE_REG_ECX, 2) @@ -147,7 +150,7 @@ const struct feature_entry rte_cpu_feature_table[] = { int rte_cpu_get_flag_enabled(enum rte_cpu_flag_t feature) { - const struct feature_entry *feat; + struct feature_entry *feat; cpuid_registers_t regs; unsigned int maxleaf; @@ -156,6 +159,8 @@ rte_cpu_get_flag_enabled(enum rte_cpu_flag_t feature) return -ENOENT; feat = &rte_cpu_feature_table[feature]; + if (feat->has_value) + return feat->value; if (!feat->leaf) /* This entry in the table wasn't filled out! */ @@ -163,8 +168,10 @@ rte_cpu_get_flag_enabled(enum rte_cpu_flag_t feature) maxleaf = __get_cpuid_max(feat->leaf & 0x80000000, NULL); - if (maxleaf < feat->leaf) - return 0; + if (maxleaf < feat->leaf) { + feat->value = 0; + goto out; + } #ifdef RTE_TOOLCHAIN_MSVC __cpuidex(regs, feat->leaf, feat->subleaf); @@ -175,7 +182,10 @@ rte_cpu_get_flag_enabled(enum rte_cpu_flag_t feature) #endif /* check if the feature is enabled */ - return (regs[feat->reg] >> feat->bit) & 1; + feat->value = (regs[feat->reg] >> feat->bit) & 1; +out: + feat->has_value = true; + return feat->value; } const char * -- 2.43.0