Re: [PATCH] x86/power/hibernate: Use local labels in asm

From: Rafael J. Wysocki
Date: Sun Apr 12 2015 - 18:42:13 EST


On Sunday, April 12, 2015 09:45:06 PM Borislav Petkov wrote:
> From: Borislav Petkov <bp@xxxxxxx>
>
> ... so that they don't appear in the object file and thus in objdump
> output. They're local anyway and have a meaning only within that file.
>
> No functionality change.
>
> Signed-off-by: Borislav Petkov <bp@xxxxxxx>
> Cc: "Rafael J. Wysocki" <rjw@xxxxxxxxxxxxx>
> Cc: Pavel Machek <pavel@xxxxxx>
> Cc: Thomas Gleixner <tglx@xxxxxxxxxxxxx>
> Cc: Ingo Molnar <mingo@xxxxxxxxxx>
> Cc: "H. Peter Anvin" <hpa@xxxxxxxxx>
> Cc: linux-pm@xxxxxxxxxxxxxxx

Acked-by: Rafael J. Wysocki <rafael.j.wysocki@xxxxxxxxx>

> ---
> arch/x86/power/hibernate_asm_64.S | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/arch/x86/power/hibernate_asm_64.S b/arch/x86/power/hibernate_asm_64.S
> index 3c4469a7a929..e2386cb4e0c3 100644
> --- a/arch/x86/power/hibernate_asm_64.S
> +++ b/arch/x86/power/hibernate_asm_64.S
> @@ -78,9 +78,9 @@ ENTRY(restore_image)
>
> /* code below has been relocated to a safe page */
> ENTRY(core_restore_code)
> -loop:
> +.Lloop:
> testq %rdx, %rdx
> - jz done
> + jz .Ldone
>
> /* get addresses from the pbe and copy the page */
> movq pbe_address(%rdx), %rsi
> @@ -91,8 +91,8 @@ loop:
>
> /* progress to the next pbe */
> movq pbe_next(%rdx), %rdx
> - jmp loop
> -done:
> + jmp .Lloop
> +.Ldone:
> /* jump to the restore_registers address from the image header */
> jmpq *%rax
> /*
>

--
I speak only for myself.
Rafael J. Wysocki, Intel Open Source Technology Center.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/