WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

[Xen-devel] [Patch] Fix cirrus and rt8139 co-exist issue

To: "Keir Fraser" <Keir.Fraser@xxxxxxxxxxxx>
Subject: [Xen-devel] [Patch] Fix cirrus and rt8139 co-exist issue
From: "Yang, Xiaowei" <xiaowei.yang@xxxxxxxxx>
Date: Mon, 17 Jul 2006 18:36:36 +0800
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Mon, 17 Jul 2006 03:37:08 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AcapjOFeexqJGNCER+CAAW2ITkiJLQ==
Thread-topic: [Patch] Fix cirrus and rt8139 co-exist issue
Keir,
This patch fixes cirrus and rt8139 co-exist issue in new qemu-dm. The
root cause is that if two MMIO spaces are continuous, qemu may misuse
last MMIO space's read/write to handle current request. 

Thanks,
Xiaowei

Signed-off-by: Yang Xiaowei <xiaowei.yang@xxxxxxxxx>
diff -r ecb8ff1fcf1f tools/ioemu/target-i386-dm/exec-dm.c
--- a/tools/ioemu/target-i386-dm/exec-dm.c      Fri Jul 14 17:53:27 2006
+++ b/tools/ioemu/target-i386-dm/exec-dm.c      Mon Jul 17 17:55:34 2006
@@ -382,7 +385,7 @@
                 start = mmio[i].start;
                 end = mmio[i].start + mmio[i].size;

-                if ((addr >= start) && (addr <= end)){
+                if ((addr >= start) && (addr < end)){
                         return (mmio[i].io_index >> IO_MEM_SHIFT) &
(IO_MEM_NB_ENTRIES - 1);
                 }
         }

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel

<Prev in Thread] Current Thread [Next in Thread>
  • [Xen-devel] [Patch] Fix cirrus and rt8139 co-exist issue, Yang, Xiaowei <=