[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 18:50:54 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.3 (gnu/linux) |
Paolo Bonzini <address@hidden> writes:
> On 07/05/2018 13:56, Markus Armbruster wrote:
>> 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.
>
> Perhaps we can deprecate "-M accel" then, and also while we're at it
> move kernel_irqchip from -machine to "-accel kvm" where it belongs?
Sounds good to me.
- Re: [Qemu-devel] [PATCH v2 2/5] qemu-options: Bail out on unsupported options instead of silently ignoring them, (continued)
- [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, 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, 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