Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756677AbdDMKLy (ORCPT ); Thu, 13 Apr 2017 06:11:54 -0400 Received: from mx2.suse.de ([195.135.220.15]:49911 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756626AbdDMKLm (ORCPT ); Thu, 13 Apr 2017 06:11:42 -0400 From: Juergen Gross To: linux-kernel@vger.kernel.org, xen-devel@lists.xenproject.org Cc: boris.ostrovsky@oracle.com, Juergen Gross Subject: [PATCH v2 06/11] x86/xen: use capabilities instead of fake cpuid values for acpi Date: Thu, 13 Apr 2017 12:11:11 +0200 Message-Id: <20170413101116.723-7-jgross@suse.com> X-Mailer: git-send-email 2.12.0 In-Reply-To: <20170413101116.723-1-jgross@suse.com> References: <20170413101116.723-1-jgross@suse.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2289 Lines: 72 When running as pv domain xen_cpuid() is being used instead of native_cpuid(). In xen_cpuid() the acpi feature is indicated as not being present by special casing the related cpuid leaf in case we are not the initial domain. Instead of delivering fake cpuid values clear the cpu capability bit for acpi instead. Signed-off-by: Juergen Gross --- arch/x86/xen/enlighten_pv.c | 11 +++-------- 1 file changed, 3 insertions(+), 8 deletions(-) diff --git a/arch/x86/xen/enlighten_pv.c b/arch/x86/xen/enlighten_pv.c index 53fe97bd720f..3196961862bc 100644 --- a/arch/x86/xen/enlighten_pv.c +++ b/arch/x86/xen/enlighten_pv.c @@ -165,7 +165,6 @@ xen_running_on_version_or_later(unsigned int major, unsigned int minor) return false; } -static __read_mostly unsigned int cpuid_leaf1_edx_mask = ~0; static __read_mostly unsigned int cpuid_leaf1_ecx_mask = ~0; static __read_mostly unsigned int cpuid_leaf1_ecx_set_mask; @@ -177,7 +176,6 @@ static void xen_cpuid(unsigned int *ax, unsigned int *bx, { unsigned maskebx = ~0; unsigned maskecx = ~0; - unsigned maskedx = ~0; unsigned setecx = 0; /* * Mask out inconvenient features, to try and disable as many @@ -187,7 +185,6 @@ static void xen_cpuid(unsigned int *ax, unsigned int *bx, case 1: maskecx = cpuid_leaf1_ecx_mask; setecx = cpuid_leaf1_ecx_set_mask; - maskedx = cpuid_leaf1_edx_mask; break; case CPUID_MWAIT_LEAF: @@ -214,7 +211,6 @@ static void xen_cpuid(unsigned int *ax, unsigned int *bx, *bx &= maskebx; *cx &= maskecx; *cx |= setecx; - *dx &= maskedx; } STACK_FRAME_NON_STANDARD(xen_cpuid); /* XEN_EMULATE_PREFIX */ @@ -294,10 +290,6 @@ static void __init xen_init_cpuid_mask(void) unsigned int ax, bx, cx, dx; unsigned int xsave_mask; - if (!xen_initial_domain()) - cpuid_leaf1_edx_mask &= - ~((1 << X86_FEATURE_ACPI)); /* disable ACPI */ - cpuid_leaf1_ecx_mask &= ~(1 << (X86_FEATURE_X2APIC % 32)); ax = 1; @@ -323,6 +315,9 @@ static void __init xen_init_capabilities(void) setup_clear_cpu_cap(X86_FEATURE_APERFMPERF); setup_clear_cpu_cap(X86_FEATURE_MTRR); setup_clear_cpu_cap(X86_FEATURE_ACC); + + if (!xen_initial_domain()) + setup_clear_cpu_cap(X86_FEATURE_ACPI); } static void xen_set_debugreg(int reg, unsigned long val) -- 2.12.0