Received-SPF: pass (google.com: domain of linux-kernel+bounces-117849-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
Precedence: bulk
MIME-Version: 1.0
References: <20240325120003.1767691-1-sashal@kernel.org> <56d3285a-ed22-44bd-8c22-ce51ad159a81@linaro.org>
 <20240325181410.GA4122244@google.com>
In-Reply-To: <20240325181410.GA4122244@google.com>
From: =?UTF-8?B?RGFuaWVsIETDrWF6?= <daniel.diaz@linaro.org>
Date: Mon, 25 Mar 2024 13:47:40 -0600
Message-ID: <CAEUSe78CQrHFEz92svQKuvjU91FDc=Dt+NNf_5_=pKeE22TzXg@mail.gmail.com>
Subject: Re: [PATCH 6.7 000/707] 6.7.11-rc2 review
To: Sami Tolvanen <samitolvanen@google.com>
Cc: Palmer Dabbelt <palmer@dabbelt.com>, Sasha Levin <sashal@kernel.org>, linux-kernel@vger.kernel.org, 
	stable@vger.kernel.org, torvalds@linux-foundation.org, 
	akpm@linux-foundation.org, linux@roeck-us.net, shuah@kernel.org, 
	patches@kernelci.org, lkft-triage@lists.linaro.org, 
	florian.fainelli@broadcom.com, pavel@denx.de
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

Hello!

On Mon, 25 Mar 2024 at 12:14, Sami Tolvanen <samitolvanen@google.com> wrote=
:
> On Mon, Mar 25, 2024 at 11:43:48AM -0600, Daniel D=C3=ADaz wrote:
> > Hello!
> >
> > On 25/03/24 6:00 a. m., Sasha Levin wrote:
> > > This is the start of the stable review cycle for the 6.7.11 release.
> > > There are 707 patches in this series, all will be posted as a respons=
e
> > > to this one.  If anyone has any issues with these being applied, plea=
se
> > > let me know.
> > >
> > > Responses should be made by Wed Mar 27 12:00:02 PM UTC 2024.
> > > Anything received after that time might be too late.
> > >
> > > The whole patch series can be found in one patch at:
> > >          https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux=
-stable-rc.git/patch/?id=3Dlinux-6.7.y&id2=3Dv6.7.10
> > > or in the git tree and branch at:
> > >          git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-s=
table-rc.git linux-6.7.y
> > > and the diffstat can be found below.
> > >
> > > Thanks,
> > > Sasha
> >
> > We see *lots* of new warnings in RISC-V with Clang 17. Here's one:
> >
> > -----8<-----
> >   /builds/linux/mm/oom_kill.c:1195:1: warning: unused function '___se_s=
ys_process_mrelease' [-Wunused-function]
> >    1195 | SYSCALL_DEFINE2(process_mrelease, int, pidfd, unsigned int, f=
lags)
> >         | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~=
~~~~~
> >   /builds/linux/include/linux/syscalls.h:221:36: note: expanded from ma=
cro 'SYSCALL_DEFINE2'
> >     221 | #define SYSCALL_DEFINE2(name, ...) SYSCALL_DEFINEx(2, _##name=
, __VA_ARGS__)
> >         |                                    ^~~~~~~~~~~~~~~~~~~~~~~~~~=
~~~~~~~~~~~~~~
> >   /builds/linux/include/linux/syscalls.h:231:2: note: expanded from mac=
ro 'SYSCALL_DEFINEx'
> >     231 |         __SYSCALL_DEFINEx(x, sname, __VA_ARGS__)
> >         |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> >   /builds/linux/arch/riscv/include/asm/syscall_wrapper.h:81:2: note: ex=
panded from macro '__SYSCALL_DEFINEx'
> >      81 |         __SYSCALL_SE_DEFINEx(x, sys, name, __VA_ARGS__)      =
                   \
> >         |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> >   /builds/linux/arch/riscv/include/asm/syscall_wrapper.h:40:14: note: e=
xpanded from macro '__SYSCALL_SE_DEFINEx'
> >      40 |         static long ___se_##prefix##name(__MAP(x,__SC_LONG,__=
VA_ARGS__))
> >         |                     ^~~~~~~~~~~~~~~~~~~~
> >   <scratch space>:30:1: note: expanded from here
> >      30 | ___se_sys_process_mrelease
> >         | ^~~~~~~~~~~~~~~~~~~~~~~~~~
> >   1 warning generated.
> > ----->8-----
>
> Yup, I can reproduce this with ToT Clang. It looks like the alias
> isn't sufficient for Clang and we need to add an explicit __used
> attribute. Can you confirm if this patch fixes the issue for you?
>
> diff --git a/arch/riscv/include/asm/syscall_wrapper.h b/arch/riscv/includ=
e/asm/syscall_wrapper.h
> index 980094c2e976..ac80216549ff 100644
> --- a/arch/riscv/include/asm/syscall_wrapper.h
> +++ b/arch/riscv/include/asm/syscall_wrapper.h
> @@ -36,7 +36,8 @@ asmlinkage long __riscv_sys_ni_syscall(const struct pt_=
regs *);
>                                         ulong)                           =
               \
>                         __attribute__((alias(__stringify(___se_##prefix##=
name))));      \
>         __diag_pop();                                                    =
               \
> -       static long noinline ___se_##prefix##name(__MAP(x,__SC_LONG,__VA_=
ARGS__));      \
> +       static long noinline ___se_##prefix##name(__MAP(x,__SC_LONG,__VA_=
ARGS__))       \
> +                       __used;                                          =
               \
>         static long ___se_##prefix##name(__MAP(x,__SC_LONG,__VA_ARGS__))

It does: the hundreds of warnings are gone. Build-tested with Clang
17. Logs, configs, binaries, etc., here:

  https://tuxapi.tuxsuite.com/v1/groups/linaro/projects/daniel/builds/2eC43=
UtTjYk6loC9pNKT28SGmd5

Tested-by: Linux Kernel Functional Testing <lkft@linaro.org>

Thanks and greetings!

Daniel D=C3=ADaz
daniel.diaz@linaro.org