Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
MIME-Version: 1.0
Message-ID: <efe3a337-efe0-4d40-a50b-216406fdadc7@default>
Date:   Thu, 14 Jun 2018 16:40:12 -0700 (PDT)
From:   Liran Alon <liran.alon@oracle.com>
To:     <vkuznets@redhat.com>
Cc:     <mmorsy@redhat.com>, <rkagan@virtuozzo.com>, <jmattson@google.com>,
        <Michael.H.Kelley@microsoft.com>, <kys@microsoft.com>,
        <rkrcmar@redhat.com>, <cavery@redhat.com>, <pbonzini@redhat.com>,
        <linux-kernel@vger.kernel.org>, <sthemmin@microsoft.com>,
        <haiyangz@microsoft.com>, <kvm@vger.kernel.org>
Subject: Re: [PATCH 1/5] KVM: hyperv: define VP assist page helpers
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable
Content-Disposition: inline
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk


----- vkuznets@redhat.com wrote:

> From: Ladi Prosek <lprosek@redhat.com>
>=20
> The state related to the VP assist page is still managed by the LAPIC
> code in the pv_eoi field.
>=20
> Signed-off-by: Ladi Prosek <lprosek@redhat.com>
> Signed-off-by: Vitaly Kuznetsov <vkuznets@redhat.com>
> ---
>  arch/x86/kvm/hyperv.c | 23 +++++++++++++++++++++--
>  arch/x86/kvm/hyperv.h |  4 ++++
>  arch/x86/kvm/lapic.c  |  4 ++--
>  arch/x86/kvm/lapic.h  |  2 +-
>  arch/x86/kvm/x86.c    |  2 +-
>  5 files changed, 29 insertions(+), 6 deletions(-)
>=20
> diff --git a/arch/x86/kvm/hyperv.c b/arch/x86/kvm/hyperv.c
> index 14e0d0ae4e0a..fdf659ca6167 100644
> --- a/arch/x86/kvm/hyperv.c
> +++ b/arch/x86/kvm/hyperv.c
> @@ -688,6 +688,24 @@ void kvm_hv_vcpu_uninit(struct kvm_vcpu *vcpu)
>  =09=09stimer_cleanup(&hv_vcpu->stimer[i]);
>  }
> =20
> +bool kvm_hv_assist_page_enabled(struct kvm_vcpu *vcpu)
> +{
> +=09if (!(vcpu->arch.hyperv.hv_vapic &
> HV_X64_MSR_VP_ASSIST_PAGE_ENABLE))
> +=09=09return false;
> +=09return vcpu->arch.pv_eoi.msr_val & KVM_MSR_ENABLED;
> +}
> +EXPORT_SYMBOL_GPL(kvm_hv_assist_page_enabled);
> +
> +bool kvm_hv_get_assist_page(struct kvm_vcpu *vcpu,
> +=09=09=09    struct hv_vp_assist_page *assist_page)
> +{
> +=09if (!kvm_hv_assist_page_enabled(vcpu))
> +=09=09return false;
> +=09return !kvm_read_guest_cached(vcpu->kvm, &vcpu->arch.pv_eoi.data,
> +=09=09=09=09      assist_page, sizeof(*assist_page));
> +}
> +EXPORT_SYMBOL_GPL(kvm_hv_get_assist_page);
> +
>  static void stimer_prepare_msg(struct kvm_vcpu_hv_stimer *stimer)
>  {
>  =09struct hv_message *msg =3D &stimer->msg;
> @@ -1048,7 +1066,7 @@ static int kvm_hv_set_msr(struct kvm_vcpu *vcpu,
> u32 msr, u64 data, bool host)
> =20
>  =09=09if (!(data & HV_X64_MSR_VP_ASSIST_PAGE_ENABLE)) {
>  =09=09=09hv->hv_vapic =3D data;
> -=09=09=09if (kvm_lapic_enable_pv_eoi(vcpu, 0))
> +=09=09=09if (kvm_lapic_enable_pv_eoi(vcpu, 0, 0))
>  =09=09=09=09return 1;
>  =09=09=09break;
>  =09=09}
> @@ -1061,7 +1079,8 @@ static int kvm_hv_set_msr(struct kvm_vcpu *vcpu,
> u32 msr, u64 data, bool host)
>  =09=09hv->hv_vapic =3D data;
>  =09=09kvm_vcpu_mark_page_dirty(vcpu, gfn);
>  =09=09if (kvm_lapic_enable_pv_eoi(vcpu,
> -=09=09=09=09=09    gfn_to_gpa(gfn) | KVM_MSR_ENABLED))
> +=09=09=09=09=09    gfn_to_gpa(gfn) | KVM_MSR_ENABLED,
> +=09=09=09=09=09    sizeof(struct hv_vp_assist_page)))
>  =09=09=09return 1;
>  =09=09break;
>  =09}
> diff --git a/arch/x86/kvm/hyperv.h b/arch/x86/kvm/hyperv.h
> index 837465d69c6d..db825bb7efc7 100644
> --- a/arch/x86/kvm/hyperv.h
> +++ b/arch/x86/kvm/hyperv.h
> @@ -62,6 +62,10 @@ void kvm_hv_vcpu_init(struct kvm_vcpu *vcpu);
>  void kvm_hv_vcpu_postcreate(struct kvm_vcpu *vcpu);
>  void kvm_hv_vcpu_uninit(struct kvm_vcpu *vcpu);
> =20
> +bool kvm_hv_assist_page_enabled(struct kvm_vcpu *vcpu);
> +bool kvm_hv_get_assist_page(struct kvm_vcpu *vcpu,
> +=09=09=09    struct hv_vp_assist_page *assist_page);
> +
>  static inline struct kvm_vcpu_hv_stimer *vcpu_to_stimer(struct
> kvm_vcpu *vcpu,
>  =09=09=09=09=09=09=09int timer_index)
>  {
> diff --git a/arch/x86/kvm/lapic.c b/arch/x86/kvm/lapic.c
> index 776391cf69a5..b6d6a36f1a33 100644
> --- a/arch/x86/kvm/lapic.c
> +++ b/arch/x86/kvm/lapic.c
> @@ -2540,7 +2540,7 @@ int kvm_hv_vapic_msr_read(struct kvm_vcpu *vcpu,
> u32 reg, u64 *data)
>  =09return 0;
>  }
> =20
> -int kvm_lapic_enable_pv_eoi(struct kvm_vcpu *vcpu, u64 data)
> +int kvm_lapic_enable_pv_eoi(struct kvm_vcpu *vcpu, u64 data, unsigned
> long len)
>  {
>  =09u64 addr =3D data & ~KVM_MSR_ENABLED;
>  =09if (!IS_ALIGNED(addr, 4))
> @@ -2550,7 +2550,7 @@ int kvm_lapic_enable_pv_eoi(struct kvm_vcpu
> *vcpu, u64 data)
>  =09if (!pv_eoi_enabled(vcpu))
>  =09=09return 0;
>  =09return kvm_gfn_to_hva_cache_init(vcpu->kvm,
> &vcpu->arch.pv_eoi.data,
> -=09=09=09=09=09 addr, sizeof(u8));
> +=09=09=09=09=09 addr, len);
>  }
> =20
>  void kvm_apic_accept_events(struct kvm_vcpu *vcpu)
> diff --git a/arch/x86/kvm/lapic.h b/arch/x86/kvm/lapic.h
> index ed0ed39abd36..ff6ef9c3d760 100644
> --- a/arch/x86/kvm/lapic.h
> +++ b/arch/x86/kvm/lapic.h
> @@ -120,7 +120,7 @@ static inline bool
> kvm_hv_vapic_assist_page_enabled(struct kvm_vcpu *vcpu)
>  =09return vcpu->arch.hyperv.hv_vapic &
> HV_X64_MSR_VP_ASSIST_PAGE_ENABLE;
>  }
> =20
> -int kvm_lapic_enable_pv_eoi(struct kvm_vcpu *vcpu, u64 data);
> +int kvm_lapic_enable_pv_eoi(struct kvm_vcpu *vcpu, u64 data, unsigned
> long len);
>  void kvm_lapic_init(void);
>  void kvm_lapic_exit(void);
> =20
> diff --git a/arch/x86/kvm/x86.c b/arch/x86/kvm/x86.c
> index 06dd4cdb2ca8..a57766b940a5 100644
> --- a/arch/x86/kvm/x86.c
> +++ b/arch/x86/kvm/x86.c
> @@ -2442,7 +2442,7 @@ int kvm_set_msr_common(struct kvm_vcpu *vcpu,
> struct msr_data *msr_info)
> =20
>  =09=09break;
>  =09case MSR_KVM_PV_EOI_EN:
> -=09=09if (kvm_lapic_enable_pv_eoi(vcpu, data))
> +=09=09if (kvm_lapic_enable_pv_eoi(vcpu, data, sizeof(u8)))
>  =09=09=09return 1;
>  =09=09break;
> =20
> --=20
> 2.14.4

Reviewed-By: Liran Alon <liran.alon@oracle.com>