From mboxrd@z Thu Jan 1 00:00:00 1970 From: Joerg Roedel Subject: Re: [PATCH V5 2/4] KVM: x86/vPMU: Create vPMU interface for VMX and SVM Date: Tue, 16 Jun 2015 17:40:32 +0200 Message-ID: <20150616154031.GP20384@8bytes.org> References: <1434087296-27141-1-git-send-email-wei@redhat.com> <1434087296-27141-3-git-send-email-wei@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: kvm@vger.kernel.org, x86@kernel.org, pbonzini@redhat.com, gleb@kernel.org, rkrcmar@redhat.com To: Wei Huang Return-path: Received: from 8bytes.org ([81.169.241.247]:40808 "EHLO theia.8bytes.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751187AbbFPPkd (ORCPT ); Tue, 16 Jun 2015 11:40:33 -0400 Content-Disposition: inline In-Reply-To: <1434087296-27141-3-git-send-email-wei@redhat.com> Sender: kvm-owner@vger.kernel.org List-ID: On Fri, Jun 12, 2015 at 01:34:54AM -0400, Wei Huang wrote: > This patch splits existing vPMU code into a common vPMU interface (pm= c.c) > and Intel specific vPMU code (pmu_intel.c) using the following steps: >=20 > - Part of arechitectural vPMU code is extracted and moved to pmu_inte= l.c > file. They are hooked up with the newly-defined intel_pmu_ops, whic= h will > be called from pmu interface; > - Create a dummy pmu_amd.c file for AMD SVM with empty functions; >=20 > All architectural vPMU functions are now called via PMU function disp= atcher. > This function dispatcher is defined in kvm_x86_ops->pmu_ops, which is > initialized by sub-arch. Also note that Intel and AMD modules are now > generated by combinig their corresponding arch files (vmx.c/svm.c) an= d pmu > files (pmu_intel.c/pmu_amd.c). >=20 > Tested-by: Radim Kr=C4=8Dm=C3=A1=C5=99 > Signed-off-by: Wei Huang Reviewed-by: Joerg Roedel