aboutsummaryrefslogtreecommitdiff
path: root/arch/x86_64/kernel/k8.c
diff options
context:
space:
mode:
authorTian Kevin <kevin.tian@intel.com>2007-06-23 17:16:52 -0700
committerLinus Torvalds <torvalds@woody.linux-foundation.org>2007-06-24 08:59:12 -0700
commitc8cbee61c9d53ee28473ad33bbb54f6a88f5e3af (patch)
treead0b767e80ef169ad319860274482ae0bbf8ab4c /arch/x86_64/kernel/k8.c
parente5d2861f31474b373ce7754dc5122b414a176c64 (diff)
ACPI: preserve the ebx value in acpi_copy_wakeup_routine
Register %ebx serves as the "global offset table base register" for position-independent code. For absolute code, %ebx serves as a local register and has no specified role in the function calling sequence. In either case, a function must preserve the register value for the caller. acpi_copy_wakeup_routine overrides %ebx without saving it, this may corrupt the called data. Kevin found that most time the value of Sx is saved in %esi, however sometimes compiler also uses %ebx. When this happens, suspends fails since sleep value in ebx is changed by acpi_copy_wakeup_routine. The same funtion in X86_64 doesn't have this problem. Signed-off-by: Zhang Rui <rui.zhang@intel.com> Looks-okay-to: Pavel Machek <pavel@ucw.cz> Signed-off-by: Rafael J. Wysocki <rjw@sisk.pl> Cc: Len Brown <lenb@kernel.org> Acked-by: Andi Kleen <ak@suse.de> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'arch/x86_64/kernel/k8.c')
0 files changed, 0 insertions, 0 deletions