ACPI: preserve the ebx value in acpi_copy_wakeup_routine
authorTian Kevin <>
Sun, 24 Jun 2007 00:16:52 +0000 (17:16 -0700)
committerLinus Torvalds <>
Sun, 24 Jun 2007 15:59:12 +0000 (08:59 -0700)
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 <>
Looks-okay-to: Pavel Machek <>
Signed-off-by: Rafael J. Wysocki <>
Cc: Len Brown <>
Acked-by: Andi Kleen <>
Signed-off-by: Andrew Morton <>
Signed-off-by: Linus Torvalds <>

index b781b38131c06cb9237745bc1144bc4bfee609ac..a2295a34b2c727577c82ed19c8656c06ec63d7c4 100644 (file)
@@ -230,6 +230,7 @@ bogus_magic:
+       pushl   %ebx
        sgdt    saved_gdt
        sidt    saved_idt
        sldt    saved_ldt
@@ -263,6 +264,7 @@ ENTRY(acpi_copy_wakeup_routine)
        movl    %edx, video_flags - wakeup_start (%eax)
        movl    $0x12345678, real_magic - wakeup_start (%eax)
        movl    $0x12345678, saved_magic
+       popl    %ebx