[sh] smp-shx3.c: error: ignoring return value of 'request_irq', declared with attribute warn_unused_result

[sh] smp-shx3.c: error: ignoring return value of 'request_irq', declared with attribute warn_unused_result

[Naresh Kamboju]

Linux next 20201209 tag the arch 'sh' defconfig build failed to build with
gcc-8, gcc-9 and gcc-10.

arch/sh/kernel/cpu/sh4a/smp-shx3.c: In function 'shx3_prepare_cpus':
arch/sh/kernel/cpu/sh4a/smp-shx3.c:76:3: error: ignoring return value
of 'request_irq', declared with attribute warn_unused_result
[-Werror=unused-result]
   request_irq(104 + i, ipi_interrupt_handler,
   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
        IRQF_PERCPU, "IPI", (void *)(long)i);
        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Reported-by: Naresh Kamboju <naresh.kamboju@linaro.org>

steps to reproduce:
-------------------
# TuxMake is a command line tool and Python library that provides
# portable and repeatable Linux kernel builds across a variety of
# architectures, toolchains, kernel configurations, and make targets.
#
# TuxMake supports the concept of runtimes.
# See https://docs.tuxmake.org/runtimes/, for that to work it requires
# that you install podman or docker on your system.
#
# To install tuxmake on your system globally:
# sudo pip3 install -U tuxmake
#
# See https://docs.tuxmake.org/ for complete documentation.

tuxmake --runtime docker --target-arch sh --toolchain gcc-9 --kconfig defconfig

metadata:
    git_repo: https://gitlab.com/Linaro/lkft/mirrors/next/linux-next
    target_arch: sh
    toolchain: gcc-9
    git_short_log: 2f1d5c77f13f (\Add linux-next specific files for 20201209\)
    git_sha: 2f1d5c77f13fe64497c2e2601605f7d7ec4da9b1
    git_describe: next-20201209
    download_url: https://builds.tuxbuild.com/1lPwmsuPj4eW5gsLeObYvbXC3fw/

Full build log,
https://gitlab.com/Linaro/lkft/mirrors/next/linux-next/-/jobs/899463251

-- 
Linaro LKFT
https://lkft.linaro.org

Re: [sh] smp-shx3.c: error: ignoring return value of 'request_irq', declared with attribute warn_unused_result

[John Paul Adrian Glaubitz]

Hello Naresh!

On 12/9/20 6:26 PM, Naresh Kamboju wrote:
> Linux next 20201209 tag the arch 'sh' defconfig build failed to build with
> gcc-8, gcc-9 and gcc-10.
> 
> arch/sh/kernel/cpu/sh4a/smp-shx3.c: In function 'shx3_prepare_cpus':
> arch/sh/kernel/cpu/sh4a/smp-shx3.c:76:3: error: ignoring return value
> of 'request_irq', declared with attribute warn_unused_result
> [-Werror=unused-result]
>    request_irq(104 + i, ipi_interrupt_handler,
>    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>         IRQF_PERCPU, "IPI", (void *)(long)i);
>         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> 
> Reported-by: Naresh Kamboju <naresh.kamboju@linaro.org>

I can't reproduce this issue on Linus' current tree as of today.

Maybe the issue has been fixed in the meantime?

Adrian

-- 
 .''`.  John Paul Adrian Glaubitz
: :' :  Debian Developer - glaubitz@debian.org
`. `'   Freie Universitaet Berlin - glaubitz@physik.fu-berlin.de
  `-    GPG: 62FF 8A75 84E0 2956 9546  0006 7426 3B37 F5B5 F913

Re: [sh] smp-shx3.c: error: ignoring return value of 'request_irq', declared with attribute warn_unused_result

[Naresh Kamboju]

On Fri, 1 Jan 2021 at 20:18, John Paul Adrian Glaubitz
<glaubitz@physik.fu-berlin.de> wrote:
>
> Hello Naresh!
>
> On 12/9/20 6:26 PM, Naresh Kamboju wrote:
> > Linux next 20201209 tag the arch 'sh' defconfig build failed to build with
> > gcc-8, gcc-9 and gcc-10.
> >
> > arch/sh/kernel/cpu/sh4a/smp-shx3.c: In function 'shx3_prepare_cpus':
> > arch/sh/kernel/cpu/sh4a/smp-shx3.c:76:3: error: ignoring return value
> > of 'request_irq', declared with attribute warn_unused_result
> > [-Werror=unused-result]
> >    request_irq(104 + i, ipi_interrupt_handler,
> >    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> >         IRQF_PERCPU, "IPI", (void *)(long)i);
> >         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> >
> > Reported-by: Naresh Kamboju <naresh.kamboju@linaro.org>
>
> I can't reproduce this issue on Linus' current tree as of today.
>
> Maybe the issue has been fixed in the meantime?

Seems like this has been fixed in the meantime.

- Naresh