fixed stupid issue with xed where it reversed the operands for an LMSW

[palacios-OLD.git] / palacios / src / devices / apic.c

diff --git a/palacios/src/devices/apic.c b/palacios/src/devices/apic.c
index 77f727b..df9dd12 100644 (file)
--- a/palacios/src/devices/apic.c
+++ b/palacios/src/devices/apic.c
@@ -382,8 +382,10 @@ static int apic_read(addr_t guest_addr, void * dst, uint_t length, void * priv_d
 
   switch (reg_addr & ~0x3) {
   case EOI_OFFSET:
-    PrintError("Attempting to read from write only register\n");
-    return -1;
+    // Well, only an idiot would read from a architectural write only register
+    // Oh, Hello Linux.
+    //    PrintError("Attempting to read from write only register\n");
+    //    return -1;
     break;
 
     // data registers
@@ -652,8 +654,12 @@ static int apic_write(addr_t guest_addr, void * src, uint_t length, void * priv_
   case TRIG_OFFSET7:
   case PPR_OFFSET:
   case EXT_APIC_FEATURE_OFFSET:
-    PrintError("Attempting to write to read only register %p\n", (void *)reg_addr);
+#if 1
+    PrintError("Attempting to write to read only register %p (ignored)\n", (void *)reg_addr);
+#else   
+    PrintError("Attempting to write to read only register %p (error)\n", (void *)reg_addr);
     return -1;
+#endif
     break;
 
     // Data registers