]> git.baikalelectronics.ru Git - kernel.git/commit
x86: fix i486 suspend to disk CR4 oops
authorDavid Fries <david@fries.net>
Mon, 18 Aug 2008 04:03:40 +0000 (23:03 -0500)
committerIngo Molnar <mingo@elte.hu>
Mon, 18 Aug 2008 06:50:19 +0000 (08:50 +0200)
commitdc4b63dbe7b10862017128bb39a772c4bb4ffc84
tree762fa0f0c22dc1af7d9d6e918f4d48ced8910a4e
parentc3b8d11b2eebd0197c1347e1add0295a6581fca2
x86: fix i486 suspend to disk CR4 oops

arch/x86/power/cpu_32.c __save_processor_state calls read_cr4()
only a i486 CPU doesn't have the CR4 register.  Trying to read it
produces an invalid opcode oops during suspend to disk.

Use the safe rc4 reading op instead. If the value to be written is
zero the write is skipped.

arch/x86/power/hibernate_asm_32.S
done: swapped the use of %eax and %ecx to use jecxz for
the zero test and jump over store to %cr4.
restore_image: s/%ecx/%eax/ to be consistent with done:

In addition to __save_processor_state, acpi_save_state_mem,
efi_call_phys_prelog, and efi_call_phys_epilog had checks added
(acpi restore was in assembly and already had a check for
non-zero).  There were other reads and writes of CR4, but MCE and
virtualization shouldn't be executed on a i486 anyway.

Signed-off-by: David Fries <david@fries.net>
Acked-by: H. Peter Anvin <hpa@zytor.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
arch/x86/kernel/acpi/sleep.c
arch/x86/kernel/efi_32.c
arch/x86/power/cpu_32.c
arch/x86/power/hibernate_asm_32.S