Re: [PATCH v2] mips: lib: uncached: fix non-standard usage of variable 'sp'

From: Naresh Kamboju
Date: Sat Dec 19 2020 - 23:09:42 EST


On Mon, 14 Dec 2020 at 21:05, Thomas Bogendoerfer
<tsbogend@xxxxxxxxxxxxxxxx> wrote:
>
> On Fri, Dec 11, 2020 at 11:24:37AM +0100, Anders Roxell wrote:
> > When building mips tinyconfig with clang the following warning show up:
> >
> > arch/mips/lib/uncached.c:45:6: warning: variable 'sp' is uninitialized when used here [-Wuninitialized]
> > if (sp >= (long)CKSEG0 && sp < (long)CKSEG2)
> > ^~
> > arch/mips/lib/uncached.c:40:18: note: initialize the variable 'sp' to silence this warning
> > register long sp __asm__("$sp");
> > ^
> > = 0
> > 1 warning generated.

<trimp>

> > [1] https://docs.w3cub.com/gcc~10/local-register-variables
> > Signed-off-by: Anders Roxell <anders.roxell@xxxxxxxxxx>
> > ---
> > arch/mips/lib/uncached.c | 4 +++-
> > 1 file changed, 3 insertions(+), 1 deletion(-)
>
> applied to mips-next.

We have noticed these build failures on stable tree also.
May I request you to initiate the process to get into the stable tree
and for the following branches.
- linux-5.10.y
- linux-5.9.y
- linux-5.4.y
- linux-4.19.y

- Naresh