From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:38428) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Ztcj6-00014e-Qk for qemu-devel@nongnu.org; Tue, 03 Nov 2015 09:38:21 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Ztcj5-0008A2-Vm for qemu-devel@nongnu.org; Tue, 03 Nov 2015 09:38:20 -0500 References: <1446559933-28965-1-git-send-email-den@openvz.org> <1446559933-28965-5-git-send-email-den@openvz.org> From: "Denis V. Lunev" Message-ID: <5638C6CD.9050104@openvz.org> Date: Tue, 3 Nov 2015 17:38:05 +0300 MIME-Version: 1.0 In-Reply-To: <1446559933-28965-5-git-send-email-den@openvz.org> Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH 04/10] blockdev: acquire AioContext in hmp_commit() List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Jeff Cody , qemu-devel@nongnu.org, Stefan Hajnoczi , qemu-stable@nongnu.org On 11/03/2015 05:12 PM, Denis V. Lunev wrote: > From: Stefan Hajnoczi > > This one slipped through. Although we acquire AioContext when > committing all devices we don't for just a single device. > > AioContext must be acquired before calling bdrv_*() functions to > synchronize access with other threads that may be using the AioContext. > > Signed-off-by: Stefan Hajnoczi > Signed-off-by: Denis V. Lunev this was Reviewed-by: Jeff Cody in the original submission. Lost that accidentally.