On 23/3/25 19:43, Richard Henderson wrote:
On 3/21/25 08:59, Philippe Mathieu-Daudé wrote:
Use the OnOffAuto type as 3-state.
Since the TCGState instance is zero-initialized, the
mttcg_enabled is initialzed as AUTO (ON_OFF_AUTO_AUTO).
In tcg_init_machine(), if mttcg_enabled is still AUTO,
set a default value (effectively inlining the
default_mttcg_enabled() method content).
Instead of emiting a warning when the 'thread' property
is set in tcg_set_thread(), emit it in tcg_init_machine()
where it is consumed.
In the tcg_get_thread() getter, consider AUTO / OFF states
as "single", otherwise ON is "multi".
Signed-off-by: Philippe Mathieu-Daudé <phi...@linaro.org>
I think it would be better to leave the warning where it is for this patch.
Move the warning when required by other changes.
Doh, I forgot to address this comment. Do you mean, defer moving the
warning to the next patch, where we remove the TARGET_SUPPORTS_MTTCG
definition?
Reviewed-by: Richard Henderson <richard.hender...@linaro.org>
r~
