[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-commits] [qemu/qemu] 8bb01b: ehci: Don't fetch a NULL current qtd
From: |
GitHub |
Subject: |
[Qemu-commits] [qemu/qemu] 8bb01b: ehci: Don't fetch a NULL current qtd but advance t... |
Date: |
Tue, 03 Jul 2018 10:43:09 -0700 |
Branch: refs/heads/master
Home: https://github.com/qemu/qemu
Commit: 8bb01b257f3398eae059e93bd7c8a3f5f54c5438
https://github.com/qemu/qemu/commit/8bb01b257f3398eae059e93bd7c8a3f5f54c5438
Author: Sebastian Bauer <address@hidden>
Date: 2018-07-03 (Tue, 03 Jul 2018)
Changed paths:
M hw/usb/hcd-ehci.c
Log Message:
-----------
ehci: Don't fetch a NULL current qtd but advance the queue instead.
Fetching qtd with the NULL address most likely makes no sense so from now
on, we handle it this case similarly as if the terminate (T) bit is not
set, which is already an exception as according to section 3.6 of the EHCI
spec there is no T bit defined for the current_qtd field.
The spec is a bit vague on how an EHCI driver should initialize these
fields: "The general operational model is that the host controller can
detect whether the overlay area contains a description of an active
transfer" (p. 49). QEMU primarily uses the QTD_TOKEN_ACTIVE bit of the
queue header to infer the activity state but there are other ways
conceivable.
This change allows QEMU to boot further into AmigaOS. The public available
version of the EHCI driver recycles queue heads in some rare conditions but
only clears the current_qtd field but not the status field. This works with
many available EHCI PCI cards but e.g., not with the Freescale USB
controller's found on the P5040. On the emulated EHCI controller of QEMU
the consequence is that some garbage was read in, which resulted in a
reset of the controller. This change fixes the problem.
Signed-off-by: Sebastian Bauer <address@hidden>
Tested-by: BALATON Zoltan <address@hidden>
Message-id: address@hidden
Signed-off-by: Gerd Hoffmann <address@hidden>
Commit: 8f36ec708834dfad58af6feb0b69bb0be6077f0f
https://github.com/qemu/qemu/commit/8f36ec708834dfad58af6feb0b69bb0be6077f0f
Author: Gerd Hoffmann <address@hidden>
Date: 2018-07-03 (Tue, 03 Jul 2018)
Changed paths:
M hw/usb/hcd-xhci.c
Log Message:
-----------
xhci: fix guest-triggerable assert
Set xhci into error state instead of throwing a core dump.
Signed-off-by: Gerd Hoffmann <address@hidden>
Message-id: address@hidden
Commit: ba2dfe6f6f4825d462052a38356eafee40f24a75
https://github.com/qemu/qemu/commit/ba2dfe6f6f4825d462052a38356eafee40f24a75
Author: Peter Maydell <address@hidden>
Date: 2018-07-03 (Tue, 03 Jul 2018)
Changed paths:
M hw/usb/hcd-ehci.c
M hw/usb/hcd-xhci.c
Log Message:
-----------
Merge remote-tracking branch 'remotes/kraxel/tags/usb-20180703-pull-request'
into staging
usb: bugfixes for ehci and xhci.
# gpg: Signature made Tue 03 Jul 2018 09:26:20 BST
# gpg: using RSA key 4CB6D8EED3E87138
# gpg: Good signature from "Gerd Hoffmann (work) <address@hidden>"
# gpg: aka "Gerd Hoffmann <address@hidden>"
# gpg: aka "Gerd Hoffmann (private) <address@hidden>"
# Primary key fingerprint: A032 8CFF B93A 17A7 9901 FE7D 4CB6 D8EE D3E8 7138
* remotes/kraxel/tags/usb-20180703-pull-request:
xhci: fix guest-triggerable assert
ehci: Don't fetch a NULL current qtd but advance the queue instead.
Signed-off-by: Peter Maydell <address@hidden>
Compare: https://github.com/qemu/qemu/compare/f988c7e19114...ba2dfe6f6f48
**NOTE:** This service been marked for deprecation:
https://developer.github.com/changes/2018-04-25-github-services-deprecation/
Functionality will be removed from GitHub.com on January 31st, 2019.
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- [Qemu-commits] [qemu/qemu] 8bb01b: ehci: Don't fetch a NULL current qtd but advance t...,
GitHub <=