Re: [PATCH 6.7 000/707] 6.7.11-rc2 review

[Daniel Díaz]

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 response
> to this one.  If anyone has any issues with these being applied, please
> 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=linux-6.7.y&id2=v6.7.10
> or in the git tree and branch at:
>          git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-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_sys_process_mrelease' [-Wunused-function]
   1195 | SYSCALL_DEFINE2(process_mrelease, int, pidfd, unsigned int, flags)
        | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
  /builds/linux/include/linux/syscalls.h:221:36: note: expanded from macro 'SYSCALL_DEFINE2'
    221 | #define SYSCALL_DEFINE2(name, ...) SYSCALL_DEFINEx(2, _##name, __VA_ARGS__)
        |                                    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
  /builds/linux/include/linux/syscalls.h:231:2: note: expanded from macro 'SYSCALL_DEFINEx'
    231 |         __SYSCALL_DEFINEx(x, sname, __VA_ARGS__)
        |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
  /builds/linux/arch/riscv/include/asm/syscall_wrapper.h:81:2: note: expanded 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: expanded 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-----

Bisection points to:

  commit 9b4a60bd0b1fa521e786c809ec8815291f150c47
  Author: Sami Tolvanen <samitolvanen@xxxxxxxxxx>
  Date:   Mon Mar 11 19:31:44 2024 +0000

      riscv: Fix syscall wrapper for >word-size arguments
      
      [ Upstream commit a9ad73295cc1e3af0253eee7d08943b2419444c4 ]

Reverting that patch makes the (300+) warnings disappear.

Reproducer:

  tuxmake --runtime podman --target-arch riscv --toolchain clang-17 --kconfig allnoconfig --kconfig-add rv32_defconfig LLVM=1

Reported-by: Linux Kernel Functional Testing <lkft@xxxxxxxxxx>

Greetings!

Daniel Díaz
daniel.diaz@xxxxxxxxxx