[pve-devel] applied: [PATCH zfsonlinux] followup: fpu struct initialized member was removed with 5.2
Thomas Lamprecht
t.lamprecht at proxmox.com
Wed Oct 2 07:37:24 CEST 2019
That's why it was guarded with the "HAVE_KERNEL_FPU_INITIALIZED"
defined in the first place..
Signed-off-by: Thomas Lamprecht <t.lamprecht at proxmox.com>
---
...r-save-restore-is-also-required-on-5.patch | 26 ++++++++++---------
1 file changed, 14 insertions(+), 12 deletions(-)
diff --git a/debian/patches/0008-SIMD-FPU-register-save-restore-is-also-required-on-5.patch b/debian/patches/0008-SIMD-FPU-register-save-restore-is-also-required-on-5.patch
index db0c8a3..bd60e80 100644
--- a/debian/patches/0008-SIMD-FPU-register-save-restore-is-also-required-on-5.patch
+++ b/debian/patches/0008-SIMD-FPU-register-save-restore-is-also-required-on-5.patch
@@ -5,16 +5,16 @@ Subject: [PATCH] [SIMD]: FPU register save/restore is also required on 5.0
kernels
NOTE: the kernel needs to have the copy_kernel_to_xregs_err,
-copy_kernel_to_fxregs_err and copy_kernel_to_fregs_err funcitons
+copy_kernel_to_fxregs_err and copy_kernel_to_fregs_err functions
backported for this to work.
Signed-off-by: Thomas Lamprecht <t.lamprecht at proxmox.com>
---
- include/linux/simd_x86.h | 11 +----------
- 1 file changed, 1 insertion(+), 10 deletions(-)
+ include/linux/simd_x86.h | 11 ++++-------
+ 1 file changed, 4 insertions(+), 7 deletions(-)
diff --git a/include/linux/simd_x86.h b/include/linux/simd_x86.h
-index edd456098..13aa77345 100644
+index edd456098..98503a29e 100644
--- a/include/linux/simd_x86.h
+++ b/include/linux/simd_x86.h
@@ -181,7 +181,6 @@ kfpu_begin(void)
@@ -25,21 +25,23 @@ index edd456098..13aa77345 100644
/*
* The current FPU registers need to be preserved by kfpu_begin()
* and restored by kfpu_end(). This is required because we can
-@@ -190,20 +189,13 @@ kfpu_begin(void)
+@@ -190,11 +189,11 @@ kfpu_begin(void)
* context switch.
*/
copy_fpregs_to_fpstate(¤t->thread.fpu);
-#elif defined(HAVE_KERNEL_FPU_INITIALIZED)
-- /*
++
++
++#if defined(HAVE_KERNEL_FPU_INITIALIZED)
+ /*
- * There is no need to preserve and restore the FPU registers.
- * They will always be restored from the task's stored FPU state
- * when switching contexts.
-- */
-+
++ * Was removed with 5.2 as it was always set to 1 there
+ */
WARN_ON_ONCE(current->thread.fpu.initialized == 0);
--#endif
- }
-
+ #endif
+@@ -203,7 +202,6 @@ kfpu_begin(void)
static inline void
kfpu_end(void)
{
@@ -47,7 +49,7 @@ index edd456098..13aa77345 100644
union fpregs_state *state = ¤t->thread.fpu.state;
int error;
-@@ -215,7 +207,6 @@ kfpu_end(void)
+@@ -215,7 +213,6 @@ kfpu_end(void)
error = copy_kernel_to_fregs_err(&state->fsave);
}
WARN_ON_ONCE(error);
--
2.20.1
More information about the pve-devel
mailing list