[hyperion] 25620: * __cpuid, __rdtsc, __readmsr, __readpmc, __writemsr, __wbinvd, __readcr0, __readcr2, __readcr3, __readcr4: show me a fan of the extended inline asm syntax and I'll show you a SPARKLING FIST OF BURNING RAGE. Sprinkled __volatile__ liberally because there is no way to say "has side effects" OR "delete if outputs are ignored", only BOTH or NEITHER. This breaks Visual C++ compatibility ever so subtly. Tread carefully. THANK YOU GCC TEAM for keeping things interesting! * __writemsr

Author: hyperion
Date: Wed Jan 24 22:07:19 2007
New Revision: 25620

URL: http://svn.reactos.org/svn/reactos?rev=25620&view=rev
Log:
 * __cpuid, __rdtsc, __readmsr, __readpmc, __writemsr, __wbinvd, __readcr0, __readcr2, __readcr3, __readcr4: show me a fan of the extended inline asm syntax and I'll show you a SPARKLING FIST OF BURNING RAGE. Sprinkled __volatile__ liberally because there is no way to say "has side effects" OR "delete if outputs are ignored", only BOTH or NEITHER. This breaks Visual C++ compatibility ever so subtly. Tread carefully. THANK YOU GCC TEAM for keeping things interesting!
 * __writemsr: pass 64-bit value with "A" constraint
 * __writecr0, __writecr3, __writecr4: "memory" clobber becase Visual C++ does, too

Modified:
    trunk/reactos/include/psdk/intrin.h

Modified: trunk/reactos/include/psdk/intrin.h
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/include/psdk/intrin.h?rev=2...
==============================================================================
--- trunk/reactos/include/psdk/intrin.h (original)
+++ trunk/reactos/include/psdk/intrin.h Wed Jan 24 22:07:19 2007
@@ -833,13 +833,13 @@
 /*** System information ***/
 static __inline__ __attribute__((always_inline)) void __cpuid(int CPUInfo[], const int InfoType)
 {
-	__asm__("cpuid" : "=a" (CPUInfo[0]), "=b" (CPUInfo[1]), "=c" (CPUInfo[2]), "=d" (CPUInfo[3]) : "a" (InfoType));
+	__asm__ __volatile__("cpuid" : "=a" (CPUInfo[0]), "=b" (CPUInfo[1]), "=c" (CPUInfo[2]), "=d" (CPUInfo[3]) : "a" (InfoType));
 }

static __inline__ __attribute__((always_inline)) unsigned long long __rdtsc(void)
 {
    unsigned long long retval;
-	__asm__("rdtsc" : "=A" (retval));
+	__asm__ __volatile__("rdtsc" : "=A"(retval));
    return retval;
 }

@@ -870,44 +870,44 @@
 static __inline__ __attribute__((always_inline)) unsigned long __readcr0(void)
 {
    unsigned long value;
-	__asm__("mov %%cr0, %[value]" : [value] "=q" (value));
+	__asm__ __volatile__("mov %%cr0, %[value]" : [value] "=q" (value));
    return value;
 }

static __inline__ __attribute__((always_inline)) unsigned long __readcr2(void)
 {
    unsigned long value;
-	__asm__("mov %%cr2, %[value]" : [value] "=q" (value));
+	__asm__ __volatile__("mov %%cr2, %[value]" : [value] "=q" (value));
    return value;
 }

static __inline__ __attribute__((always_inline)) unsigned long __readcr3(void)
 {
    unsigned long value;
-	__asm__("mov %%cr3, %[value]" : [value] "=q" (value));
+	__asm__ __volatile__("mov %%cr3, %[value]" : [value] "=q" (value));
    return value;
 }

static __inline__ __attribute__((always_inline)) unsigned long __readcr4(void)
 {
    unsigned long value;
-	__asm__("mov %%cr4, %[value]" : [value] "=q" (value));
+	__asm__ __volatile__("mov %%cr4, %[value]" : [value] "=q" (value));
    return value;
 }

static __inline__ __attribute__((always_inline)) void __writecr0(const unsigned long long Data)
 {
-	__asm__("mov %[Data], %%cr0" : : [Data] "q" ((const unsigned long)(Data & 0xFFFFFFFF)));
+	__asm__("mov %[Data], %%cr0" : : [Data] "q" ((const unsigned long)(Data & 0xFFFFFFFF)) : "memory");
 }

static __inline__ __attribute__((always_inline)) void __writecr3(const unsigned long long Data)
 {
-	__asm__("mov %[Data], %%cr3" : : [Data] "q" ((const unsigned long)(Data & 0xFFFFFFFF)));
+	__asm__("mov %[Data], %%cr3" : : [Data] "q" ((const unsigned long)(Data & 0xFFFFFFFF)) : "memory");
 }

static __inline__ __attribute__((always_inline)) void __writecr4(const unsigned long long Data)
 {
-	__asm__("mov %[Data], %%cr4" : : [Data] "q" ((const unsigned long)(Data & 0xFFFFFFFF)));
+	__asm__("mov %[Data], %%cr4" : : [Data] "q" ((const unsigned long)(Data & 0xFFFFFFFF)) : "memory");
 }

static __inline__ __attribute__((always_inline)) void __invlpg(void * const Address)
@@ -920,26 +920,19 @@
 static __inline__ __attribute__((always_inline)) unsigned long long __readmsr(const int reg)
 {
    unsigned long long retval;
-	__asm__("rdmsr" : "=A" (retval) : "c" (reg));
+	__asm__ __volatile__("rdmsr" : "=A" (retval) : "c" (reg));
    return retval;
 }

static __inline__ __attribute__((always_inline)) void __writemsr(const unsigned long Register, const unsigned long long Value)
 {
-	__asm__
-	(
-		"wrmsr" :
-		:
-		"a" ((unsigned long)((Value >>  0) & 0xFFFFFFFF)),
-		"d" ((unsigned long)((Value >> 32) & 0xFFFFFFFF)),
-		"c" (Register)
-	);
+	__asm__ __volatile__("wrmsr" : : "A" (Value), "c" (Register));
 }

static __inline__ __attribute__((always_inline)) unsigned long long __readpmc(const int counter)
 {
    unsigned long long retval;
-	__asm__("rdpmc" : "=A" (retval) : "c" (counter));
+	__asm__ __volatile__("rdpmc" : "=A" (retval) : "c" (counter));
    return retval;
 }

@@ -953,7 +946,7 @@

static __inline__ __attribute__((always_inline)) void __wbinvd(void)
 {
-	__asm__("wbinvd");
+	__asm__ __volatile__("wbinvd");
 }

#else