From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from [140.186.70.92] (port=43680 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1OCuhT-00083e-DJ for qemu-devel@nongnu.org; Fri, 14 May 2010 09:13:12 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.69) (envelope-from ) id 1OCuhR-00017z-Dl for qemu-devel@nongnu.org; Fri, 14 May 2010 09:13:11 -0400 Received: from mx1.redhat.com ([209.132.183.28]:53743) by eggs.gnu.org with esmtp (Exim 4.69) (envelope-from ) id 1OCuhR-00017p-4n for qemu-devel@nongnu.org; Fri, 14 May 2010 09:13:09 -0400 Message-ID: <4BED4C44.7090504@redhat.com> Date: Fri, 14 May 2010 15:12:36 +0200 From: Kevin Wolf MIME-Version: 1.0 References: <1273707388-5213-1-git-send-email-agraf@suse.de> <1273707388-5213-2-git-send-email-agraf@suse.de> In-Reply-To: <1273707388-5213-2-git-send-email-agraf@suse.de> Content-Type: text/plain; charset=ISO-8859-15 Content-Transfer-Encoding: 7bit Subject: [Qemu-devel] Re: [PATCH 1/2] Add no-op aio emulation stub List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Alexander Graf Cc: qemu-devel@nongnu.org, hch@lst.de Am 13.05.2010 01:36, schrieb Alexander Graf: > We need to be able to do nothing in AIO fashion. Since I suspect this > could be useful for more cases than the non flushing, I figured I'd > create a new function that does everything AIO-like, but doesn't do > anything. > > Signed-off-by: Alexander Graf This isn't bisectable: cc1: warnings being treated as errors block.c:2208: error: 'bdrv_aio_noop_em' defined but not used Maybe put everything in one patch, it's small enough now. Also, in qemu-options.hx you could explain what volatile actually means. Below the line that you change there are some paragraphs that explain the behaviour for each value. Other than that the series looks good to me. I'm not sure if Anthony is already convinced, though. Everyone else seems to agree with the intention of the series. Kevin