[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] Replace '-machine accel=xyz' with '-accel xyz'
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH] Replace '-machine accel=xyz' with '-accel xyz' |
Date: |
Mon, 23 Sep 2019 13:52:16 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.8.0 |
On 23/09/19 13:30, Markus Armbruster wrote:
> -accel is yet another convenience option. We have so many of them. I
> dislike the complexity they add to the CLI. Here's how this one got in:
>
> commit 8d4e9146b3568022ea5730d92841345d41275d66
> Author: KONRAD Frederic <address@hidden>
> Date: Thu Feb 23 18:29:08 2017 +0000
>
> 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>
>
> Not a peep on why the existing options are so insufficient we must have
> another one.
>
> Our CLI will remain the steaming mess it has become until we reform the
> habits that got us there.
-accel's accel suboption is currently defined as a convenience option,
but it shouldn't be. It's the older "-M accel=foo:bar" that should
become "-accel foo -accel bar" and -accel then is the preferred way.
The existing option "-M accel" was insufficient because it didn't allow
accelerator-specific suboptions; they were all over the place ("-machine
kernel_irqchip", "-tb-size", etc.) and indeed mostly in wrong places.
Paolo