[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] accel: Improve selection of the default acceler
From: |
Daniel P . Berrangé |
Subject: |
Re: [Qemu-devel] [PATCH] accel: Improve selection of the default accelerator |
Date: |
Tue, 9 Oct 2018 15:23:09 +0100 |
User-agent: |
Mutt/1.10.1 (2018-07-13) |
On Tue, Oct 09, 2018 at 02:58:41PM +0100, Peter Maydell wrote:
> On 9 October 2018 at 14:43, Cornelia Huck <address@hidden> wrote:
> > I'm not sure why a single accelerator (any of them) would be a good
> > default. A list (tcg:kvm:<whatever>) sounds much saner, as it would
> > continue to work even if some accelerators have been disabled (right?)
> >
> > (And I'd prefer kvm to be first in that list; anything that relies on
> > tcg being used should specify it explicitly... a normal user will
> > likely always want the fast variant.)
>
> tcg should be the default for binaries without KVM compiled in,
> of course... But as Thomas points out, the reason for our current
> default is the usual "because we tend not to change things that
> would break existing working command lines".
Putting KVM first shouldn't break existing working command lines
in general.
If user doesn't have KVM it won't have any impact as it'll still
fallback to TCG. If user does have KVM, their VMs will be faster.
It could conceivably break if
- TCG had some different behaviour to KVM that
the guest relied upon (we should fix such differences)
- KVM kmod was buggy (eg broken nested virt, again should be
something we fix)
- If the guest OS relied on things being slow (that's already
fragile)
IMHO we should assume that TCG & KVM are functionally equivalent
and non-broken, and thus changing default accelerator should not
be considered an incompatible change.
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
- Re: [Qemu-devel] [PATCH] accel: Improve selection of the default accelerator, (continued)
- Re: [Qemu-devel] [PATCH] accel: Improve selection of the default accelerator, Peter Maydell, 2018/10/05
- Re: [Qemu-devel] [PATCH] accel: Improve selection of the default accelerator, Paolo Bonzini, 2018/10/05
- Re: [Qemu-devel] [PATCH] accel: Improve selection of the default accelerator, Markus Armbruster, 2018/10/09
- Re: [Qemu-devel] [PATCH] accel: Improve selection of the default accelerator, Markus Armbruster, 2018/10/09
- Re: [Qemu-devel] [PATCH] accel: Improve selection of the default accelerator, Thomas Huth, 2018/10/09
- Re: [Qemu-devel] [PATCH] accel: Improve selection of the default accelerator, Daniel P . Berrangé, 2018/10/09
- Re: [Qemu-devel] [PATCH] accel: Improve selection of the default accelerator, Cornelia Huck, 2018/10/09
- Re: [Qemu-devel] [PATCH] accel: Improve selection of the default accelerator, Peter Maydell, 2018/10/09
- Re: [Qemu-devel] [PATCH] accel: Improve selection of the default accelerator,
Daniel P . Berrangé <=
- Re: [Qemu-devel] [PATCH] accel: Improve selection of the default accelerator, Peter Maydell, 2018/10/09
- Re: [Qemu-devel] [PATCH] accel: Improve selection of the default accelerator, Cornelia Huck, 2018/10/09
- Re: [Qemu-devel] [PATCH] accel: Improve selection of the default accelerator, Paolo Bonzini, 2018/10/09
Re: [Qemu-devel] [PATCH] accel: Improve selection of the default accelerator, Philippe Mathieu-Daudé, 2018/10/05