[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 02/18] vl: move icount configuration earlier
From: |
Richard Henderson |
Subject: |
Re: [PATCH v2 02/18] vl: move icount configuration earlier |
Date: |
Wed, 11 Dec 2019 20:41:48 -0800 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.2.1 |
On 12/9/19 7:01 AM, Paolo Bonzini wrote:
> Once qemu_tcg_configure is turned into a QOM property setter, it will not
> be able to set a default value for mttcg_enabled. Setting the default will
> move to the TCG instance_init function, which currently runs before "-icount"
> is processed.
>
> However, it is harmless to do configure_icount for all accelerators; we will
> just fail later if a non-TCG accelerator is selected. So do that.
>
> Reviewed-by: Marc-André Lureau <address@hidden>
> Signed-off-by: Paolo Bonzini <address@hidden>
> ---
> vl.c | 18 +++++++++++-------
> 1 file changed, 11 insertions(+), 7 deletions(-)
Reviewed-by: Richard Henderson <address@hidden>
r~
- [PATCH v2 00/18] Complete the implementation of -accel, Paolo Bonzini, 2019/12/09
- [PATCH v2 01/18] memory: do not look at current_machine->accel, Paolo Bonzini, 2019/12/09
- [PATCH v2 02/18] vl: move icount configuration earlier, Paolo Bonzini, 2019/12/09
- Re: [PATCH v2 02/18] vl: move icount configuration earlier,
Richard Henderson <=
- [PATCH v2 04/18] vl: extract accelerator option processing to a separate function, Paolo Bonzini, 2019/12/09
- [PATCH v2 05/18] vl: merge -accel processing into configure_accelerators, Paolo Bonzini, 2019/12/09
- [PATCH v2 03/18] tcg: move qemu_tcg_configure to accel/tcg/tcg-all.c, Paolo Bonzini, 2019/12/09
- [PATCH v2 06/18] accel: compile accel/accel.c just once, Paolo Bonzini, 2019/12/09