Re: [PATCH 3/9] block: Generalize get_current_ioprio() for any task
From: kernel test robot <hidden>
Date: 2022-06-21 23:59:40
Also in:
llvm
Hi Jan, I love your patch! Perhaps something to improve: [auto build test WARNING on axboe-block/for-next] [also build test WARNING on linus/master v5.19-rc3 next-20220621] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting patch, we suggest to use '--base' as documented in https://git-scm.com/docs/git-format-patch] url: https://github.com/intel-lab-lkp/linux/commits/Jan-Kara/block-Fix-IO-priority-mess/20220621-183235 base: https://git.kernel.org/pub/scm/linux/kernel/git/axboe/linux-block.git for-next config: hexagon-randconfig-r016-20220622 (https://download.01.org/0day-ci/archive/20220622/202206220716.sxn2tinw-lkp@intel.com/config) compiler: clang version 15.0.0 (https://github.com/llvm/llvm-project af6d2a0b6825e71965f3e2701a63c239fa0ad70f) 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 # https://github.com/intel-lab-lkp/linux/commit/8421c851d4fe5f4b9d9d6870ada8ccd0b48a4012 git remote add linux-review https://github.com/intel-lab-lkp/linux git fetch --no-tags linux-review Jan-Kara/block-Fix-IO-priority-mess/20220621-183235 git checkout 8421c851d4fe5f4b9d9d6870ada8ccd0b48a4012 # save the config file mkdir build_dir && cp config build_dir/.config COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=hexagon prepare If you fix the issue, kindly add following tag where applicable Reported-by: kernel test robot <redacted> All warnings (new ones prefixed by >>): In file included from arch/hexagon/kernel/asm-offsets.c:12: In file included from include/linux/compat.h:17: In file included from include/linux/fs.h:38:
quoted
include/linux/ioprio.h:60:27: warning: incompatible pointer to integer conversion passing 'struct task_struct *' to parameter of type 'int' [-Wint-conversion]
return __get_task_ioprio(current);
^~~~~~~
include/asm-generic/current.h:8:17: note: expanded from macro 'current'
#define current get_current()
^~~~~~~~~~~~~
include/asm-generic/current.h:7:23: note: expanded from macro 'get_current'
#define get_current() (current_thread_info()->task)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/ioprio.h:52:41: note: passing argument to parameter 'ioprio' here
static inline int __get_task_ioprio(int ioprio)
^
1 warning generated.
--
In file included from drivers/iio/proximity/isl29501.c:13:
In file included from include/linux/i2c.h:19:
In file included from include/linux/regulator/consumer.h:35:
In file included from include/linux/suspend.h:5:
In file included from include/linux/swap.h:9:
In file included from include/linux/memcontrol.h:13:
In file included from include/linux/cgroup.h:17:
In file included from include/linux/fs.h:38:quoted
include/linux/ioprio.h:60:27: warning: incompatible pointer to integer conversion passing 'struct task_struct *' to parameter of type 'int' [-Wint-conversion]
return __get_task_ioprio(current);
^~~~~~~
include/asm-generic/current.h:8:17: note: expanded from macro 'current'
#define current get_current()
^~~~~~~~~~~~~
include/asm-generic/current.h:7:23: note: expanded from macro 'get_current'
#define get_current() (current_thread_info()->task)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/ioprio.h:52:41: note: passing argument to parameter 'ioprio' here
static inline int __get_task_ioprio(int ioprio)
^
drivers/iio/proximity/isl29501.c:1000:34: warning: unused variable 'isl29501_i2c_matches' [-Wunused-const-variable]
static const struct of_device_id isl29501_i2c_matches[] = {
^
2 warnings generated.
--
In file included from drivers/iio/proximity/sx9500.c:13:
In file included from include/linux/i2c.h:19:
In file included from include/linux/regulator/consumer.h:35:
In file included from include/linux/suspend.h:5:
In file included from include/linux/swap.h:9:
In file included from include/linux/memcontrol.h:13:
In file included from include/linux/cgroup.h:17:
In file included from include/linux/fs.h:38:quoted
include/linux/ioprio.h:60:27: warning: incompatible pointer to integer conversion passing 'struct task_struct *' to parameter of type 'int' [-Wint-conversion]
return __get_task_ioprio(current);
^~~~~~~
include/asm-generic/current.h:8:17: note: expanded from macro 'current'
#define current get_current()
^~~~~~~~~~~~~
include/asm-generic/current.h:7:23: note: expanded from macro 'get_current'
#define get_current() (current_thread_info()->task)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/ioprio.h:52:41: note: passing argument to parameter 'ioprio' here
static inline int __get_task_ioprio(int ioprio)
^
drivers/iio/proximity/sx9500.c:1035:36: warning: unused variable 'sx9500_acpi_match' [-Wunused-const-variable]
static const struct acpi_device_id sx9500_acpi_match[] = {
^
2 warnings generated.
--
In file included from arch/hexagon/kernel/asm-offsets.c:12:
In file included from include/linux/compat.h:17:
In file included from include/linux/fs.h:38:quoted
include/linux/ioprio.h:60:27: warning: incompatible pointer to integer conversion passing 'struct task_struct *' to parameter of type 'int' [-Wint-conversion]
return __get_task_ioprio(current);
^~~~~~~
include/asm-generic/current.h:8:17: note: expanded from macro 'current'
#define current get_current()
^~~~~~~~~~~~~
include/asm-generic/current.h:7:23: note: expanded from macro 'get_current'
#define get_current() (current_thread_info()->task)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/ioprio.h:52:41: note: passing argument to parameter 'ioprio' here
static inline int __get_task_ioprio(int ioprio)
^
1 warning generated.
<stdin>:1517:2: warning: syscall clone3 not implemented [-W#warnings]
#warning syscall clone3 not implemented
^
1 warning generated.
vim +60 include/linux/ioprio.h
57
58 static inline int get_current_ioprio(void)
59 {
> 60 return __get_task_ioprio(current);
61 }
62
--
0-DAY CI Kernel Test Service
https://01.org/lkp