From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:59658) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZZJGB-0001ap-KL for qemu-devel@nongnu.org; Tue, 08 Sep 2015 09:48:32 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZZJG8-00036e-91 for qemu-devel@nongnu.org; Tue, 08 Sep 2015 09:48:31 -0400 Date: Tue, 8 Sep 2015 14:48:25 +0100 From: Stefan Hajnoczi Message-ID: <20150908134825.GA19999@stefanha-thinkpad.redhat.com> References: <1437407656-26726-1-git-send-email-stefanha@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1437407656-26726-1-git-send-email-stefanha@redhat.com> Subject: Re: [Qemu-devel] [Qemu-block] [PATCH] virtio-blk: use blk_io_plug/unplug for Linux AIO batching List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Stefan Hajnoczi Cc: atheurer@redhat.com, Kevin Wolf , qemu-block@nongnu.org, psuriset@redhat.com, qemu-devel@nongnu.org, Ming Lei , Paolo Bonzini On Mon, Jul 20, 2015 at 04:54:16PM +0100, Stefan Hajnoczi wrote: > The raw-posix block driver implements Linux AIO batching so multiple > requests can be submitted with a single io_submit(2) system call. > Batching is currently only used by virtio-scsi and > virtio-blk-data-plane. > > Enable batching for regular virtio-blk so the number of io_submit(2) > system calls is reduced for workloads with queue depth > 1. > > In 4KB random read performance tests with queue depth 32, the CPU > utilization on the host is reduced by 9.4%. The fio job is as follows: > > [global] > bs=4k > ioengine=libaio > iodepth=32 > direct=1 > sync=0 > time_based=1 > runtime=30 > clocksource=gettimeofday > ramp_time=5 > > [job1] > rw=randread > filename=/dev/vdb > size=4096M > write_bw_log=fio > write_iops_log=fio > write_lat_log=fio > log_avg_msec=1000 > > This benchmark was run on an raw image on LVM. The disk was an SSD > drive and -drive cache=none,aio=native was used. > > Tested-by: Pradeep Surisetty > Signed-off-by: Stefan Hajnoczi > --- > hw/block/virtio-blk.c | 4 ++++ > 1 file changed, 4 insertions(+) Thanks, applied to my master tree: https://github.com/stefanha/qemu/commits/master Stefan