Re: [PATCH 2/6] FRV: Fix FRV arch compile errors [try #3]

From: Andrew Morton
Date: Thu Jul 06 2006 - 13:30:22 EST


On Thu, 06 Jul 2006 13:47:21 +0100
David Howells <dhowells@xxxxxxxxxx> wrote:

> --- a/include/linux/bootmem.h
> +++ b/include/linux/bootmem.h
> @@ -91,7 +91,7 @@ static inline void *alloc_remap(int nid,
> }
> #endif
>
> -extern unsigned long nr_kernel_pages;
> +extern unsigned long __initdata nr_kernel_pages;

- The __init-style tags on declarations don't actually do anything and
the compiler doesn't check for consistency with the definition - it's
best to just omit it from the declaration.

- Setting nr_kernel_pages to be unloaded at free_initmem() seems risky.

- nr_kernel_pages is actually __meminitdata.

So I'll drop this hunk.
-
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/