[coreboot-gerrit] Patch set updated for coreboot: x86: Make reading / writing of CRx registers 64bit proof

[Stefan Reinauer]

Stefan Reinauer ([e-mail]) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/10576

-gerrit

commit b8e78d1d8955347a698b0261270a007bbdaf7d86
Author: Stefan Reinauer 
Date:   Wed Jun 17 16:11:18 2015 -0700

x86: Make reading / writing of CRx registers 64bit proof

Change-Id: I782007fe9754ec3ae0b5dc31e7865f7e46cfbc74
    Signed-off-by: Stefan Reinauer 
    Signed-off-by: Scott Duplichan 
---
 src/include/cpu/x86/cr.h | 29 +++++++++++++++++++----------
 1 file changed, 19 insertions(+), 10 deletions(-)

diff --git a/src/include/cpu/x86/cr.h b/src/include/cpu/x86/cr.h
index 4d2be3a..8f77aec 100644
--- a/src/include/cpu/x86/cr.h
+++ b/src/include/cpu/x86/cr.h
@@ -32,46 +32,55 @@
 #define COMPILER_BARRIER "memory"
 #endif

-static alwaysinline uint32_t read_cr0(void)
+#ifdef __x86_64__
+#define CRx_TYPE uint64_t
+#define CRx_IN   "q"
+#define CRx_RET  "=q"
+#else
+#define CRx_TYPE uint32_t
+#define CRx_IN   "r"
+#define CRx_RET  "=r"
+#endif
+static alwaysinline CRx_TYPE read_cr0(void)
 {
-	uint32_t value;
+	CRx_TYPE value;
    __asm__ __volatile__ (
    	"mov %%cr0, %0"
-		: "=r" (value)
+		: CRx_RET (value)
    	:
    	: COMPILER_BARRIER
    );
    return value;
 }

-static alwaysinline void write_cr0(uint32_t data)
+static alwaysinline void write_cr0(CRx_TYPE data)
 {
    __asm__ __volatile__ (
    	"mov %0, %%cr0"
    	:
-		: "r" (data)
+		: CRx_IN (data)
    	: COMPILER_BARRIER
    );
 }

-static alwaysinline uint32_t read_cr4(void)
+static alwaysinline CRx_TYPE read_cr4(void)
 {
-	uint32_t value;
+	CRx_TYPE value;
    __asm__ __volatile__ (
    	"mov %%cr4, %0"
-		: "=r" (value)
+		: CRx_RET (value)
    	:
    	: COMPILER_BARRIER
    );
    return value;
 }

-static alwaysinline void write_cr4(uint32_t data)
+static alwaysinline void write_cr4(CRx_TYPE data)
 {
    __asm__ __volatile__ (
    	"mov %0, %%cr4"
    	:
-		: "r" (data)
+		: CRx_IN (data)
    	: COMPILER_BARRIER
    );
 }