public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: "Arnd Bergmann" <arnd@arndb.de>
To: "kernel test robot" <yujie.liu@intel.com>,
	"Linus Walleij" <linus.walleij@linaro.org>
Cc: "kernel test robot" <lkp@intel.com>,
	oe-kbuild-all@lists.linux.dev, linux-kernel@vger.kernel.org,
	"Richard Henderson" <richard.henderson@linaro.org>,
	"Ivan Kokshaysky" <ink@jurassic.park.msu.ru>,
	"Matt Turner" <mattst88@gmail.com>
Subject: Re: arch/alpha/include/asm/core_t2.h:587:23: warning: no previous prototype for 't2_ioread64'
Date: Tue, 09 Jan 2024 09:23:57 +0100	[thread overview]
Message-ID: <d9d3dbe5-1f5a-45cd-a5cc-a54836d8e077@app.fastmail.com> (raw)
In-Reply-To: <ZZvGxhON7wNBgyg5@yujie-X299>

On Mon, Jan 8, 2024, at 10:56, Yujie Liu wrote:
> On Sun, Jan 07, 2024 at 09:09:21PM +0100, Linus Walleij wrote:
>> Hi robot,
>> 
>> On Sat, Jan 6, 2024 at 10:40 AM kernel test robot <lkp@intel.com> wrote:
>> 
>> 
>> > Hi Arnd,
>> >
>> > FYI, the error/warning still remains.
>> 
>> FYI I have already said I can't reproduce this at all. Not using
>> allnoconfig, not using the .config attached to the report.
>> 
>> I have no idea what this is.
>
> We double checked this, and looks like it needs to be reproduced by
> "make W=1". Could you please recheck with W=1?
>
> The following link has some hints of handling W=1 warnings:
>
> https://github.com/intel/lkp-tests/wiki/LKP-FAQ#how-is-one-meant-to-act-on-w1-reports-like--wmissing-prototypes
>
> Please kindly let us know if you'd like to ignore these W=1 warnings,
> and we will configure the bot for you to avoid further bothering.

This warning used to be part of the W=1 set (as well as C=1
with sparse) but is now enabled by default once my patch from
linux-mm makes it into mainline, but there are more things going
on here:

- the particular warning about t2_ioread64() gets flagged by lkp
  because it was introduced when I extended the ioread family
  to also include the 64-bit access rather than just 8/16/32 bit
  functions. The other ones were always there but are ignored
  by lkp.

- Someone who cares about alpha should just address all
  the warnings, including the old one about 'clone3' that needs
  to be done by someone that understands the alpha calling
  conventions, see below for the current output in linux-next.

- part of the problem of the missing-prototypes warnings on
  alpha is the extensive use of 'extern inline' on this
  architecture that should be replaced with 'static inline' in
  almost all cases (as we did on all other architectures
  decades ago).

    Arnd

