[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 0/5] Removal of deprecated -no-kvm* options
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH v2 0/5] Removal of deprecated -no-kvm* options |
Date: |
Mon, 07 May 2018 13:56:56 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.3 (gnu/linux) |
Paolo Bonzini <address@hidden> writes:
> On 07/05/2018 09:56, Markus Armbruster wrote:
>>>>> I'm not that sure anymore about -no-kvm. It can come in handy for
>>>>> distros (*cough* RHEL *cough) that only ship a qemu-kvm binary with
>>>>> default accelerator "-machine accel=kvm:tcg".
>>>>>
>>>>> Yeah, "-accel tcg" is only a few characters later, but it is a
>>>>> relatively recent addition.
>>>>
>>>> 2012 is "relatively recent" in the alternate RHEL universe, perhaps :)
>>>
>>> That was the "-machine accel=tcg", but Paolo was rather talking about
>>> "-accel tcg", which has been added last year. Yes, we've got three (!)
>>> ways of dealing with accelerators now: "-M accel=", "-accel" and
>>> "-enable-kvm + -no-kvm". We're very userfriendly, aren't we? ;-)
>>
>> Ugh. Apparently, vl.c still is a playground without adult supervision.
>
> I'm not sure I get it... "-accel" was added in order to provide
> accelerator options. If anything, "-machine accel" (added back in 2010)
> is the one that makes little sense these days.
Adding more and more ways to do the same stuff does not improve an
interface. Interface design needs to be *opinionated*. If we decide
-machine accel=tcg isn't a nice interface, by all means create a better
one, but as replacement[*], not as addition.
Furthermore:
tcg: add options for enabling MTTCG
We know there will be cases where MTTCG won't work until additional work
is done in the front/back ends to support. It will however be useful to
be able to turn it on.
As a result MTTCG will default to off unless the combination is
supported. However the user can turn it on for the sake of testing.
Signed-off-by: KONRAD Frederic <address@hidden>
[AJB: move to -accel tcg,thread=multi|single, defaults]
Signed-off-by: Alex Bennée <address@hidden>
Reviewed-by: Richard Henderson <address@hidden>
I'm sorry, but this us sub-par. Yes, the commit is also about "enabling
MTTCG", but it also adds a new way to select accelerators, without ever
spelling that out. It should've been split, and properly described.
[*] Replacements generally involve deprecation and a grace period.
- [Qemu-devel] [PATCH v2 2/5] qemu-options: Bail out on unsupported options instead of silently ignoring them, (continued)
- [Qemu-devel] [PATCH v2 2/5] qemu-options: Bail out on unsupported options instead of silently ignoring them, Thomas Huth, 2018/05/04
- [Qemu-devel] [PATCH v2 5/5] qemu-options: Remove deprecated -no-kvm, Thomas Huth, 2018/05/04
- Re: [Qemu-devel] [PATCH v2 0/5] Removal of deprecated -no-kvm* options, Paolo Bonzini, 2018/05/04
- Re: [Qemu-devel] [PATCH v2 0/5] Removal of deprecated -no-kvm* options, Thomas Huth, 2018/05/07
- Re: [Qemu-devel] [PATCH v2 0/5] Removal of deprecated -no-kvm* options, Markus Armbruster, 2018/05/07
- Re: [Qemu-devel] [PATCH v2 0/5] Removal of deprecated -no-kvm* options, Thomas Huth, 2018/05/07
- Re: [Qemu-devel] [PATCH v2 0/5] Removal of deprecated -no-kvm* options, Markus Armbruster, 2018/05/07
- Re: [Qemu-devel] [PATCH v2 0/5] Removal of deprecated -no-kvm* options, Paolo Bonzini, 2018/05/07
- Re: [Qemu-devel] [PATCH v2 0/5] Removal of deprecated -no-kvm* options,
Markus Armbruster <=
- Re: [Qemu-devel] [PATCH v2 0/5] Removal of deprecated -no-kvm* options, Paolo Bonzini, 2018/05/07
- Re: [Qemu-devel] [PATCH v2 0/5] Removal of deprecated -no-kvm* options, Markus Armbruster, 2018/05/07
- Re: [Qemu-devel] [PATCH v2 0/5] Removal of deprecated -no-kvm* options, Paolo Bonzini, 2018/05/07
- Re: [Qemu-devel] [PATCH v2 0/5] Removal of deprecated -no-kvm* options, Thomas Huth, 2018/05/16
- Re: [Qemu-devel] [PATCH v2 0/5] Removal of deprecated -no-kvm* options, Paolo Bonzini, 2018/05/07
- Re: [Qemu-devel] [PATCH v2 0/5] Removal of deprecated -no-kvm* options, Markus Armbruster, 2018/05/07
Re: [Qemu-devel] [PATCH v2 0/5] Removal of deprecated -no-kvm* options, Paolo Bonzini, 2018/05/04