From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
	id S1751908AbcF2QpZ (ORCPT <rfc822;w@1wt.eu>);
	Wed, 29 Jun 2016 12:45:25 -0400
Received: from mga11.intel.com ([192.55.52.93]:48601 "EHLO mga11.intel.com"
	rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP
	id S1751460AbcF2QpY convert rfc822-to-8bit (ORCPT
	<rfc822;linux-kernel@vger.kernel.org>);
	Wed, 29 Jun 2016 12:45:24 -0400
X-ExtLoop1: 1
X-IronPort-AV: E=Sophos;i="5.26,547,1459839600"; 
   d="scan'208";a="1012154145"
Date: Wed, 29 Jun 2016 09:38:40 -0700
From: yunhong jiang <yunhong.jiang@linux.intel.com>
To: Wanpeng Li <kernellwp@gmail.com>
Cc: linux-kernel@vger.kernel.org, kvm@vger.kernel.org,
        Wanpeng Li <wanpeng.li@hotmail.com>,
        Paolo Bonzini <pbonzini@redhat.com>,
        Radim =?UTF-8?B?S3LEjW3DocWZ?= <rkrcmar@redhat.com>,
        Yunhong Jiang <yunhong.jiang@intel.com>
Subject: Re: [PATCH v2 1/2] KVM: x86: introduce cancel_hv_tscdeadline
Message-ID: <20160629093840.69814be3@jnakajim-build>
In-Reply-To: <1467199437-3102-1-git-send-email-wanpeng.li@hotmail.com>
References: <1467199437-3102-1-git-send-email-wanpeng.li@hotmail.com>
Organization: otc
X-Mailer: Claws Mail 3.11.1 (GTK+ 2.24.28; x86_64-redhat-linux-gnu)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8BIT
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Wed, 29 Jun 2016 19:23:56 +0800
Wanpeng Li <kernellwp@gmail.com> wrote:

> From: Wanpeng Li <wanpeng.li@hotmail.com>
> 
> Introduce cancel_hv_tscdeadline() to encapsulate preemption 
> timer cancel stuff.
> 
> Cc: Paolo Bonzini <pbonzini@redhat.com>
> Cc: Radim Krčmář <rkrcmar@redhat.com>
> Cc: Yunhong Jiang <yunhong.jiang@intel.com>
> Signed-off-by: Wanpeng Li <wanpeng.li@hotmail.com>

It's ok for me, thanks for the patch, Wanpeng.

--jyh

> ---
>  arch/x86/kvm/lapic.c | 18 ++++++++++--------
>  1 file changed, 10 insertions(+), 8 deletions(-)
> 
> diff --git a/arch/x86/kvm/lapic.c b/arch/x86/kvm/lapic.c
> index fdc05ae..9c20ac1 100644
> --- a/arch/x86/kvm/lapic.c
> +++ b/arch/x86/kvm/lapic.c
> @@ -1349,14 +1349,19 @@ bool kvm_lapic_hv_timer_in_use(struct
> kvm_vcpu *vcpu) }
>  EXPORT_SYMBOL_GPL(kvm_lapic_hv_timer_in_use);
>  
> +static void cancel_hv_tscdeadline(struct kvm_lapic *apic)
> +{
> +	kvm_x86_ops->cancel_hv_timer(apic->vcpu);
> +	apic->lapic_timer.hv_timer_in_use = false;
> +}
> +
>  void kvm_lapic_expired_hv_timer(struct kvm_vcpu *vcpu)
>  {
>  	struct kvm_lapic *apic = vcpu->arch.apic;
>  
>  	WARN_ON(!apic->lapic_timer.hv_timer_in_use);
>  	WARN_ON(swait_active(&vcpu->wq));
> -	kvm_x86_ops->cancel_hv_timer(vcpu);
> -	apic->lapic_timer.hv_timer_in_use = false;
> +	cancel_hv_tscdeadline(apic);
>  	apic_timer_expired(apic);
>  }
>  EXPORT_SYMBOL_GPL(kvm_lapic_expired_hv_timer);
> @@ -1376,10 +1381,8 @@ void kvm_lapic_switch_to_hv_timer(struct
> kvm_vcpu *vcpu) hrtimer_cancel(&apic->lapic_timer.timer);
>  
>  			/* In case the sw timer triggered in the
> window */
> -			if (atomic_read(&apic->lapic_timer.pending))
> {
> -				apic->lapic_timer.hv_timer_in_use =
> false;
> -
> kvm_x86_ops->cancel_hv_timer(apic->vcpu);
> -			}
> +			if (atomic_read(&apic->lapic_timer.pending))
> +				cancel_hv_tscdeadline(apic);
>  		}
>  		trace_kvm_hv_timer_state(vcpu->vcpu_id,
>  				apic->lapic_timer.hv_timer_in_use);
> @@ -1395,8 +1398,7 @@ void kvm_lapic_switch_to_sw_timer(struct
> kvm_vcpu *vcpu) if (!apic->lapic_timer.hv_timer_in_use)
>  		return;
>  
> -	kvm_x86_ops->cancel_hv_timer(vcpu);
> -	apic->lapic_timer.hv_timer_in_use = false;
> +	cancel_hv_tscdeadline(apic);
>  
>  	if (atomic_read(&apic->lapic_timer.pending))
>  		return;