[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] rbd block driver fix race between aio completit
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH] rbd block driver fix race between aio completition and aio cancel |
Date: |
Wed, 21 Nov 2012 16:34:25 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:16.0) Gecko/20121016 Thunderbird/16.0.1 |
Il 21/11/2012 10:19, Stefan Priebe - Profihost AG ha scritto:
> Hello Stefan,
> hello Paolo,
>
> most of the ideas and removing the whole cancellation stuff came from
> Paolo. Maybe he can comment also?
I agree with all of Stefan's comments. This includes putting back
acb->cancelled---but this time done right: note that
qemu_rbd_complete_aio was always releasing the AIOCB
if (acb->cancelled) {
qemu_vfree(acb->bounce);
qemu_aio_release(acb);
goto done;
}
and according to Stefan's review it shouldn't.
Paolo