From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from [140.186.70.92] (port=36379 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1P3xfD-0005rn-LZ for qemu-devel@nongnu.org; Thu, 07 Oct 2010 17:06:13 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1P3xdI-0001W1-FD for qemu-devel@nongnu.org; Thu, 07 Oct 2010 17:04:09 -0400 Received: from mail-gx0-f173.google.com ([209.85.161.173]:38210) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1P3xdI-0001Vq-D3 for qemu-devel@nongnu.org; Thu, 07 Oct 2010 17:04:08 -0400 Received: by gxk22 with SMTP id 22so168065gxk.4 for ; Thu, 07 Oct 2010 14:04:07 -0700 (PDT) Message-ID: <4CAE35C5.2010809@codemonkey.ws> Date: Thu, 07 Oct 2010 16:04:05 -0500 From: Anthony Liguori MIME-Version: 1.0 Subject: Re: [Qemu-devel] [PATCH] ceph/rbd block driver for qemu-kvm (v4) References: <20100802194631.GA4923@chb-desktop> <20100803201407.GD1475@chb-desktop> <4CADD567.9010606@codemonkey.ws> <4CAE13BA.70707@codemonkey.ws> <4CAE24C5.8030007@codemonkey.ws> In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Yehuda Sadeh Weinraub Cc: Kevin Wolf , ceph-devel@vger.kernel.org, qemu-devel@nongnu.org, kvm@vger.kernel.org, Christian Brunner On 10/07/2010 03:47 PM, Yehuda Sadeh Weinraub wrote: >> How is that possible? Are the callbacks delivered in the context of a >> different thread? If so, don't you need locking? >> > Not sure I'm completely following you. The callbacks are delivered in > the context of a different thread, but won't run concurrently. Concurrently to what? How do you prevent them from running concurrently with qemu? If you saw lock ups, I bet that's what it was from. Regards, Anthony Liguori > Do you > see any specific concurrency issue? We can add some mutex protection > around at the aio callback, so that if librados turns multithreaded at > this point we're covered. > > > Thanks, > Yehuda > >