56753ff
From 4c245e5ecbc7d5c30c8e8bb4bfcd18c79fafddfe Mon Sep 17 00:00:00 2001
56753ff
From: Hans de Goede <hdegoede@redhat.com>
56753ff
Date: Tue, 3 Apr 2012 14:04:31 +0200
56753ff
Subject: [PATCH 147/181] usb-ehci: Ensure frindex writes leave a valid
56753ff
 frindex value
56753ff
56753ff
frindex is a 14 bits counter, so bits 31-14 should always be 0, and
56753ff
after the commit titled "usb-ehci: frindex always is a 14 bits counter"
56753ff
we rely on frindex always being a multiple of 8. I've not seen this in
56753ff
practice, but theoretically a guest can write a value >= 0x4000 or a value
56753ff
which is not a multiple of 8 value to frindex, this patch ensures that
56753ff
things will still work when that happens.
56753ff
56753ff
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
56753ff
---
56753ff
 hw/usb-ehci.c |    4 ++++
56753ff
 1 file changed, 4 insertions(+)
56753ff
56753ff
diff --git a/hw/usb-ehci.c b/hw/usb-ehci.c
56753ff
index ff69587..16e6053 100644
56753ff
--- a/hw/usb-ehci.c
56753ff
+++ b/hw/usb-ehci.c
56753ff
@@ -1081,6 +1081,10 @@ static void ehci_mem_writel(void *ptr, target_phys_addr_t addr, uint32_t val)
56753ff
         val &= USBINTR_MASK;
56753ff
         break;
56753ff
 
56753ff
+    case FRINDEX:
56753ff
+        val &= 0x00003ff8; /* frindex is 14bits and always a multiple of 8 */
56753ff
+        break;
56753ff
+
56753ff
     case CONFIGFLAG:
56753ff
         val &= 0x1;
56753ff
         if (val) {
56753ff
-- 
56753ff
1.7.10
56753ff