On Thu, May 31, 2018 at 08:00:34PM -0400, John Snow wrote: > On 05/30/2018 08:43 PM, John Snow wrote: > > 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(-) > > > > Thanks for the testing and reviews, everyone! > > Thanks, applied to my IDE tree: > > https://github.com/jnsnow/qemu/commits/ide > https://github.com/jnsnow/qemu.git
Also CCing stable. This should go into QEMU 2.12.1. Stefan
signature.asc
Description: PGP signature