linux-kernel - [RFC PATCH 06/10] x86/fpu/xcr: Return all enabled xfeature if CPU doesn't support

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <20211203003636.11417-7-jiaxun.yang@flygoat.com>
Date:   Fri,  3 Dec 2021 00:36:32 +0000
From:   Jiaxun Yang <jiaxun.yang@...goat.com>
To:     x86@...nel.org
Cc:     tglx@...utronix.de, mingo@...hat.com, bp@...en8.de,
        dave.hansen@...ux.intel.com, hpa@...or.com,
        chang.seok.bae@...el.com, linux-kernel@...r.kernel.org,
        Jiaxun Yang <jiaxun.yang@...goat.com>,
        Jiaxun Yang <j.yang-87@....ed.ac.uk>
Subject: [RFC PATCH 06/10] x86/fpu/xcr: Return all enabled xfeature if CPU doesn't support

Read from XCR0 when CPU doesn't support X86_FEATURE_XGETBV1.
It just assumes that all enabled features are in use.

Signed-off-by: Jiaxun Yang <j.yang-87@....ed.ac.uk>
---
 arch/x86/include/asm/fpu/xcr.h | 16 ++++++++++++++--
 1 file changed, 14 insertions(+), 2 deletions(-)

diff --git a/arch/x86/include/asm/fpu/xcr.h b/arch/x86/include/asm/fpu/xcr.h
index 9656a5bc6fea..e718074ef141 100644
--- a/arch/x86/include/asm/fpu/xcr.h
+++ b/arch/x86/include/asm/fpu/xcr.h
@@ -2,6 +2,8 @@
 #ifndef _ASM_X86_FPU_XCR_H
 #define _ASM_X86_FPU_XCR_H
 
+#include <asm/alternative.h>
+
 #define XCR_XFEATURE_ENABLED_MASK	0x00000000
 #define XCR_XFEATURE_IN_USE_MASK	0x00000001
 
@@ -25,11 +27,21 @@ static inline void xsetbv(u32 index, u64 value)
  * Return a mask of xfeatures which are currently being tracked
  * by the processor as being in the initial configuration.
  *
- * Callers should check X86_FEATURE_XGETBV1.
+ * It will return XCR0, which is all enabled xfeatures in case
+ * CPU doesn't support X86_FEATURE_XGETBV1.
  */
 static inline u64 xfeatures_in_use(void)
 {
-	return xgetbv(XCR_XFEATURE_IN_USE_MASK);
+	u32 eax, edx;
+
+	asm volatile(ALTERNATIVE(
+			"mov  $0, %%ecx",
+			"mov  $1, %%ecx",
+			X86_FEATURE_XGETBV1)
+			"xgetbv"
+			: "=a" (eax), "=d" (edx) :: "ecx");
+
+	return eax + ((u64)edx << 32);
 }
 
 #endif /* _ASM_X86_FPU_XCR_H */
-- 
2.30.2