From mboxrd@z Thu Jan 1 00:00:00 1970 From: Boris Ostrovsky Subject: [PATCH v5 RESEND 14/17] x86/VPMU: Save VPMU state for PV guests during context switch Date: Wed, 23 Apr 2014 08:50:35 -0400 Message-ID: <1398257438-4994-15-git-send-email-boris.ostrovsky@oracle.com> References: <1398257438-4994-1-git-send-email-boris.ostrovsky@oracle.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1398257438-4994-1-git-send-email-boris.ostrovsky@oracle.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: kevin.tian@intel.com Cc: keir@xen.org, jun.nakajima@intel.com, andrew.cooper3@citrix.com, eddie.dong@intel.com, donald.d.dugger@intel.com, xen-devel@lists.xen.org, dietmar.hahn@ts.fujitsu.com, JBeulich@suse.com, boris.ostrovsky@oracle.com, suravee.suthikulpanit@amd.com List-Id: xen-devel@lists.xenproject.org Save VPMU state during context switch for both HVM and PV guests unless we are in PMU privileged mode (i.e. dom0 is doing all profiling). Signed-off-by: Boris Ostrovsky --- xen/arch/x86/domain.c | 12 +++++------- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/xen/arch/x86/domain.c b/xen/arch/x86/domain.c index b389abc..87c3a02 100644 --- a/xen/arch/x86/domain.c +++ b/xen/arch/x86/domain.c @@ -1461,16 +1461,14 @@ void context_switch(struct vcpu *prev, struct vcpu *next) } if ( prev != next ) - _update_runstate_area(prev); - - if ( is_hvm_vcpu(prev) ) { - if ( (prev != next) && (vpmu_mode & XENPMU_MODE_ON) ) + _update_runstate_area(prev); + if ( vpmu_mode & XENPMU_MODE_ON ) vpmu_save(prev); + } - if ( !list_empty(&prev->arch.hvm_vcpu.tm_list) ) + if ( is_hvm_vcpu(prev) && !list_empty(&prev->arch.hvm_vcpu.tm_list) ) pt_save_timer(prev); - } local_irq_disable(); @@ -1508,7 +1506,7 @@ void context_switch(struct vcpu *prev, struct vcpu *next) (next->domain->domain_id != 0)); } - if ( is_hvm_vcpu(next) && (prev != next) && (vpmu_mode & XENPMU_MODE_ON) ) + if ( (prev != next) && (vpmu_mode & XENPMU_MODE_ON) ) /* Must be done with interrupts enabled */ vpmu_load(next); -- 1.8.3.1