Received: by 2002:a05:6a10:22f:0:0:0:0 with SMTP id 15csp577620pxk; Thu, 24 Sep 2020 12:45:32 -0700 (PDT) X-Google-Smtp-Source: ABdhPJwyVDcRbNxQR1aNBHacwItf6S+oQDV0LX9qElxtx4khktl8wwikYr0yao1W2Dny3yfoAJHz X-Received: by 2002:aa7:d747:: with SMTP id a7mr419858eds.304.1600976732537; Thu, 24 Sep 2020 12:45:32 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1600976732; cv=none; d=google.com; s=arc-20160816; b=YSVdUHPqfqOXxRuNbSPqoPqogENeBRLXXv+MjlexLRc+yd236GCDCIhx9pA+33eC5i xl24xj98zkDlEgcP4R1+bUm3umrgmFmBmmnz9y6NjoMsqypkhrVRuEKL3qNtl1iJ/5Mo 8XOUexVKU9IzqSDjmVCBCeK82KtvxPGS2B7atMccT1tRvN9SNVoLUMB6Dxqw0spdPAVQ TU+6FIUScfC0lT8zWs8whunCDfTbIct3yghb0hmLSkOZEqw9BLqjzXuKIihTD9flsqOp 7ISLa5lwOVT+4/TuSfFvGSbw64s8E6lXAKl9ytiRGlobJEYB/t0hlHo+j5/tBebl9Pul Y6xQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from :ironport-sdr:ironport-sdr; bh=TPxqxxMmfRvXXA46+pbscuuDF7VI1NZs1oMmpFY12xA=; b=RqppLRHjJvnQXsxMUh9+CevIZctn8/i7Ml1N4XbTUb++fYysOYnP8xGzlXuO7V8m5q /ehqaVCADmscELG85FR9ld4sjxpvLJKgcYlnRNdOImAsWg3tdgEdgFDR2SUhkZsrap1y uG3k0I2Qp/ekIcpEahbDKUPJkHpTwNJGfSAspOfRPpI9MysAvURaqwNKM8eNwrGTdKSz 4Dg9edpFQ1IyA3wYyBOx823J9Hfjol7BdtD4wAR+i4tl06u651GmYNCHpKqNa+PkhCLx JyR747kMqO9ZJF2W9EAr6VYQjukkGammSx1CgZS1VD/RCVPGqIG0ZkeS1NiWG4hbsvhQ 3UDg== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=intel.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id e3si267439edr.606.2020.09.24.12.45.09; Thu, 24 Sep 2020 12:45:32 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=intel.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726200AbgIXTmz (ORCPT + 99 others); Thu, 24 Sep 2020 15:42:55 -0400 Received: from mga18.intel.com ([134.134.136.126]:31793 "EHLO mga18.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725208AbgIXTmz (ORCPT ); Thu, 24 Sep 2020 15:42:55 -0400 IronPort-SDR: zLJCHE2dhFIe648NZH7wi5uvD7Ci/ZSZKiXFKBDDB0lz3Vz71W9Tsoqq8oRVQpWQt6bHtZ4cbY J177IJYOOQnw== X-IronPort-AV: E=McAfee;i="6000,8403,9754"; a="149076387" X-IronPort-AV: E=Sophos;i="5.77,299,1596524400"; d="scan'208";a="149076387" X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga003.fm.intel.com ([10.253.24.29]) by orsmga106.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 24 Sep 2020 12:42:52 -0700 IronPort-SDR: dsuICRaW7RQs8ixQFzR01Icla57G8r4Ce37vrZK30G2Ghw6iTDvDUFeX/G1xSzG9IMvlTeZKyn dEYtQEt/7ZOA== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.77,299,1596524400"; d="scan'208";a="347953045" Received: from sjchrist-coffee.jf.intel.com ([10.54.74.160]) by FMSMGA003.fm.intel.com with ESMTP; 24 Sep 2020 12:42:52 -0700 From: Sean Christopherson To: Paolo Bonzini Cc: Sean Christopherson , Vitaly Kuznetsov , Wanpeng Li , Jim Mattson , Joerg Roedel , kvm@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH v3 1/5] KVM: VMX: Use precomputed MAXPHYADDR for RTIT base MSR check Date: Thu, 24 Sep 2020 12:42:46 -0700 Message-Id: <20200924194250.19137-2-sean.j.christopherson@intel.com> X-Mailer: git-send-email 2.28.0 In-Reply-To: <20200924194250.19137-1-sean.j.christopherson@intel.com> References: <20200924194250.19137-1-sean.j.christopherson@intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Use cpuid_maxphyaddr() instead of cpuid_query_maxphyaddr() for the RTIT base MSR check. There is no reason to recompute MAXPHYADDR as the precomputed version is synchronized with CPUID updates, and MSR_IA32_RTIT_OUTPUT_BASE is not written between stuffing CPUID and refreshing vcpu->arch.maxphyaddr. Signed-off-by: Sean Christopherson --- arch/x86/kvm/vmx/vmx.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/x86/kvm/vmx/vmx.c b/arch/x86/kvm/vmx/vmx.c index 6f9a0c6d5dc5..be82da055fc4 100644 --- a/arch/x86/kvm/vmx/vmx.c +++ b/arch/x86/kvm/vmx/vmx.c @@ -147,7 +147,7 @@ module_param_named(preemption_timer, enable_preemption_timer, bool, S_IRUGO); RTIT_STATUS_BYTECNT)) #define MSR_IA32_RTIT_OUTPUT_BASE_MASK \ - (~((1UL << cpuid_query_maxphyaddr(vcpu)) - 1) | 0x7f) + (~((1UL << cpuid_maxphyaddr(vcpu)) - 1) | 0x7f) /* * These 2 parameters are used to config the controls for Pause-Loop Exiting: -- 2.28.0