Re: [Qemu-devel] [PATCH 0/3] ahci: fix completion race condition

2018-05-31 Thread John Snow
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. > >

Re: [Qemu-devel] [PATCH 0/3] ahci: fix completion race condition

2018-05-31 Thread Bruce Rogers
>>> On 5/31/2018 at 7:06 AM, Philippe Mathieu-Daudé wrote: > Hi Bruce, > > On 05/31/2018 09:21 AM, Bruce Rogers wrote: > On 5/30/2018 at 6:43 PM, John Snow wrote: >>> Commit d759c951f changed the main thread lock release/reacquisition, >>> and in so doing apparently jostled loose a race cond

Re: [Qemu-devel] [PATCH 0/3] ahci: fix completion race condition

2018-05-31 Thread Philippe Mathieu-Daudé
Hi Bruce, On 05/31/2018 09:21 AM, Bruce Rogers wrote: On 5/30/2018 at 6: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

Re: [Qemu-devel] [PATCH 0/3] ahci: fix completion race condition

2018-05-31 Thread Bruce Rogers
>>> On 5/30/2018 at 6: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. >

[Qemu-devel] [PATCH 0/3] ahci: fix completion race condition

2018-05-30 Thread John Snow
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 http