Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752107AbdHHHjn (ORCPT ); Tue, 8 Aug 2017 03:39:43 -0400 Received: from mail-wm0-f46.google.com ([74.125.82.46]:37320 "EHLO mail-wm0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750818AbdHHHjj (ORCPT ); Tue, 8 Aug 2017 03:39:39 -0400 Date: Tue, 8 Aug 2017 09:39:31 +0200 From: Christoffer Dall To: "Longpeng(Mike)" Cc: pbonzini@redhat.com, rkrcmar@redhat.com, agraf@suse.com, borntraeger@de.ibm.com, cohuck@redhat.com, christoffer.dall@linaro.org, marc.zyngier@arm.com, james.hogan@imgtec.com, kvm@vger.kernel.org, linux-kernel@vger.kernel.org, weidong.huang@huawei.com, arei.gonglei@huawei.com, wangxinxin.wang@huawei.com, longpeng.mike@gmail.com, david@redhat.com Subject: Re: [PATCH v2 4/4] KVM: arm: implements the kvm_arch_vcpu_in_kernel() Message-ID: <20170808073931.GB26652@cbox> References: <1502165135-4784-1-git-send-email-longpeng2@huawei.com> <1502165135-4784-5-git-send-email-longpeng2@huawei.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1502165135-4784-5-git-send-email-longpeng2@huawei.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 903 Lines: 30 On Tue, Aug 08, 2017 at 12:05:35PM +0800, Longpeng(Mike) wrote: > This implements the kvm_arch_vcpu_in_kernel() for ARM. > > Signed-off-by: Longpeng(Mike) > --- > virt/kvm/arm/arm.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/virt/kvm/arm/arm.c b/virt/kvm/arm/arm.c > index 862f820..b9f68e4 100644 > --- a/virt/kvm/arm/arm.c > +++ b/virt/kvm/arm/arm.c > @@ -418,7 +418,7 @@ int kvm_arch_vcpu_runnable(struct kvm_vcpu *v) > > bool kvm_arch_vcpu_in_kernel(struct kvm_vcpu *vcpu) > { > - return false; > + return vcpu_mode_priv(vcpu); > } > > /* Just ensure a guest exit from a particular CPU */ > -- > 1.8.3.1 > > I'm not taking any position on whether this series makes sense overall or not (it's a bit concerning to do this without having measured a benefit), but for the arm/arm64 change: Acked-by: Christoffer Dall