Re: [PATCH] ARM: zImage: Fix stack overflow in merge_fdt_bootargs()

2017-07-18 Thread Rask Ingemann Lambertsen
On Tue, Jul 18, 2017 at 09:39:10AM +0200, Richard Genoud wrote: > On 16/07/2017 23:43, Rask Ingemann Lambertsen wrote: [snip] > > +/* This is called early on from head.S, so it can't use much stack. */ > > +static void merge_fdt_bootargs(void *fdt, const char *atag_cmdline) > > +{ > > + char

Re: [PATCH] ARM: zImage: Fix stack overflow in merge_fdt_bootargs()

2017-07-18 Thread Rask Ingemann Lambertsen
On Tue, Jul 18, 2017 at 09:39:10AM +0200, Richard Genoud wrote: > On 16/07/2017 23:43, Rask Ingemann Lambertsen wrote: [snip] > > +/* This is called early on from head.S, so it can't use much stack. */ > > +static void merge_fdt_bootargs(void *fdt, const char *atag_cmdline) > > +{ > > + char

Re: [PATCH] ARM: zImage: Fix stack overflow in merge_fdt_bootargs()

2017-07-18 Thread Richard Genoud
On 16/07/2017 23:43, Rask Ingemann Lambertsen wrote: > This function is called very early on from head.S and currently sets up a > stack frame of more than 1024 bytes: > > atags_to_fdt.c: In function ‘merge_fdt_bootargs’: > atags_to_fdt.c:98:1: warning: the frame size of 1032 bytes is larger than

Re: [PATCH] ARM: zImage: Fix stack overflow in merge_fdt_bootargs()

2017-07-18 Thread Richard Genoud
On 16/07/2017 23:43, Rask Ingemann Lambertsen wrote: > This function is called very early on from head.S and currently sets up a > stack frame of more than 1024 bytes: > > atags_to_fdt.c: In function ‘merge_fdt_bootargs’: > atags_to_fdt.c:98:1: warning: the frame size of 1032 bytes is larger than

[PATCH] ARM: zImage: Fix stack overflow in merge_fdt_bootargs()

2017-07-16 Thread Rask Ingemann Lambertsen
This function is called very early on from head.S and currently sets up a stack frame of more than 1024 bytes: atags_to_fdt.c: In function ‘merge_fdt_bootargs’: atags_to_fdt.c:98:1: warning: the frame size of 1032 bytes is larger than 1024 bytes [-Wframe-larger-than=] This causes a crash and

[PATCH] ARM: zImage: Fix stack overflow in merge_fdt_bootargs()

2017-07-16 Thread Rask Ingemann Lambertsen
This function is called very early on from head.S and currently sets up a stack frame of more than 1024 bytes: atags_to_fdt.c: In function ‘merge_fdt_bootargs’: atags_to_fdt.c:98:1: warning: the frame size of 1032 bytes is larger than 1024 bytes [-Wframe-larger-than=] This causes a crash and