8<---
<stdin>:1521:2: warning: #warning syscall clone3 not implemented [-Wcpp]
/home/arnd/arm-soc/arch/alpha/kernel/traps.c:178:1: warning: no previous prototype for 'do_entArith' [-Wmissing-prototypes]
/home/arnd/arm-soc/arch/alpha/kernel/traps.c:200:1: warning: no previous prototype for 'do_entIF' [-Wmissing-prototypes]
/home/arnd/arm-soc/arch/alpha/kernel/traps.c:341:1: warning: no previous prototype for 'do_entDbg' [-Wmissing-prototypes]
/home/arnd/arm-soc/arch/alpha/kernel/traps.c:377:1: warning: no previous prototype for 'do_entUna' [-Wmissing-prototypes]
/home/arnd/arm-soc/arch/alpha/kernel/traps.c:662:1: warning: no previous prototype for 'do_entUnaUser' [-Wmissing-prototypes]
/home/arnd/arm-soc/arch/alpha/kernel/irq.c:96:1: warning: no previous prototype for 'handle_irq' [-Wmissing-prototypes]
/home/arnd/arm-soc/arch/alpha/kernel/irq_alpha.c:45:1: warning: no previous prototype for 'do_entInt' [-Wmissing-prototypes]
/home/arnd/arm-soc/arch/alpha/kernel/signal.c:205:1: warning: no previous prototype for 'do_sigreturn' [-Wmissing-prototypes]
/home/arnd/arm-soc/arch/alpha/kernel/signal.c:233:1: warning: no previous prototype for 'do_rt_sigreturn' [-Wmissing-prototypes]
/home/arnd/arm-soc/arch/alpha/kernel/signal.c:523:1: warning: no previous prototype for 'do_work_pending' [-Wmissing-prototypes]
/home/arnd/arm-soc/arch/alpha/kernel/ptrace.c:322:26: warning: no previous prototype for 'syscall_trace_enter' [-Wmissing-prototypes]
/home/arnd/arm-soc/arch/alpha/kernel/ptrace.c:334:1: warning: no previous prototype for 'syscall_trace_leave' [-Wmissing-prototypes]
/home/arnd/arm-soc/arch/alpha/kernel/io.c:654:1: warning: no previous prototype for 'scr_memcpyw' [-Wmissing-prototypes]
/home/arnd/arm-soc/arch/alpha/kernel/bugs.c:19:9: warning: no previous prototype for 'cpu_show_meltdown' [-Wmissing-prototypes]
/home/arnd/arm-soc/arch/alpha/kernel/bugs.c:28:9: warning: no previous prototype for 'cpu_show_spectre_v1' [-Wmissing-prototypes]
/home/arnd/arm-soc/arch/alpha/kernel/bugs.c:37:9: warning: no previous prototype for 'cpu_show_spectre_v2' [-Wmissing-prototypes]
/home/arnd/arm-soc/arch/alpha/kernel/pci.c:285:1: warning: no previous prototype for 'pcibios_claim_one_bus' [-Wmissing-prototypes]
/home/arnd/arm-soc/arch/alpha/include/asm/core_t2.h:457:20: warning: no previous prototype for 't2_readb' [-Wmissing-prototypes]
/home/arnd/arm-soc/arch/alpha/include/asm/core_t2.h:468:21: warning: no previous prototype for 't2_readw' [-Wmissing-prototypes]
/home/arnd/arm-soc/arch/alpha/include/asm/core_t2.h:483:21: warning: no previous prototype for 't2_readl' [-Wmissing-prototypes]
/home/arnd/arm-soc/arch/alpha/include/asm/core_t2.h:494:21: warning: no previous prototype for 't2_readq' [-Wmissing-prototypes]
/home/arnd/arm-soc/arch/alpha/include/asm/core_t2.h:507:22: warning: no previous prototype for 't2_writeb' [-Wmissing-prototypes]
/home/arnd/arm-soc/arch/alpha/include/asm/core_t2.h:518:22: warning: no previous prototype for 't2_writew' [-Wmissing-prototypes]
/home/arnd/arm-soc/arch/alpha/include/asm/core_t2.h:533:22: warning: no previous prototype for 't2_writel' [-Wmissing-prototypes]
/home/arnd/arm-soc/arch/alpha/include/asm/core_t2.h:542:22: warning: no previous prototype for 't2_writeq' [-Wmissing-prototypes]
/home/arnd/arm-soc/arch/alpha/include/asm/core_t2.h:554:31: warning: no previous prototype for 't2_ioportmap' [-Wmissing-prototypes]
/home/arnd/arm-soc/arch/alpha/include/asm/core_t2.h:559:31: warning: no previous prototype for 't2_ioremap' [-Wmissing-prototypes]
/home/arnd/arm-soc/arch/alpha/include/asm/core_t2.h:565:21: warning: no previous prototype for 't2_is_ioaddr' [-Wmissing-prototypes]
/home/arnd/arm-soc/arch/alpha/include/asm/core_t2.h:570:21: warning: no previous prototype for 't2_is_mmio' [-Wmissing-prototypes]
/home/arnd/arm-soc/arch/alpha/include/asm/core_t2.h:579:23: warning: no previous prototype for 't2_ioread8' [-Wmissing-prototypes]
/home/arnd/arm-soc/arch/alpha/include/asm/core_t2.h:586:22: warning: no previous prototype for 't2_iowrite8' [-Wmissing-prototypes]
/home/arnd/arm-soc/arch/alpha/include/asm/core_t2.h:579:23: warning: no previous prototype for 't2_ioread16' [-Wmissing-prototypes]
/home/arnd/arm-soc/arch/alpha/include/asm/core_t2.h:586:22: warning: no previous prototype for 't2_iowrite16' [-Wmissing-prototypes]
/home/arnd/arm-soc/arch/alpha/include/asm/core_t2.h:579:23: warning: no previous prototype for 't2_ioread32' [-Wmissing-prototypes]
/home/arnd/arm-soc/arch/alpha/include/asm/core_t2.h:586:22: warning: no previous prototype for 't2_iowrite32' [-Wmissing-prototypes]
/home/arnd/arm-soc/arch/alpha/include/asm/core_t2.h:579:23: warning: no previous prototype for 't2_ioread64' [-Wmissing-prototypes]
/home/arnd/arm-soc/arch/alpha/include/asm/core_t2.h:586:22: warning: no previous prototype for 't2_iowrite64' [-Wmissing-prototypes]
/home/arnd/arm-soc/arch/alpha/include/asm/core_t2.h:606:33: warning: no previous prototype for 't2_iounmap' [-Wmissing-prototypes]
/home/arnd/arm-soc/arch/alpha/kernel/srmcons.c:267:1: warning: no previous prototype for 'register_srm_console' [-Wmissing-prototypes]
/home/arnd/arm-soc/arch/alpha/kernel/srmcons.c:277:1: warning: no previous prototype for 'unregister_srm_console' [-Wmissing-prototypes]
/home/arnd/arm-soc/arch/alpha/kernel/console.c:39:1: warning: no previous prototype for 'locate_and_init_vga' [-Wmissing-prototypes]
/home/arnd/arm-soc/arch/alpha/kernel/console.c:72:1: warning: no previous prototype for 'find_console_vga_hose' [-Wmissing-prototypes]
/home/arnd/arm-soc/arch/alpha/kernel/core_cia.c:284:1: warning: no previous prototype for 'cia_pci_tbi_try2' [-Wmissing-prototypes]
/home/arnd/arm-soc/arch/alpha/kernel/core_cia.c:580:1: warning: no previous prototype for 'cia_save_srm_settings' [-Wmissing-prototypes]
/home/arnd/arm-soc/arch/alpha/kernel/core_cia.c:606:1: warning: no previous prototype for 'cia_restore_srm_settings' [-Wmissing-prototypes]
/home/arnd/arm-soc/arch/alpha/kernel/sys_nautilus.c:82:1: warning: no previous prototype for 'nautilus_kill_arch' [-Wmissing-prototypes]
/home/arnd/arm-soc/arch/alpha/kernel/sys_nautilus.c:131:1: warning: no previous prototype for 'nautilus_machine_check' [-Wmissing-prototypes]
/home/arnd/arm-soc/arch/alpha/kernel/sys_nautilus.c:201:1: warning: no previous prototype for 'nautilus_init_pci' [-Wmissing-prototypes]
/home/arnd/arm-soc/arch/alpha/kernel/smc37c669.c:2434:1: warning: no previous prototype for 'SMC37c669_dump_registers' [-Wmissing-prototypes]
/home/arnd/arm-soc/arch/alpha/kernel/smc37c669.c:2467:13: warning: no previous prototype for 'SMC669_Init' [-Wmissing-prototypes]
/home/arnd/arm-soc/arch/alpha/kernel/smc37c93x.c:238:12: warning: no previous prototype for 'SMC93x_Init' [-Wmissing-prototypes]
/home/arnd/arm-soc/arch/alpha/mm/init.c:261:1: warning: no previous prototype for 'srm_paging_stop' [-Wmissing-prototypes]

  reply	other threads:[~2024-01-09  8:25 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-01-06  9:40 arch/alpha/include/asm/core_t2.h:587:23: warning: no previous prototype for 't2_ioread64' kernel test robot
2024-01-07 20:09 ` Linus Walleij
2024-01-08  9:56   ` Yujie Liu
2024-01-09  8:23     ` Arnd Bergmann [this message]
  -- strict thread matches above, loose matches on Subject: below --
2023-09-08 20:34 kernel test robot
2022-10-09  2:06 kernel test robot

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=d9d3dbe5-1f5a-45cd-a5cc-a54836d8e077@app.fastmail.com \
    --to=arnd@arndb.de \
    --cc=ink@jurassic.park.msu.ru \
    --cc=linus.walleij@linaro.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lkp@intel.com \
    --cc=mattst88@gmail.com \
    --cc=oe-kbuild-all@lists.linux.dev \
    --cc=richard.henderson@linaro.org \
    --cc=yujie.liu@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox