[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PULL 7/7] ohci: clear pending SOF on suspend
From: |
Gerd Hoffmann |
Subject: |
[Qemu-devel] [PULL 7/7] ohci: clear pending SOF on suspend |
Date: |
Fri, 8 Jan 2016 11:40:15 +0100 |
From: Laurent Vivier <address@hidden>
On overcommitted CPU, kernel can be so slow that an interrupt can
be triggered by the device whereas the driver is not ready to receive
it. This drives us into an infinite loop.
On suspend, if a SOF interrupt is raised between the stop of the
device processing and the change of the device internal state to
OHCI_USB_SUSPEND (QEMU stops SOF timer on this state change), this
interrupt is never acknowledged.
This patch clears pending SOF interrupt on OHCI_USB_SUSPEND setting.
Some details:
- ohci_irq(): the OHCI interrupt handler, acknowledges the SOF IRQ
only if the state of the driver (rh_state) is OHCI_STATE_RUNNING.
So if this interrupt happens and the driver is not in this state,
the function is called again and again, moving the system to a
CPU starvation.
- ohci_rh_suspend(): the function stop the operation and acknowledge
pending interrupts (but doesn't disable it). Later in the function,
the device is moved to OHCI_SUSPEND_STATE, and the driver to
OHCI_RH_SUSPENDED. If between the moment when the interrupt is
acknowledged and the moment when the device is suspended a new
interrupt is raised, it will be never acknowledged because the
driver is now not in OHCI_RH_RUNNING state.
Signed-off-by: Laurent Vivier <address@hidden>
Reviewed-by: Thomas Huth <address@hidden>
Message-id: address@hidden
Signed-off-by: Gerd Hoffmann <address@hidden>
---
hw/usb/hcd-ohci.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/hw/usb/hcd-ohci.c b/hw/usb/hcd-ohci.c
index ff5658e..efeaf73 100644
--- a/hw/usb/hcd-ohci.c
+++ b/hw/usb/hcd-ohci.c
@@ -1456,6 +1456,9 @@ static void ohci_set_ctl(OHCIState *ohci, uint32_t val)
break;
case OHCI_USB_SUSPEND:
ohci_bus_stop(ohci);
+ /* clear pending SF otherwise linux driver loops in ohci_irq() */
+ ohci->intr_status &= ~OHCI_INTR_SF;
+ ohci_intr_update(ohci);
break;
case OHCI_USB_RESUME:
trace_usb_ohci_resume(ohci->name);
--
1.8.3.1
- [Qemu-devel] [PULL 0/7] usb patch queue, Gerd Hoffmann, 2016/01/08
- [Qemu-devel] [PULL 2/7] ohci: fix Host Controller USBRESET, Gerd Hoffmann, 2016/01/08
- [Qemu-devel] [PULL 6/7] ohci: delay first SOF interrupt, Gerd Hoffmann, 2016/01/08
- [Qemu-devel] [PULL 5/7] usb-mtp: fix call to trace function, Gerd Hoffmann, 2016/01/08
- [Qemu-devel] [PULL 7/7] ohci: clear pending SOF on suspend,
Gerd Hoffmann <=
- [Qemu-devel] [PULL 3/7] ohci: fix command HostControllerReset, Gerd Hoffmann, 2016/01/08
- [Qemu-devel] [PULL 1/7] ohci: split reset method in 3 parts, Gerd Hoffmann, 2016/01/08
- [Qemu-devel] [PULL 4/7] usb-mtp: use safe variant when cleaning events list, Gerd Hoffmann, 2016/01/08
- Re: [Qemu-devel] [PULL 0/7] usb patch queue, Peter Maydell, 2016/01/08