Re: [PATCH] FRV: Fix the extern declaration of kallsyms_num_syms

From: Andrew Morton
Date: Fri Nov 09 2007 - 14:52:50 EST


On Tue, 06 Nov 2007 21:48:20 +0000
David Howells <dhowells@xxxxxxxxxx> wrote:

> From: David Howells <dhowells@xxxxxxxxxx>
>
> Fix the extern declaration of kallsyms_num_syms to indicate that the symbol
> does not reside in the small-data storage space, and so may not be accessed
> relative to the small data base register.
>
> Signed-off-by: David Howells <dhowells@xxxxxxxxxx>
> ---
>
> kernel/kallsyms.c | 3 ++-
> 1 files changed, 2 insertions(+), 1 deletions(-)
>
> diff --git a/kernel/kallsyms.c b/kernel/kallsyms.c
> index 474219a..fa84b83 100644
> --- a/kernel/kallsyms.c
> +++ b/kernel/kallsyms.c
> @@ -32,7 +32,8 @@
>
> /* These will be re-linked against their real values during the second link stage */
> extern const unsigned long kallsyms_addresses[] __attribute__((weak));
> -extern const unsigned long kallsyms_num_syms __attribute__((weak));
> +extern const unsigned long kallsyms_num_syms
> +__nongpreldata __attribute__((weak));

umm, this makes Linux an frv-only operating system...

> extern const u8 kallsyms_names[] __attribute__((weak));
>
> extern const u8 kallsyms_token_table[] __attribute__((weak));
-
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/