Commit b226fe95 authored Dec 10, 2018 by Talons Lee Committed by Greg Kroah-Hartman Feb 27, 2019

always clear the X2APIC_ENABLE bit for PV guest



[ Upstream commit 5268c8f39e0efef81af2aaed160272d9eb507beb ]

Commit e657fccb clears cpu capability bit instead of using fake cpuid
value, the EXTD should always be off for PV guest without depending
on cpuid value. So remove the cpuid check in xen_read_msr_safe() to
always clear the X2APIC_ENABLE bit.

Signed-off-by: Talons Lee <xin.li@citrix.com>
Reviewed-by: Juergen Gross <jgross@suse.com>
Signed-off-by: Boris Ostrovsky <boris.ostrovsky@oracle.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>

parent 88c55679

arch/x86/xen/enlighten_pv.c

+1 −4

Original line number	Diff line number	Diff line
		@@ -900,9 +900,6 @@ static u64 xen_read_msr_safe(unsigned int msr, int *err)
		val = native_read_msr_safe(msr, err);
		switch (msr) {
		case MSR_IA32_APICBASE:
		#ifdef CONFIG_X86_X2APIC
		if (!(cpuid_ecx(1) & (1 << (X86_FEATURE_X2APIC & 31))))
		#endif
		val &= ~X2APIC_ENABLE;
		break;
		}