Received: by 2002:a05:6a10:206:0:0:0:0 with SMTP id 6csp4275258pxj; Wed, 12 May 2021 01:48:39 -0700 (PDT) X-Google-Smtp-Source: ABdhPJw5F1CZ4eXZpAmTcLR/dFr1Q9O85Imh20Fc3DbLtiTdLLsktLgu3NVuNeXsbQML03hWR8MO X-Received: by 2002:a02:a30e:: with SMTP id q14mr31091037jai.4.1620809318985; Wed, 12 May 2021 01:48:38 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1620809318; cv=none; d=google.com; s=arc-20160816; b=BE9VgwShR7eH+LVPA9IXMqb5IdmyGrpJiC/8OWlSlg+NzE+K7IKW8qBVVGOUSS2oSF nUZ4GDBfXjrYcAT1IFyxlcSy26hL1rOfOEopalUTazjsFMzlUdKek0oaUbG3UQwIgK9f LATLw6fKirnNT9lP7C1TYOdAdQSAtUjzK8nmHnCI2qcRlxI3pPuiwhSJ+h78wVgTqFgk izToVjR3W1kDma5EY4DjJh2tYlnrdJ2jUjddR9DqnfCKzBPF1/c9yj0bnBszTRF3SA7a Vz+ciDyZiOsOYYBBD+EgTOIONzynXgXWyYueEaGrxudtidHYQftcHxwDbAsgm5XKBn7P Pnww== 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=QlqOHuaqLe2h0ArmEsAMZA4k9JM5ghG7WZFgQ7stykM=; b=VCKgooMiaRe8fhIOm/t9LeOW+HrL6fuvFlHZxNu2si9rP9hBKzSKGCTozjFMmJFB2x 6zU6ezUO9NJyZ0JcR8DpUa5wtIyZ1ZbaNrGqK/HVrrtqsqoDaA1G3hAeY05s1znMPTpv CrECEjzL2yphAUzOrHM9ucJdvLdBsq8+YTj/FtbZ7rfNPzEmRqJUU8PRQJ6oh7j0tYg2 uyYCpFoiKfMt1deh3Mm9sbCxVvKYgni0xVomCR3kVgBS9Kh7WyoTaDq3e+iC+pNw+Tg+ TwCMjc3qWA1gy/VckMpgr5UWzfujPEE0RGZOBI99kxGhpzwvSTac+kBKej1hHauOO4zY KkpQ== 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 q13si21095175ilj.21.2021.05.12.01.48.26; Wed, 12 May 2021 01:48:38 -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 S231154AbhELIrL (ORCPT + 99 others); Wed, 12 May 2021 04:47:11 -0400 Received: from mga17.intel.com ([192.55.52.151]:10038 "EHLO mga17.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230181AbhELIqp (ORCPT ); Wed, 12 May 2021 04:46:45 -0400 IronPort-SDR: uwsVPVwyvcvwOLJGY9MxY6Y4dzPRHJ4Ir3Tuykuw/OPToSYFSdeaSGh5QiEb2myW5yDbcIUQmL fXEOFpmwngWw== X-IronPort-AV: E=McAfee;i="6200,9189,9981"; a="179918834" X-IronPort-AV: E=Sophos;i="5.82,293,1613462400"; d="scan'208";a="179918834" Received: from orsmga006.jf.intel.com ([10.7.209.51]) by fmsmga107.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 12 May 2021 01:45:36 -0700 IronPort-SDR: urEe+zPi+pjlkkhP5Mkd9dSdduCvsW1Erwnna7vaqztMAccftphu8Z8dajWXrgNxP8biPocIza OMCwmTphwOmA== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.82,293,1613462400"; d="scan'208";a="392636360" Received: from clx-ap-likexu.sh.intel.com ([10.239.48.108]) by orsmga006.jf.intel.com with ESMTP; 12 May 2021 01:45:33 -0700 From: Like Xu To: Paolo Bonzini , peterz@infradead.org Cc: Sean Christopherson , Vitaly Kuznetsov , Wanpeng Li , Jim Mattson , Joerg Roedel , weijiang.yang@intel.com, eranian@google.com, wei.w.wang@intel.com, kvm@vger.kernel.org, x86@kernel.org, linux-kernel@vger.kernel.org, Luwei Kang Subject: [PATCH v3 2/5] KVM: x86/pmu: Add the base address parameter for get_fixed_pmc() Date: Wed, 12 May 2021 16:44:43 +0800 Message-Id: <20210512084446.342526-3-like.xu@linux.intel.com> X-Mailer: git-send-email 2.31.1 In-Reply-To: <20210512084446.342526-1-like.xu@linux.intel.com> References: <20210512084446.342526-1-like.xu@linux.intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Luwei Kang Introduce a new base MSR address pass-in parameter for get_fixed_pmc() so that the caller can define its base for fixed counters (such as MSR_RELOAD_FIXED_CTRx that can be used in the PBES-via-PT feature). Refactor the code using the current value MSR_CORE_PERF_FIXED_CTR0. Signed-off-by: Luwei Kang --- The checkpatch.pl yell "ERROR: do not use assignment in if condition", should I fix the orignal coding style ? arch/x86/kvm/pmu.h | 5 ++--- arch/x86/kvm/vmx/pmu_intel.c | 17 +++++++++++------ 2 files changed, 13 insertions(+), 9 deletions(-) diff --git a/arch/x86/kvm/pmu.h b/arch/x86/kvm/pmu.h index 832cf56e6924..6720881b8370 100644 --- a/arch/x86/kvm/pmu.h +++ b/arch/x86/kvm/pmu.h @@ -126,10 +126,9 @@ static inline struct kvm_pmc *get_gp_pmc(struct kvm_pmu *pmu, u32 msr, } /* returns fixed PMC with the specified MSR */ -static inline struct kvm_pmc *get_fixed_pmc(struct kvm_pmu *pmu, u32 msr) +static inline struct kvm_pmc *get_fixed_pmc(struct kvm_pmu *pmu, + u32 msr, u32 base) { - int base = MSR_CORE_PERF_FIXED_CTR0; - if (msr >= base && msr < base + pmu->nr_arch_fixed_counters) { u32 index = array_index_nospec(msr - base, pmu->nr_arch_fixed_counters); diff --git a/arch/x86/kvm/vmx/pmu_intel.c b/arch/x86/kvm/vmx/pmu_intel.c index a706d3597720..c10cb3008bf1 100644 --- a/arch/x86/kvm/vmx/pmu_intel.c +++ b/arch/x86/kvm/vmx/pmu_intel.c @@ -44,7 +44,8 @@ static void reprogram_fixed_counters(struct kvm_pmu *pmu, u64 data) u8 old_ctrl = fixed_ctrl_field(pmu->fixed_ctr_ctrl, i); struct kvm_pmc *pmc; - pmc = get_fixed_pmc(pmu, MSR_CORE_PERF_FIXED_CTR0 + i); + pmc = get_fixed_pmc(pmu, MSR_CORE_PERF_FIXED_CTR0 + i, + MSR_CORE_PERF_FIXED_CTR0); if (old_ctrl == new_ctrl) continue; @@ -114,7 +115,8 @@ static struct kvm_pmc *intel_pmc_idx_to_pmc(struct kvm_pmu *pmu, int pmc_idx) else { u32 idx = pmc_idx - INTEL_PMC_IDX_FIXED; - return get_fixed_pmc(pmu, idx + MSR_CORE_PERF_FIXED_CTR0); + return get_fixed_pmc(pmu, idx + MSR_CORE_PERF_FIXED_CTR0, + MSR_CORE_PERF_FIXED_CTR0); } } @@ -222,7 +224,8 @@ static bool intel_is_valid_msr(struct kvm_vcpu *vcpu, u32 msr) default: ret = get_gp_pmc(pmu, msr, MSR_IA32_PERFCTR0) || get_gp_pmc(pmu, msr, MSR_P6_EVNTSEL0) || - get_fixed_pmc(pmu, msr) || get_fw_gp_pmc(pmu, msr) || + get_fixed_pmc(pmu, msr, MSR_CORE_PERF_FIXED_CTR0) || + get_fw_gp_pmc(pmu, msr) || intel_pmu_is_valid_lbr_msr(vcpu, msr); break; } @@ -235,7 +238,7 @@ static struct kvm_pmc *intel_msr_idx_to_pmc(struct kvm_vcpu *vcpu, u32 msr) struct kvm_pmu *pmu = vcpu_to_pmu(vcpu); struct kvm_pmc *pmc; - pmc = get_fixed_pmc(pmu, msr); + pmc = get_fixed_pmc(pmu, msr, MSR_CORE_PERF_FIXED_CTR0); pmc = pmc ? pmc : get_gp_pmc(pmu, msr, MSR_P6_EVNTSEL0); pmc = pmc ? pmc : get_gp_pmc(pmu, msr, MSR_IA32_PERFCTR0); @@ -382,7 +385,8 @@ static int intel_pmu_get_msr(struct kvm_vcpu *vcpu, struct msr_data *msr_info) msr_info->data = val & pmu->counter_bitmask[KVM_PMC_GP]; return 0; - } else if ((pmc = get_fixed_pmc(pmu, msr))) { + } else if ((pmc = get_fixed_pmc(pmu, msr, + MSR_CORE_PERF_FIXED_CTR0))) { u64 val = pmc_read_counter(pmc); msr_info->data = val & pmu->counter_bitmask[KVM_PMC_FIXED]; @@ -470,7 +474,8 @@ static int intel_pmu_set_msr(struct kvm_vcpu *vcpu, struct msr_data *msr_info) perf_event_period(pmc->perf_event, get_sample_period(pmc, data)); return 0; - } else if ((pmc = get_fixed_pmc(pmu, msr))) { + } else if ((pmc = get_fixed_pmc(pmu, msr, + MSR_CORE_PERF_FIXED_CTR0))) { pmc->counter += data - pmc_read_counter(pmc); if (pmc->perf_event) perf_event_period(pmc->perf_event, -- 2.31.1