diff options
author | Quentin Casasnovas <quentin.casasnovas@oracle.com> | 2015-03-05 15:19:22 +0300 |
---|---|---|
committer | Ingo Molnar <mingo@kernel.org> | 2015-03-05 20:20:36 +0300 |
commit | 06c8173eb92bbfc03a0fe8bb64315857d0badd06 (patch) | |
tree | 73ecc10d3017c9ae7a6403fca3afbf3d7ec12997 /arch/avr32/Makefile | |
parent | 9ab6eb51ef4ad63cb71533d3a4dfb09ea8f69b4c (diff) | |
download | linux-06c8173eb92bbfc03a0fe8bb64315857d0badd06.tar.xz |
x86/fpu/xsaves: Fix improper uses of __ex_table
Commit:
f31a9f7c7169 ("x86/xsaves: Use xsaves/xrstors to save and restore xsave area")
introduced alternative instructions for XSAVES/XRSTORS and commit:
adb9d526e982 ("x86/xsaves: Add xsaves and xrstors support for booting time")
added support for the XSAVES/XRSTORS instructions at boot time.
Unfortunately both failed to properly protect them against faulting:
The 'xstate_fault' macro will use the closest label named '1'
backward and that ends up in the .altinstr_replacement section
rather than in .text. This means that the kernel will never find
in the __ex_table the .text address where this instruction might
fault, leading to serious problems if userspace manages to
trigger the fault.
Signed-off-by: Quentin Casasnovas <quentin.casasnovas@oracle.com>
Signed-off-by: Jamie Iles <jamie.iles@oracle.com>
[ Improved the changelog, fixed some whitespace noise. ]
Acked-by: Borislav Petkov <bp@alien8.de>
Acked-by: Linus Torvalds <torvalds@linux-foundation.org>
Cc: <stable@vger.kernel.org>
Cc: Allan Xavier <mr.a.xavier@gmail.com>
Cc: H. Peter Anvin <hpa@zytor.com>
Cc: Thomas Gleixner <tglx@linutronix.de>
Fixes: adb9d526e982 ("x86/xsaves: Add xsaves and xrstors support for booting time")
Fixes: f31a9f7c7169 ("x86/xsaves: Use xsaves/xrstors to save and restore xsave area")
Signed-off-by: Ingo Molnar <mingo@kernel.org>
Diffstat (limited to 'arch/avr32/Makefile')
0 files changed, 0 insertions, 0 deletions