From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:43136) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bF5ZB-0005h9-Jk for qemu-devel@nongnu.org; Mon, 20 Jun 2016 16:13:10 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bF5Z4-000353-EA for qemu-devel@nongnu.org; Mon, 20 Jun 2016 16:13:04 -0400 Received: from mx1.redhat.com ([209.132.183.28]:36419) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bF5Z4-00034s-8o for qemu-devel@nongnu.org; Mon, 20 Jun 2016 16:12:58 -0400 From: Eduardo Habkost Date: Mon, 20 Jun 2016 17:12:43 -0300 Message-Id: <1466453564-7572-3-git-send-email-ehabkost@redhat.com> In-Reply-To: <1466453564-7572-1-git-send-email-ehabkost@redhat.com> References: <1466453564-7572-1-git-send-email-ehabkost@redhat.com> Subject: [Qemu-devel] [PATCH 2/3] target-i386: Introduce x86_cpu_load_host_data() function List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: Jiri Denemark , dahi@linux.vnet.ibm.com, libvir-list@redhat.com, Igor Mammedov The code that loads host-specific information inside x86_cpu_realizefn() will be reused by the implementation of query-host-cpu, so move it to a separate function. Signed-off-by: Eduardo Habkost --- target-i386/cpu.c | 23 ++++++++++++++++------- 1 file changed, 16 insertions(+), 7 deletions(-) diff --git a/target-i386/cpu.c b/target-i386/cpu.c index aadd0b9..3d3635d 100644 --- a/target-i386/cpu.c +++ b/target-i386/cpu.c @@ -1491,6 +1491,20 @@ void x86_cpu_change_kvm_default(const char *prop, const char *value) static uint32_t x86_cpu_get_supported_feature_word(FeatureWord w, bool migratable_only); +/* Load host-dependent CPU information, when applicable */ +static void x86_cpu_load_host_data(X86CPU *cpu) +{ + CPUX86State *env = &cpu->env; + FeatureWord w; + + if (cpu->host_features) { + for (w = 0; w < FEATURE_WORDS; w++) { + env->features[w] = + x86_cpu_get_supported_feature_word(w, cpu->migratable); + } + } +} + #ifdef CONFIG_KVM static int cpu_x86_fill_model_id(char *str) @@ -3012,18 +3026,13 @@ static void x86_cpu_realizefn(DeviceState *dev, Error **errp) return; } + x86_cpu_load_host_data(cpu); + /*TODO: cpu->host_features incorrectly overwrites features * set using "feat=on|off". Once we fix this, we can convert * plus_features & minus_features to global properties * inside x86_cpu_parse_featurestr() too. */ - if (cpu->host_features) { - for (w = 0; w < FEATURE_WORDS; w++) { - env->features[w] = - x86_cpu_get_supported_feature_word(w, cpu->migratable); - } - } - for (w = 0; w < FEATURE_WORDS; w++) { cpu->env.features[w] |= plus_features[w]; cpu->env.features[w] &= ~minus_features[w]; -- 2.5.5