Re: [PATCH] sparc: correct section of current_pc()

From: David Miller
Date: Wed Oct 22 2008 - 00:57:39 EST


From: Frederic Weisbecker <fweisbec@xxxxxxxxx>
Date: Sat, 18 Oct 2008 18:32:45 +0200

[ sparclinux@xxxxxxxxxxxxxxx CC:'d ]

> Latest mainline gives this section mismatch on sparc:
>
> The function current_pc() references
> the variable __init no_sun4u_here.
> This is often because current_pc lacks a __init
> annotation or the annotation of no_sun4u_here is wrong.
>
> Since current_pc() is used only in early time, it is correct to
> put it in .init section.
>
> Signed-off-by: Frederic Weisbecker <fweisbec@xxxxxxxxx>

Also applied, thanks.

> diff --git a/arch/sparc/kernel/head.S b/arch/sparc/kernel/head.S
> index 2d325fd..2fe2c11 100644
> --- a/arch/sparc/kernel/head.S
> +++ b/arch/sparc/kernel/head.S
> @@ -465,6 +465,7 @@ gokernel:
> mov %o7, %g4 ! Save %o7
>
> /* Jump to it, and pray... */
> + __INIT
> current_pc:
> call 1f
> nop
--
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/