ahci: fix completion race condition
Status from this message: Thanks, applied to my IDE tree:
Get series with b4
b4 am [email protected]Versions newest first
Patches in v1 Applied
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/3 | ahci: trim signatures on raise/lowerhw/ | +4 | 3 replies | |
| 2/3 | ahci: fix PxCI register racehw/ | +6 | 2 replies | |
| 3/3 | ahci: don't schedule unnecessary BHhw/ | +1 | 1 reply |
Reviews trailers per patch
All 3 patches have a Reviewed-by or Acked-by of their own; the series has one for all of them.
| Patch | Stefan Hajnoczi | Jeff Cody | Bruce Rogers |
|---|---|---|---|
| Series | R-b | ||
| 1/3 | R-b | ||
| 2/3 | R-b | T-b | |
| 3/3 | R-b |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes. Scroll the table sideways for all 3 reviewers.
Cover letter [Qemu-devel] [PATCH 0/3] ahci: fix completion race condition
Commit d759c951f changed the main thread lock release/reacquisition, and in so doing apparently jostled loose a race condition in the AHCI code.
Patch 2 should be sufficient to fix this, and patches 1 and 3 are just little trivial fixes.
This might be sufficient to fix the bug as reported at
https://bugs.launchpad.net/qemu/+bug/1769189
but the nature of the timing changes make it difficult to confirm, so I am posting this patchset for the reporters to help test.
John Snow (3):
ahci: trim signatures on raise/lower
ahci: fix PxCI register race
ahci: don't schedule unnecessary BH
hw/ide/ahci.c | 24 +++++++++++------------- 1 file changed, 11 insertions(+), 13 deletions(-)
Signature · 1 line
2.14.3