Hi Nicholas, I love your patch! Perhaps something to improve:
[auto build test WARNING on powerpc/next] [also build test WARNING on v5.12-rc5 next-20210401] [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/0day-ci/linux/commits/Nicholas-Piggin/KVM-PPC-Book3S-C-ify-the-P9-entry-exit-code/20210401-232743 base: https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git next config: powerpc-randconfig-s031-20210401 (attached as .config) compiler: powerpc64-linux-gcc (GCC) 9.3.0 reproduce: wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # apt-get install sparse # sparse version: v0.6.3-279-g6d5d9b42-dirty # https://github.com/0day-ci/linux/commit/4c3e837b20a4ebe406d7b8fddbec2f6fa86755a2 git remote add linux-review https://github.com/0day-ci/linux git fetch --no-tags linux-review Nicholas-Piggin/KVM-PPC-Book3S-C-ify-the-P9-entry-exit-code/20210401-232743 git checkout 4c3e837b20a4ebe406d7b8fddbec2f6fa86755a2 # save the attached .config to linux build tree COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' ARCH=powerpc If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <l...@intel.com> sparse warnings: (new ones prefixed by >>) >> arch/powerpc/kernel/time.c:877:9: sparse: sparse: incorrect type in >> initializer (different address spaces) @@ expected void const [noderef] >> __percpu *__vpp_verify @@ got unsigned long long * @@ arch/powerpc/kernel/time.c:877:9: sparse: expected void const [noderef] __percpu *__vpp_verify arch/powerpc/kernel/time.c:877:9: sparse: got unsigned long long * >> arch/powerpc/kernel/time.c:877:9: sparse: sparse: incorrect type in >> initializer (different address spaces) @@ expected void const [noderef] >> __percpu *__vpp_verify @@ got unsigned long long * @@ arch/powerpc/kernel/time.c:877:9: sparse: expected void const [noderef] __percpu *__vpp_verify arch/powerpc/kernel/time.c:877:9: sparse: got unsigned long long * >> arch/powerpc/kernel/time.c:877:9: sparse: sparse: incorrect type in >> initializer (different address spaces) @@ expected void const [noderef] >> __percpu *__vpp_verify @@ got unsigned long long * @@ arch/powerpc/kernel/time.c:877:9: sparse: expected void const [noderef] __percpu *__vpp_verify arch/powerpc/kernel/time.c:877:9: sparse: got unsigned long long * >> arch/powerpc/kernel/time.c:877:9: sparse: sparse: incorrect type in >> initializer (different address spaces) @@ expected void const [noderef] >> __percpu *__vpp_verify @@ got unsigned long long * @@ arch/powerpc/kernel/time.c:877:9: sparse: expected void const [noderef] __percpu *__vpp_verify arch/powerpc/kernel/time.c:877:9: sparse: got unsigned long long * >> arch/powerpc/kernel/time.c:877:9: sparse: sparse: incorrect type in >> initializer (different address spaces) @@ expected void const [noderef] >> __percpu *__vpp_verify @@ got unsigned long long * @@ arch/powerpc/kernel/time.c:877:9: sparse: expected void const [noderef] __percpu *__vpp_verify arch/powerpc/kernel/time.c:877:9: sparse: got unsigned long long * arch/powerpc/kernel/time.c:130:1: sparse: sparse: symbol 'rtc_lock' was not declared. Should it be static? arch/powerpc/kernel/time.c:649:24: sparse: sparse: incorrect type in initializer (different address spaces) @@ expected void const [noderef] __percpu *__vpp_verify @@ got unsigned long long * @@ arch/powerpc/kernel/time.c:649:24: sparse: expected void const [noderef] __percpu *__vpp_verify arch/powerpc/kernel/time.c:649:24: sparse: got unsigned long long * arch/powerpc/kernel/time.c:580:24: sparse: sparse: incorrect type in initializer (different address spaces) @@ expected void const [noderef] __percpu *__vpp_verify @@ got unsigned long long * @@ arch/powerpc/kernel/time.c:580:24: sparse: expected void const [noderef] __percpu *__vpp_verify arch/powerpc/kernel/time.c:580:24: sparse: got unsigned long long * vim +877 arch/powerpc/kernel/time.c 4a4cfe3836916e Tony Breeds 2007-09-22 873 d831d0b83f2058 Tony Breeds 2007-09-21 874 static int decrementer_set_next_event(unsigned long evt, d831d0b83f2058 Tony Breeds 2007-09-21 875 struct clock_event_device *dev) d831d0b83f2058 Tony Breeds 2007-09-21 876 { 6601ec1c2ba929 Christophe Leroy 2020-09-29 @877 __this_cpu_write(decrementers_next_tb, get_tb() + evt); d831d0b83f2058 Tony Breeds 2007-09-21 878 set_dec(evt); 0215f7d8c53fb1 Benjamin Herrenschmidt 2014-01-14 879 0215f7d8c53fb1 Benjamin Herrenschmidt 2014-01-14 880 /* We may have raced with new irq work */ 0215f7d8c53fb1 Benjamin Herrenschmidt 2014-01-14 881 if (test_irq_work_pending()) 0215f7d8c53fb1 Benjamin Herrenschmidt 2014-01-14 882 set_dec(1); 0215f7d8c53fb1 Benjamin Herrenschmidt 2014-01-14 883 d831d0b83f2058 Tony Breeds 2007-09-21 884 return 0; d831d0b83f2058 Tony Breeds 2007-09-21 885 } d831d0b83f2058 Tony Breeds 2007-09-21 886 --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-...@lists.01.org
.config.gz
Description: application/gzip
