Source-Changes-HG archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

[src/trunk]: src/sys/arch/x86/x86 x86/fpu: Align savefpu to 64 bytes in fpuin...



details:   https://anonhg.NetBSD.org/src/rev/b1c711b8b67b
branches:  trunk
changeset: 373815:b1c711b8b67b
user:      riastradh <riastradh%NetBSD.org@localhost>
date:      Fri Mar 03 14:40:16 2023 +0000

description:
x86/fpu: Align savefpu to 64 bytes in fpuinit_mxcsr_mask.

16 bytes is not enough.

(Is this why it never worked on Xen some years back?  Got lucky and
accidentally had 64-byte alignment on native x86, but not in the call
stack in Xen?)

XXX pullup-10

diffstat:

 sys/arch/x86/x86/fpu.c |  6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diffs (27 lines):

diff -r c23c760d5b2a -r b1c711b8b67b sys/arch/x86/x86/fpu.c
--- a/sys/arch/x86/x86/fpu.c    Fri Mar 03 14:40:00 2023 +0000
+++ b/sys/arch/x86/x86/fpu.c    Fri Mar 03 14:40:16 2023 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: fpu.c,v 1.85 2023/03/03 14:32:48 riastradh Exp $       */
+/*     $NetBSD: fpu.c,v 1.86 2023/03/03 14:40:16 riastradh Exp $       */
 
 /*
  * Copyright (c) 2008, 2019 The NetBSD Foundation, Inc.  All
@@ -96,7 +96,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: fpu.c,v 1.85 2023/03/03 14:32:48 riastradh Exp $");
+__KERNEL_RCSID(0, "$NetBSD: fpu.c,v 1.86 2023/03/03 14:40:16 riastradh Exp $");
 
 #include "opt_multiprocessor.h"
 
@@ -188,7 +188,7 @@
 fpuinit_mxcsr_mask(void)
 {
 #ifndef XENPV
-       union savefpu fpusave __aligned(16);
+       union savefpu fpusave __aligned(64);
        u_long psl;
 
        memset(&fpusave, 0, sizeof(fpusave));



Home | Main Index | Thread Index | Old Index