Bug 571171

Summary: Change the block IO host implementation to aio
Product: Red Hat Enterprise Linux 7 Reporter: Dor Laor <dlaor>
Component: qemu-kvmAssignee: Kevin Wolf <kwolf>
Status: CLOSED NOTABUG QA Contact:
Severity: medium Docs Contact:
Priority: low    
Version: 7.0CC: aliguori, amit.shah, ddutile, dshaks, ehabkost, gcosta, juzhang, khong, knoel, pbonzini, perfbz, syeghiay, tburke, virt-maint
Target Milestone: beta   
Target Release: 7.0   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2013-03-15 12:05:24 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 589985, 615309, 689830    
Bug Blocks: 580954    

Description Dor Laor 2010-03-07 11:41:56 UTC
Currently we do not using configure with --enable-linux-aio and we should switch to that since Shak has performance results that state it is better than aio=threads.

Comment 1 RHEL Program Management 2010-03-07 12:13:45 UTC
This request was evaluated by Red Hat Product Management for inclusion in a Red
Hat Enterprise Linux major release.  Product Management has requested further
review of this request by Red Hat Engineering, for potential inclusion in a Red
Hat Enterprise Linux Major release.  This request is not yet committed for
inclusion.

Comment 2 chellwig@redhat.com 2010-03-07 12:45:35 UTC
Note that Linux aio support gets automatically built if we find libaio and it's headers.  So what needs to be done here it pull libaio-devel in as a build requirement.

Also note that we'll need a clone of this bug for libvirt to enable aio=native on the qemu command line,

Comment 3 Dor Laor 2010-03-07 13:26:33 UTC
(In reply to comment #2)
> Note that Linux aio support gets automatically built if we find libaio and it's
> headers.  So what needs to be done here it pull libaio-devel in as a build
> requirement.

That's a rpm dependency change

> 
> Also note that we'll need a clone of this bug for libvirt to enable aio=native
> on the qemu command line,    

Isn't the above sufficient? Alternatively we can change the default ourselves in qemu.

Comment 4 chellwig@redhat.com 2010-05-07 11:06:39 UTC
Given that this recently came up on irc, what's the status?  Do we still want to change the default aio backend, and if yes do that in libvirt or by patching qemu?

Comment 5 Dor Laor 2010-05-09 14:40:16 UTC
(In reply to comment #4)
> Given that this recently came up on irc, what's the status?  Do we still want
> to change the default aio backend, and if yes do that in libvirt or by patching
> qemu?    

I think we rather change the default in qemu, there are far too many folks try qemu directly. It's the same like cache=off in the past.

Comment 6 chellwig@redhat.com 2010-05-09 17:10:10 UTC
If we're doing this as a RHEL only patch it's easy given that both raw block device (disk/lvm) and nfs support proper AIO.  If we want to do this upstream we also need to special case regular files on various local filesystems, especially if they are thinly provisioned.

Dor, is this okay as a RHEL-only patch?

Comment 7 Dor Laor 2010-05-10 09:01:41 UTC
If you can change upstream default for our use cases it will be even better. If not, rhel-only will do too. Adding Anthony to the BZ

Comment 16 chellwig@redhat.com 2011-06-07 11:29:04 UTC
Reassigning to RHEL7, as we're unlikely to get the upstream fix in in time.

Comment 18 Paolo Bonzini 2013-03-15 12:05:24 UTC
A non-issue for RHEL, since we always go through libvirt.  If the default has to change, the change has to be in libvirt.