]> git.ipfire.org Git - thirdparty/kernel/stable-queue.git/commitdiff
delete queue-4.9/kvm-pkeys-do-not-use-pkru-value-in-vcpu-arch.guest_fpu.state.patch
authorGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sun, 27 Aug 2017 10:37:36 +0000 (12:37 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sun, 27 Aug 2017 10:37:36 +0000 (12:37 +0200)
queue-4.9/kvm-pkeys-do-not-use-pkru-value-in-vcpu-arch.guest_fpu.state.patch [deleted file]
queue-4.9/series

diff --git a/queue-4.9/kvm-pkeys-do-not-use-pkru-value-in-vcpu-arch.guest_fpu.state.patch b/queue-4.9/kvm-pkeys-do-not-use-pkru-value-in-vcpu-arch.guest_fpu.state.patch
deleted file mode 100644 (file)
index bfee50c..0000000
+++ /dev/null
@@ -1,93 +0,0 @@
-From 38cfd5e3df9c4f88e76b547eee2087ee5c042ae2 Mon Sep 17 00:00:00 2001
-From: Paolo Bonzini <pbonzini@redhat.com>
-Date: Wed, 23 Aug 2017 23:16:29 +0200
-Subject: KVM, pkeys: do not use PKRU value in vcpu->arch.guest_fpu.state
-
-From: Paolo Bonzini <pbonzini@redhat.com>
-
-commit 38cfd5e3df9c4f88e76b547eee2087ee5c042ae2 upstream.
-
-The host pkru is restored right after vcpu exit (commit 1be0e61), so
-KVM_GET_XSAVE will return the host PKRU value instead.  Fix this by
-using the guest PKRU explicitly in fill_xsave and load_xsave.  This
-part is based on a patch by Junkang Fu.
-
-The host PKRU data may also not match the value in vcpu->arch.guest_fpu.state,
-because it could have been changed by userspace since the last time
-it was saved, so skip loading it in kvm_load_guest_fpu.
-
-Reported-by: Junkang Fu <junkang.fjk@alibaba-inc.com>
-Cc: Yang Zhang <zy107165@alibaba-inc.com>
-Fixes: 1be0e61c1f255faaeab04a390e00c8b9b9042870
-Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
-Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
----
- arch/x86/include/asm/fpu/internal.h |    6 +++---
- arch/x86/kvm/x86.c                  |   17 ++++++++++++++---
- 2 files changed, 17 insertions(+), 6 deletions(-)
-
---- a/arch/x86/include/asm/fpu/internal.h
-+++ b/arch/x86/include/asm/fpu/internal.h
-@@ -445,10 +445,10 @@ static inline int copy_fpregs_to_fpstate
-       return 0;
- }
--static inline void __copy_kernel_to_fpregs(union fpregs_state *fpstate)
-+static inline void __copy_kernel_to_fpregs(union fpregs_state *fpstate, u64 mask)
- {
-       if (use_xsave()) {
--              copy_kernel_to_xregs(&fpstate->xsave, -1);
-+              copy_kernel_to_xregs(&fpstate->xsave, mask);
-       } else {
-               if (use_fxsr())
-                       copy_kernel_to_fxregs(&fpstate->fxsave);
-@@ -472,7 +472,7 @@ static inline void copy_kernel_to_fpregs
-                       : : [addr] "m" (fpstate));
-       }
--      __copy_kernel_to_fpregs(fpstate);
-+      __copy_kernel_to_fpregs(fpstate, -1);
- }
- extern int copy_fpstate_to_sigframe(void __user *buf, void __user *fp, int size);
---- a/arch/x86/kvm/x86.c
-+++ b/arch/x86/kvm/x86.c
-@@ -3179,7 +3179,12 @@ static void fill_xsave(u8 *dest, struct
-                       u32 size, offset, ecx, edx;
-                       cpuid_count(XSTATE_CPUID, index,
-                                   &size, &offset, &ecx, &edx);
--                      memcpy(dest + offset, src, size);
-+                      if (feature == XFEATURE_MASK_PKRU)
-+                              memcpy(dest + offset, &vcpu->arch.pkru,
-+                                     sizeof(vcpu->arch.pkru));
-+                      else
-+                              memcpy(dest + offset, src, size);
-+
-               }
-               valid -= feature;
-@@ -3217,7 +3222,11 @@ static void load_xsave(struct kvm_vcpu *
-                       u32 size, offset, ecx, edx;
-                       cpuid_count(XSTATE_CPUID, index,
-                                   &size, &offset, &ecx, &edx);
--                      memcpy(dest, src + offset, size);
-+                      if (feature == XFEATURE_MASK_PKRU)
-+                              memcpy(&vcpu->arch.pkru, src + offset,
-+                                     sizeof(vcpu->arch.pkru));
-+                      else
-+                              memcpy(dest, src + offset, size);
-               }
-               valid -= feature;
-@@ -7453,7 +7462,9 @@ void kvm_load_guest_fpu(struct kvm_vcpu
-        */
-       vcpu->guest_fpu_loaded = 1;
-       __kernel_fpu_begin();
--      __copy_kernel_to_fpregs(&vcpu->arch.guest_fpu.state);
-+      /* PKRU is separately restored in kvm_x86_ops->run.  */
-+      __copy_kernel_to_fpregs(&vcpu->arch.guest_fpu.state,
-+                              ~XFEATURE_MASK_PKRU);
-       trace_kvm_fpu(1);
- }
index 91ae90ae424f96b7f3908a640851b8379ed541de..bde31e8d311668b374006a8c17a92108b654ed8f 100644 (file)
@@ -29,7 +29,6 @@ input-elan_i2c-add-elan0602-acpi-id-to-support-lenovo-yoga310.patch
 input-alps-fix-two-finger-scroll-breakage-in-right-side-on-alps-touchpad.patch
 kvm-s390-sthyi-fix-sthyi-inline-assembly.patch
 kvm-s390-sthyi-fix-specification-exception-detection.patch
-kvm-pkeys-do-not-use-pkru-value-in-vcpu-arch.guest_fpu.state.patch
 kvm-x86-block-guest-protection-keys-unless-the-host-has-them-enabled.patch
 alsa-usb-audio-add-delay-quirk-for-h650e-jabra-550a-usb-headsets.patch
 alsa-core-fix-unexpected-error-at-replacing-user-tlv.patch