Re: [PATCH libvirt] qemu: show a warning when using aio=native without cache=none

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Jul 08, 2014 at 02:08:38PM +0200, Giuseppe Scrivano wrote:
> Qemu will fallback to aio=threads when the cache mode doesn't use
> O_DIRECT, even if aio=native was explictly set.
> 
> Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1086704
> 
> Signed-off-by: Giuseppe Scrivano <gscrivan@xxxxxxxxxx>
> ---
>  src/qemu/qemu_command.c | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c
> index fb64cda..92a6c9a 100644
> --- a/src/qemu/qemu_command.c
> +++ b/src/qemu/qemu_command.c
> @@ -3477,6 +3477,13 @@ qemuBuildDriveStr(virConnectPtr conn,
>              mode = qemuDiskCacheV1TypeToString(disk->cachemode);
>          }
>  
> +        if (disk->iomode == VIR_DOMAIN_DISK_IO_NATIVE &&
> +            disk->cachemode != VIR_DOMAIN_DISK_CACHE_DIRECTSYNC) {
> +            VIR_WARN("native I/O needs either no disk cache "
> +                     "or directsync cache mode, QEMU will fallback "
> +                     "to aio=threads");
> +        }

VIR_WARN is not really appropriate for warning about things that
are client application mistakes. If this combination of options
does not work or is not supported then we should report an
error about this.

Regards,
Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]