]> www.infradead.org Git - users/dwmw2/qemu.git/commit
Revert "ehci: don't flush cache on doorbell rings."
authorHans de Goede <hdegoede@redhat.com>
Wed, 29 Aug 2012 08:12:52 +0000 (10:12 +0200)
committerMichael Roth <mdroth@linux.vnet.ibm.com>
Fri, 12 Oct 2012 02:44:13 +0000 (21:44 -0500)
commita37d5e521a948dc582c66e52e84a0348a8c11de9
treec60324e1ba2949490c8e95746bc85483dbd2159c
parentd6e508d3a5dacacf5b93daaf9eb4bc35e01a124d
Revert "ehci: don't flush cache on doorbell rings."

This reverts commit 9bc3a3a216e2689bfcdd36c3e079333bbdbf3ba0, which got
added to fix an issue where the real, underlying cause was not stopping
the ep queue on an error.

Now that the underlying cause is fixed by the "usb: Halt ep queue and
cancel pending packets on a packet error" patch, the "don't flush" fix
is no longer needed.

Not only is it not needed, it causes us to see cancellations (unlinks)
done by the Linux EHCI driver too late, which in combination with the new
usb-core packet-id generation where qtd addresses are used as ids, causes
duplicate ids for in flight packets.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
(cherry picked from commit 66f092d25697e11847b61d761c38ddebedaed8d1)

Signed-off-by: Michael Roth <mdroth@linux.vnet.ibm.com>
hw/usb/hcd-ehci.c