From: Kevin O'Connor <kevin@koconnor.net>
Date: Mon, 28 May 2012 22:57:06 +0000 (-0400)
Subject: Make sure to set dependency to segment registers in inline asm.
X-Git-Tag: rel-1.7.1~43
X-Git-Url: http://xenbits.xensource.com/gitweb?a=commitdiff_plain;h=41966dac84ab7918ec57421a2122eaca0bfcff05;p=seabios.git

Make sure to set dependency to segment registers in inline asm.

Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
---

diff --git a/src/system.c b/src/system.c
index 898c3cc..6658e22 100644
--- a/src/system.c
+++ b/src/system.c
@@ -162,8 +162,8 @@ handle_1587(struct bregs *regs)
         // Restore %ds (from %ss)
         "  movw %%ss, %%ax\n"
         "  movw %%ax, %%ds\n"
-        : "+c"(count), "+S"(si)
-        : : "eax", "di", "cc"); // XXX - also clobbers %es
+        : "+c"(count), "+S"(si), "+m" (__segment_ES)
+        : : "eax", "di", "cc");
 
     set_a20(prev_a20_enable);
 
@@ -222,7 +222,7 @@ handle_1589(struct bregs *regs)
         "  movw %%ax, %%ds\n"
         "  movw %%ax, %%ss\n"
         :
-        : "S"(gdt_far)
+        : "S"(gdt_far), "m" (__segment_ES)
         : "eax", "cc");
 }
 
diff --git a/src/util.c b/src/util.c
index 0e69913..dcc0b71 100644
--- a/src/util.c
+++ b/src/util.c
@@ -93,7 +93,7 @@ memset_far(u16 d_seg, void *d_far, u8 c, size_t len)
     asm volatile(
         "rep stosb %%es:(%%di)"
         : "+c"(len), "+D"(d_far)
-        : "a"(c)
+        : "a"(c), "m" (__segment_ES)
         : "cc", "memory");
 }
 
@@ -105,7 +105,7 @@ memset16_far(u16 d_seg, void *d_far, u16 c, size_t len)
     asm volatile(
         "rep stosw %%es:(%%di)"
         : "+c"(len), "+D"(d_far)
-        : "a"(c)
+        : "a"(c), "m" (__segment_ES)
         : "cc", "memory");
 }
 
@@ -137,7 +137,7 @@ memcpy_far(u16 d_seg, void *d_far, u16 s_seg, const void *s_far, size_t len)
         "rep movsb (%%si),%%es:(%%di)\n"
         "movw %w0, %%ds"
         : "=&r"(bkup_ds), "+c"(len), "+S"(s_far), "+D"(d_far)
-        : "r"(s_seg)
+        : "r"(s_seg), "m" (__segment_ES)
         : "cc", "memory");
 }
 
@@ -166,7 +166,7 @@ memcpy(void *d1, const void *s1, size_t len)
         asm volatile(
             "rep movsb (%%esi),%%es:(%%edi)"
             : "+c"(len), "+S"(s1), "+D"(d)
-            : : "cc", "memory");
+            : "m" (__segment_ES) : "cc", "memory");
         return d1;
     }
     // Common case - use 4-byte copy
@@ -174,7 +174,7 @@ memcpy(void *d1, const void *s1, size_t len)
     asm volatile(
         "rep movsl (%%esi),%%es:(%%edi)"
         : "+c"(len), "+S"(s1), "+D"(d)
-        : : "cc", "memory");
+        : "m" (__segment_ES) : "cc", "memory");
     return d1;
 }
 
@@ -183,6 +183,7 @@ memcpy(void *d1, const void *s1, size_t len)
 void
 iomemcpy(void *d, const void *s, u32 len)
 {
+    ASSERT32FLAT();
     yield();
     while (len > 3) {
         u32 copylen = len;