On Sat, Nov 7, 2009 at 7:20 PM, Atsushi Nemoto <anemo@xxxxxxxxxxxxx> wrote:
> Since commit 22242681 ("MIPS: Extend COMMAND_LINE_SIZE"), CL_SIZE is
> 4096 and local array variables with this size will cause an build
> failure with default CONFIG_FRAME_WARN settings.
>
> Although current users of such array variables are all early bootstrap
> code and might not cause real stack overflow (thread_info corruption),
> it would be safe to declare these arrays static with __initdata.
>
> Signed-off-by: Atsushi Nemoto <anemo@xxxxxxxxxxxxx>
> ---
> arch/mips/bcm47xx/prom.c | 2 +-
> arch/mips/mti-malta/malta-memory.c | 3 ++-
> arch/mips/rb532/prom.c | 2 +-
> arch/mips/txx9/generic/setup.c | 4 ++--
> 4 files changed, 6 insertions(+), 5 deletions(-)
>
> diff --git a/arch/mips/bcm47xx/prom.c b/arch/mips/bcm47xx/prom.c
> index 079e33d..fb284c3 100644
> --- a/arch/mips/bcm47xx/prom.c
> +++ b/arch/mips/bcm47xx/prom.c
> @@ -100,7 +100,7 @@ static __init void prom_init_console(void)
>
> static __init void prom_init_cmdline(void)
> {
> - char buf[CL_SIZE];
> + static char buf[CL_SIZE] __initdata;
If this is intended for -queue, this patch won't apply, because
CL_SIZE was recently removed in favor of using CONFIG_CMDLINE_SIZE
directly.
Also, I think it's more common to place __initdata before the variable
name, not after it, although tastes do differ. :)
Thanks,
Dmitri
