From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:59759) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Zd1Uq-0004Nc-LV for qemu-devel@nongnu.org; Fri, 18 Sep 2015 15:39:01 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Zd1Up-00029W-QX for qemu-devel@nongnu.org; Fri, 18 Sep 2015 15:39:00 -0400 Received: from mx1.redhat.com ([209.132.183.28]:46513) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Zd1Up-00029O-MD for qemu-devel@nongnu.org; Fri, 18 Sep 2015 15:38:59 -0400 From: Eduardo Habkost Date: Fri, 18 Sep 2015 16:38:15 -0300 Message-Id: <1442605096-14103-4-git-send-email-ehabkost@redhat.com> In-Reply-To: <1442605096-14103-1-git-send-email-ehabkost@redhat.com> References: <1442605096-14103-1-git-send-email-ehabkost@redhat.com> Subject: [Qemu-devel] [PATCH v2 3/4] target-i386: Move TCG initialization to realize time List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: zhugh.fnst@cn.fujitsu.com, tangchen@cn.fujitsu.com, chen.fan.fnst@cn.fujitsu.com, isimatu.yasuaki@jp.fujitsu.com, Paolo Bonzini , Gu Zheng , Igor Mammedov , anshul.makkar@profitbricks.com, =?UTF-8?q?Andreas=20F=C3=A4rber?= QOM instance_init functions are not supposed to have any side-effects, as new objects may be created at any moment for querying property information (see qmp_device_list_properties()). Move TCG initialization to realize time so it won't be called when just doing object_new() on a X86CPU subclass. Signed-off-by: Eduardo Habkost --- Changes v1 -> v2: * Now the inited/tcg_initialized variable doesn't exist anymore * Move tcg_x86_init() call after basic parameter validation inside realizefn --- target-i386/cpu.c | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/target-i386/cpu.c b/target-i386/cpu.c index f054a69..2e5a303 100644 --- a/target-i386/cpu.c +++ b/target-i386/cpu.c @@ -2832,6 +2832,10 @@ static void x86_cpu_realizefn(DeviceState *dev, Error **errp) goto out; } + if (tcg_enabled()) { + tcg_x86_init(); + } + #ifndef CONFIG_USER_ONLY qemu_register_reset(x86_cpu_machine_reset_cb, cpu); @@ -3069,11 +3073,6 @@ static void x86_cpu_initfn(Object *obj) } x86_cpu_load_def(cpu, xcc->cpu_def, &error_abort); - - /* init various static tables used in TCG mode */ - if (tcg_enabled()) { - tcg_x86_init(); - } } static int64_t x86_cpu_get_arch_id(CPUState *cs) -- 2.1.0