This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCH v2 23/28] arm64/sve: KVM: Treat guest SVE use as undefined instruction execution
- From: Dave Martin <Dave dot Martin at arm dot com>
- To: Alex Bennée <alex dot bennee at linaro dot org>
- Cc: linux-arch at vger dot kernel dot org, libc-alpha at sourceware dot org, Ard Biesheuvel <ard dot biesheuvel at linaro dot org>, Szabolcs Nagy <szabolcs dot nagy at arm dot com>, Catalin Marinas <catalin dot marinas at arm dot com>, Will Deacon <will dot deacon at arm dot com>, Marc Zyngier <marc dot zyngier at arm dot com>, Christoffer Dall <christoffer dot dall at linaro dot org>, Richard Sandiford <richard dot sandiford at arm dot com>, kvmarm at lists dot cs dot columbia dot edu, linux-arm-kernel at lists dot infradead dot org
- Date: Fri, 29 Sep 2017 14:00:30 +0100
- Subject: Re: [PATCH v2 23/28] arm64/sve: KVM: Treat guest SVE use as undefined instruction execution
- Authentication-results: sourceware.org; auth=none
- References: <1504198860-12951-1-git-send-email-Dave.Martin@arm.com> <1504198860-12951-24-git-send-email-Dave.Martin@arm.com> <874ls59zku.fsf@linaro.org>
On Thu, Sep 14, 2017 at 02:31:13PM +0100, Alex Bennée wrote:
>
> Dave Martin <Dave.Martin@arm.com> writes:
>
> > When trapping forbidden attempts by a guest to use SVE, we want the
> > guest to see a trap consistent with SVE not being implemented.
> >
> > This patch injects an undefined instruction exception into the
> > guest in response to such an exception.
>
> I do wonder if this should be merged with the previous trap enabling
> patch though?
Yes, that would make sense now I look at it.
Can I keep your Reviewed-by on the combined patch?
Cheers
---Dave
>
> >
> > Signed-off-by: Dave Martin <Dave.Martin@arm.com>
> > ---
> > arch/arm64/kvm/handle_exit.c | 8 ++++++++
> > 1 file changed, 8 insertions(+)
> >
> > diff --git a/arch/arm64/kvm/handle_exit.c b/arch/arm64/kvm/handle_exit.c
> > index 17d8a16..e3e42d0 100644
> > --- a/arch/arm64/kvm/handle_exit.c
> > +++ b/arch/arm64/kvm/handle_exit.c
> > @@ -147,6 +147,13 @@ static int kvm_handle_unknown_ec(struct kvm_vcpu *vcpu, struct kvm_run *run)
> > return 1;
> > }
> >
> > +static int handle_sve(struct kvm_vcpu *vcpu, struct kvm_run *run)
> > +{
> > + /* Until SVE is supported for guests: */
> > + kvm_inject_undefined(vcpu);
> > + return 1;
> > +}
> > +
> > static exit_handle_fn arm_exit_handlers[] = {
> > [0 ... ESR_ELx_EC_MAX] = kvm_handle_unknown_ec,
> > [ESR_ELx_EC_WFx] = kvm_handle_wfx,
> > @@ -160,6 +167,7 @@ static exit_handle_fn arm_exit_handlers[] = {
> > [ESR_ELx_EC_HVC64] = handle_hvc,
> > [ESR_ELx_EC_SMC64] = handle_smc,
> > [ESR_ELx_EC_SYS64] = kvm_handle_sys_reg,
> > + [ESR_ELx_EC_SVE] = handle_sve,
> > [ESR_ELx_EC_IABT_LOW] = kvm_handle_guest_abort,
> > [ESR_ELx_EC_DABT_LOW] = kvm_handle_guest_abort,
> > [ESR_ELx_EC_SOFTSTP_LOW]= kvm_handle_guest_debug,
>
>
> --
> Alex Bennée
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel