LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
* [block:for-5.15/io_uring 43/45] kernel/exit.c:780:24: error: too few arguments to function call, single argument 'files' was not specified
@ 2021-08-12 21:39 kernel test robot
  0 siblings, 0 replies; 2+ messages in thread
From: kernel test robot @ 2021-08-12 21:39 UTC (permalink / raw)
  To: Hao Xu; +Cc: clang-built-linux, kbuild-all, linux-kernel, Jens Axboe

[-- Attachment #1: Type: text/plain, Size: 8878 bytes --]

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/axboe/linux-block.git for-5.15/io_uring
head:   22ec865ddfba31ddf26b9d2b4a01e8437ca1f727
commit: 6a32aa46fa0b531970581d5b425514da7f9f8e46 [43/45] io_uring: remove files pointer in cancellation functions
config: s390-randconfig-r014-20210812 (attached as .config)
compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project 767496d19cb9a1fbba57ff08095faa161998ee36)
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # install s390 cross compiling tool for clang build
        # apt-get install binutils-s390x-linux-gnu
        # https://git.kernel.org/pub/scm/linux/kernel/git/axboe/linux-block.git/commit/?id=6a32aa46fa0b531970581d5b425514da7f9f8e46
        git remote add block https://git.kernel.org/pub/scm/linux/kernel/git/axboe/linux-block.git
        git fetch --no-tags block for-5.15/io_uring
        git checkout 6a32aa46fa0b531970581d5b425514da7f9f8e46
        # save the attached .config to linux build tree
        mkdir build_dir
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross O=build_dir ARCH=s390 SHELL=/bin/bash

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>

All errors (new ones prefixed by >>):

   In file included from kernel/exit.c:51:
   In file included from include/linux/blkdev.h:25:
   In file included from include/linux/scatterlist.h:9:
   In file included from arch/s390/include/asm/io.h:75:
   include/asm-generic/io.h:464:31: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           val = __raw_readb(PCI_IOBASE + addr);
                             ~~~~~~~~~~ ^
   include/asm-generic/io.h:477:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           val = __le16_to_cpu((__le16 __force)__raw_readw(PCI_IOBASE + addr));
                                                           ~~~~~~~~~~ ^
   include/uapi/linux/byteorder/big_endian.h:36:59: note: expanded from macro '__le16_to_cpu'
   #define __le16_to_cpu(x) __swab16((__force __u16)(__le16)(x))
                                                             ^
   include/uapi/linux/swab.h:102:54: note: expanded from macro '__swab16'
   #define __swab16(x) (__u16)__builtin_bswap16((__u16)(x))
                                                        ^
   In file included from kernel/exit.c:51:
   In file included from include/linux/blkdev.h:25:
   In file included from include/linux/scatterlist.h:9:
   In file included from arch/s390/include/asm/io.h:75:
   include/asm-generic/io.h:490:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           val = __le32_to_cpu((__le32 __force)__raw_readl(PCI_IOBASE + addr));
                                                           ~~~~~~~~~~ ^
   include/uapi/linux/byteorder/big_endian.h:34:59: note: expanded from macro '__le32_to_cpu'
   #define __le32_to_cpu(x) __swab32((__force __u32)(__le32)(x))
                                                             ^
   include/uapi/linux/swab.h:115:54: note: expanded from macro '__swab32'
   #define __swab32(x) (__u32)__builtin_bswap32((__u32)(x))
                                                        ^
   In file included from kernel/exit.c:51:
   In file included from include/linux/blkdev.h:25:
   In file included from include/linux/scatterlist.h:9:
   In file included from arch/s390/include/asm/io.h:75:
   include/asm-generic/io.h:501:33: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           __raw_writeb(value, PCI_IOBASE + addr);
                               ~~~~~~~~~~ ^
   include/asm-generic/io.h:511:59: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           __raw_writew((u16 __force)cpu_to_le16(value), PCI_IOBASE + addr);
                                                         ~~~~~~~~~~ ^
   include/asm-generic/io.h:521:59: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           __raw_writel((u32 __force)cpu_to_le32(value), PCI_IOBASE + addr);
                                                         ~~~~~~~~~~ ^
   include/asm-generic/io.h:609:20: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           readsb(PCI_IOBASE + addr, buffer, count);
                  ~~~~~~~~~~ ^
   include/asm-generic/io.h:617:20: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           readsw(PCI_IOBASE + addr, buffer, count);
                  ~~~~~~~~~~ ^
   include/asm-generic/io.h:625:20: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           readsl(PCI_IOBASE + addr, buffer, count);
                  ~~~~~~~~~~ ^
   include/asm-generic/io.h:634:21: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           writesb(PCI_IOBASE + addr, buffer, count);
                   ~~~~~~~~~~ ^
   include/asm-generic/io.h:643:21: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           writesw(PCI_IOBASE + addr, buffer, count);
                   ~~~~~~~~~~ ^
   include/asm-generic/io.h:652:21: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           writesl(PCI_IOBASE + addr, buffer, count);
                   ~~~~~~~~~~ ^
>> kernel/exit.c:780:24: error: too few arguments to function call, single argument 'files' was not specified
           io_uring_files_cancel();
           ~~~~~~~~~~~~~~~~~~~~~ ^
   include/linux/io_uring.h:36:20: note: 'io_uring_files_cancel' declared here
   static inline void io_uring_files_cancel(struct files_struct *files)
                      ^
   12 warnings and 1 error generated.


vim +/files +780 kernel/exit.c

   726	
   727	void __noreturn do_exit(long code)
   728	{
   729		struct task_struct *tsk = current;
   730		int group_dead;
   731	
   732		/*
   733		 * We can get here from a kernel oops, sometimes with preemption off.
   734		 * Start by checking for critical errors.
   735		 * Then fix up important state like USER_DS and preemption.
   736		 * Then do everything else.
   737		 */
   738	
   739		WARN_ON(blk_needs_flush_plug(tsk));
   740	
   741		if (unlikely(in_interrupt()))
   742			panic("Aiee, killing interrupt handler!");
   743		if (unlikely(!tsk->pid))
   744			panic("Attempted to kill the idle task!");
   745	
   746		/*
   747		 * If do_exit is called because this processes oopsed, it's possible
   748		 * that get_fs() was left as KERNEL_DS, so reset it to USER_DS before
   749		 * continuing. Amongst other possible reasons, this is to prevent
   750		 * mm_release()->clear_child_tid() from writing to a user-controlled
   751		 * kernel address.
   752		 */
   753		force_uaccess_begin();
   754	
   755		if (unlikely(in_atomic())) {
   756			pr_info("note: %s[%d] exited with preempt_count %d\n",
   757				current->comm, task_pid_nr(current),
   758				preempt_count());
   759			preempt_count_set(PREEMPT_ENABLED);
   760		}
   761	
   762		profile_task_exit(tsk);
   763		kcov_task_exit(tsk);
   764	
   765		ptrace_event(PTRACE_EVENT_EXIT, code);
   766	
   767		validate_creds_for_do_exit(tsk);
   768	
   769		/*
   770		 * We're taking recursive faults here in do_exit. Safest is to just
   771		 * leave this task alone and wait for reboot.
   772		 */
   773		if (unlikely(tsk->flags & PF_EXITING)) {
   774			pr_alert("Fixing recursive fault but reboot is needed!\n");
   775			futex_exit_recursive(tsk);
   776			set_current_state(TASK_UNINTERRUPTIBLE);
   777			schedule();
   778		}
   779	
 > 780		io_uring_files_cancel();
   781		exit_signals(tsk);  /* sets PF_EXITING */
   782	
   783		/* sync mm's RSS info before statistics gathering */
   784		if (tsk->mm)
   785			sync_mm_rss(tsk->mm);
   786		acct_update_integrals(tsk);
   787		group_dead = atomic_dec_and_test(&tsk->signal->live);
   788		if (group_dead) {
   789			/*
   790			 * If the last thread of global init has exited, panic
   791			 * immediately to get a useable coredump.
   792			 */
   793			if (unlikely(is_global_init(tsk)))
   794				panic("Attempted to kill init! exitcode=0x%08x\n",
   795					tsk->signal->group_exit_code ?: (int)code);
   796	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 12695 bytes --]

^ permalink raw reply	[flat|nested] 2+ messages in thread

* [block:for-5.15/io_uring 43/45] kernel/exit.c:780:24: error: too few arguments to function call, single argument 'files' was not specified
@ 2021-08-12 21:45 kernel test robot
  0 siblings, 0 replies; 2+ messages in thread
From: kernel test robot @ 2021-08-12 21:45 UTC (permalink / raw)
  To: Hao Xu; +Cc: clang-built-linux, kbuild-all, linux-kernel, Jens Axboe

[-- Attachment #1: Type: text/plain, Size: 8878 bytes --]

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/axboe/linux-block.git for-5.15/io_uring
head:   22ec865ddfba31ddf26b9d2b4a01e8437ca1f727
commit: 6a32aa46fa0b531970581d5b425514da7f9f8e46 [43/45] io_uring: remove files pointer in cancellation functions
config: s390-randconfig-r044-20210812 (attached as .config)
compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project 767496d19cb9a1fbba57ff08095faa161998ee36)
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # install s390 cross compiling tool for clang build
        # apt-get install binutils-s390x-linux-gnu
        # https://git.kernel.org/pub/scm/linux/kernel/git/axboe/linux-block.git/commit/?id=6a32aa46fa0b531970581d5b425514da7f9f8e46
        git remote add block https://git.kernel.org/pub/scm/linux/kernel/git/axboe/linux-block.git
        git fetch --no-tags block for-5.15/io_uring
        git checkout 6a32aa46fa0b531970581d5b425514da7f9f8e46
        # save the attached .config to linux build tree
        mkdir build_dir
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross O=build_dir ARCH=s390 SHELL=/bin/bash

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>

All errors (new ones prefixed by >>):

   In file included from kernel/exit.c:51:
   In file included from include/linux/blkdev.h:25:
   In file included from include/linux/scatterlist.h:9:
   In file included from arch/s390/include/asm/io.h:75:
   include/asm-generic/io.h:464:31: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           val = __raw_readb(PCI_IOBASE + addr);
                             ~~~~~~~~~~ ^
   include/asm-generic/io.h:477:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           val = __le16_to_cpu((__le16 __force)__raw_readw(PCI_IOBASE + addr));
                                                           ~~~~~~~~~~ ^
   include/uapi/linux/byteorder/big_endian.h:36:59: note: expanded from macro '__le16_to_cpu'
   #define __le16_to_cpu(x) __swab16((__force __u16)(__le16)(x))
                                                             ^
   include/uapi/linux/swab.h:102:54: note: expanded from macro '__swab16'
   #define __swab16(x) (__u16)__builtin_bswap16((__u16)(x))
                                                        ^
   In file included from kernel/exit.c:51:
   In file included from include/linux/blkdev.h:25:
   In file included from include/linux/scatterlist.h:9:
   In file included from arch/s390/include/asm/io.h:75:
   include/asm-generic/io.h:490:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           val = __le32_to_cpu((__le32 __force)__raw_readl(PCI_IOBASE + addr));
                                                           ~~~~~~~~~~ ^
   include/uapi/linux/byteorder/big_endian.h:34:59: note: expanded from macro '__le32_to_cpu'
   #define __le32_to_cpu(x) __swab32((__force __u32)(__le32)(x))
                                                             ^
   include/uapi/linux/swab.h:115:54: note: expanded from macro '__swab32'
   #define __swab32(x) (__u32)__builtin_bswap32((__u32)(x))
                                                        ^
   In file included from kernel/exit.c:51:
   In file included from include/linux/blkdev.h:25:
   In file included from include/linux/scatterlist.h:9:
   In file included from arch/s390/include/asm/io.h:75:
   include/asm-generic/io.h:501:33: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           __raw_writeb(value, PCI_IOBASE + addr);
                               ~~~~~~~~~~ ^
   include/asm-generic/io.h:511:59: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           __raw_writew((u16 __force)cpu_to_le16(value), PCI_IOBASE + addr);
                                                         ~~~~~~~~~~ ^
   include/asm-generic/io.h:521:59: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           __raw_writel((u32 __force)cpu_to_le32(value), PCI_IOBASE + addr);
                                                         ~~~~~~~~~~ ^
   include/asm-generic/io.h:609:20: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           readsb(PCI_IOBASE + addr, buffer, count);
                  ~~~~~~~~~~ ^
   include/asm-generic/io.h:617:20: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           readsw(PCI_IOBASE + addr, buffer, count);
                  ~~~~~~~~~~ ^
   include/asm-generic/io.h:625:20: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           readsl(PCI_IOBASE + addr, buffer, count);
                  ~~~~~~~~~~ ^
   include/asm-generic/io.h:634:21: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           writesb(PCI_IOBASE + addr, buffer, count);
                   ~~~~~~~~~~ ^
   include/asm-generic/io.h:643:21: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           writesw(PCI_IOBASE + addr, buffer, count);
                   ~~~~~~~~~~ ^
   include/asm-generic/io.h:652:21: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           writesl(PCI_IOBASE + addr, buffer, count);
                   ~~~~~~~~~~ ^
>> kernel/exit.c:780:24: error: too few arguments to function call, single argument 'files' was not specified
           io_uring_files_cancel();
           ~~~~~~~~~~~~~~~~~~~~~ ^
   include/linux/io_uring.h:36:20: note: 'io_uring_files_cancel' declared here
   static inline void io_uring_files_cancel(struct files_struct *files)
                      ^
   12 warnings and 1 error generated.


vim +/files +780 kernel/exit.c

   726	
   727	void __noreturn do_exit(long code)
   728	{
   729		struct task_struct *tsk = current;
   730		int group_dead;
   731	
   732		/*
   733		 * We can get here from a kernel oops, sometimes with preemption off.
   734		 * Start by checking for critical errors.
   735		 * Then fix up important state like USER_DS and preemption.
   736		 * Then do everything else.
   737		 */
   738	
   739		WARN_ON(blk_needs_flush_plug(tsk));
   740	
   741		if (unlikely(in_interrupt()))
   742			panic("Aiee, killing interrupt handler!");
   743		if (unlikely(!tsk->pid))
   744			panic("Attempted to kill the idle task!");
   745	
   746		/*
   747		 * If do_exit is called because this processes oopsed, it's possible
   748		 * that get_fs() was left as KERNEL_DS, so reset it to USER_DS before
   749		 * continuing. Amongst other possible reasons, this is to prevent
   750		 * mm_release()->clear_child_tid() from writing to a user-controlled
   751		 * kernel address.
   752		 */
   753		force_uaccess_begin();
   754	
   755		if (unlikely(in_atomic())) {
   756			pr_info("note: %s[%d] exited with preempt_count %d\n",
   757				current->comm, task_pid_nr(current),
   758				preempt_count());
   759			preempt_count_set(PREEMPT_ENABLED);
   760		}
   761	
   762		profile_task_exit(tsk);
   763		kcov_task_exit(tsk);
   764	
   765		ptrace_event(PTRACE_EVENT_EXIT, code);
   766	
   767		validate_creds_for_do_exit(tsk);
   768	
   769		/*
   770		 * We're taking recursive faults here in do_exit. Safest is to just
   771		 * leave this task alone and wait for reboot.
   772		 */
   773		if (unlikely(tsk->flags & PF_EXITING)) {
   774			pr_alert("Fixing recursive fault but reboot is needed!\n");
   775			futex_exit_recursive(tsk);
   776			set_current_state(TASK_UNINTERRUPTIBLE);
   777			schedule();
   778		}
   779	
 > 780		io_uring_files_cancel();
   781		exit_signals(tsk);  /* sets PF_EXITING */
   782	
   783		/* sync mm's RSS info before statistics gathering */
   784		if (tsk->mm)
   785			sync_mm_rss(tsk->mm);
   786		acct_update_integrals(tsk);
   787		group_dead = atomic_dec_and_test(&tsk->signal->live);
   788		if (group_dead) {
   789			/*
   790			 * If the last thread of global init has exited, panic
   791			 * immediately to get a useable coredump.
   792			 */
   793			if (unlikely(is_global_init(tsk)))
   794				panic("Attempted to kill init! exitcode=0x%08x\n",
   795					tsk->signal->group_exit_code ?: (int)code);
   796	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 28132 bytes --]

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2021-08-12 21:46 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-08-12 21:39 [block:for-5.15/io_uring 43/45] kernel/exit.c:780:24: error: too few arguments to function call, single argument 'files' was not specified kernel test robot
2021-08-12 21:45 kernel test robot

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).