Re: [PATCH v2 4/5] cxl: CXL Performance Monitoring Unit driver

From: kernel test robot
Date: Fri Mar 24 2023 - 17:43:47 EST


Hi Jonathan,

I love your patch! Yet something to improve:

[auto build test ERROR on acme/perf/core]
[also build test ERROR on tip/perf/core cxl/next cxl/pending linus/master v6.3-rc3 next-20230324]
[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#_base_tree_information]

url: https://github.com/intel-lab-lkp/linux/commits/Jonathan-Cameron/cxl-Add-function-to-count-regblocks-of-a-given-type/20230325-011827
base: https://git.kernel.org/pub/scm/linux/kernel/git/acme/linux.git perf/core
patch link: https://lore.kernel.org/r/20230324171313.18448-5-Jonathan.Cameron%40huawei.com
patch subject: [PATCH v2 4/5] cxl: CXL Performance Monitoring Unit driver
config: alpha-buildonly-randconfig-r006-20230322 (https://download.01.org/0day-ci/archive/20230325/202303250549.dFFcvnrK-lkp@xxxxxxxxx/config)
compiler: alpha-linux-gcc (GCC) 12.1.0
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/a04f0052a1fa10bda54569c8c7b3ab7fe60ca975
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review Jonathan-Cameron/cxl-Add-function-to-count-regblocks-of-a-given-type/20230325-011827
git checkout a04f0052a1fa10bda54569c8c7b3ab7fe60ca975
# save the config file
mkdir build_dir && cp config build_dir/.config
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 O=build_dir ARCH=alpha olddefconfig
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 O=build_dir ARCH=alpha SHELL=/bin/bash drivers/cxl/

If you fix the issue, kindly add following tag where applicable
| Reported-by: kernel test robot <lkp@xxxxxxxxx>
| Link: https://lore.kernel.org/oe-kbuild-all/202303250549.dFFcvnrK-lkp@xxxxxxxxx/

All error/warnings (new ones prefixed by >>):

| ^~
include/linux/container_of.h:19:33: note: in definition of macro 'container_of'
19 | void *__mptr = (void *)(ptr); \
| ^~~
drivers/cxl/cpmu.c:653:34: note: in expansion of macro 'pmu_to_cpmu_info'
653 | struct cpmu_info *info = pmu_to_cpmu_info(event->pmu);
| ^~~~~~~~~~~~~~~~
drivers/cxl/cpmu.c:653:56: error: 'struct perf_event' has no member named 'pmu'
653 | struct cpmu_info *info = pmu_to_cpmu_info(event->pmu);
| ^~
include/linux/build_bug.h:78:56: note: in definition of macro '__static_assert'
78 | #define __static_assert(expr, msg, ...) _Static_assert(expr, msg)
| ^~~~
include/linux/container_of.h:20:9: note: in expansion of macro 'static_assert'
20 | static_assert(__same_type(*(ptr), ((type *)0)->member) || \
| ^~~~~~~~~~~~~
include/linux/container_of.h:20:23: note: in expansion of macro '__same_type'
20 | static_assert(__same_type(*(ptr), ((type *)0)->member) || \
| ^~~~~~~~~~~
drivers/cxl/cpmu.c:82:32: note: in expansion of macro 'container_of'
82 | #define pmu_to_cpmu_info(_pmu) container_of(_pmu, struct cpmu_info, pmu)
| ^~~~~~~~~~~~
drivers/cxl/cpmu.c:653:34: note: in expansion of macro 'pmu_to_cpmu_info'
653 | struct cpmu_info *info = pmu_to_cpmu_info(event->pmu);
| ^~~~~~~~~~~~~~~~
drivers/cxl/cpmu.c:653:56: error: 'struct perf_event' has no member named 'pmu'
653 | struct cpmu_info *info = pmu_to_cpmu_info(event->pmu);
| ^~
include/linux/build_bug.h:78:56: note: in definition of macro '__static_assert'
78 | #define __static_assert(expr, msg, ...) _Static_assert(expr, msg)
| ^~~~
include/linux/container_of.h:20:9: note: in expansion of macro 'static_assert'
20 | static_assert(__same_type(*(ptr), ((type *)0)->member) || \
| ^~~~~~~~~~~~~
include/linux/container_of.h:21:23: note: in expansion of macro '__same_type'
21 | __same_type(*(ptr), void), \
| ^~~~~~~~~~~
drivers/cxl/cpmu.c:82:32: note: in expansion of macro 'container_of'
82 | #define pmu_to_cpmu_info(_pmu) container_of(_pmu, struct cpmu_info, pmu)
| ^~~~~~~~~~~~
drivers/cxl/cpmu.c:653:34: note: in expansion of macro 'pmu_to_cpmu_info'
653 | struct cpmu_info *info = pmu_to_cpmu_info(event->pmu);
| ^~~~~~~~~~~~~~~~
include/linux/compiler_types.h:338:27: error: expression in static assertion is not an integer
338 | #define __same_type(a, b) __builtin_types_compatible_p(typeof(a), typeof(b))
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/build_bug.h:78:56: note: in definition of macro '__static_assert'
78 | #define __static_assert(expr, msg, ...) _Static_assert(expr, msg)
| ^~~~
include/linux/container_of.h:20:9: note: in expansion of macro 'static_assert'
20 | static_assert(__same_type(*(ptr), ((type *)0)->member) || \
| ^~~~~~~~~~~~~
include/linux/container_of.h:20:23: note: in expansion of macro '__same_type'
20 | static_assert(__same_type(*(ptr), ((type *)0)->member) || \
| ^~~~~~~~~~~
drivers/cxl/cpmu.c:82:32: note: in expansion of macro 'container_of'
82 | #define pmu_to_cpmu_info(_pmu) container_of(_pmu, struct cpmu_info, pmu)
| ^~~~~~~~~~~~
drivers/cxl/cpmu.c:653:34: note: in expansion of macro 'pmu_to_cpmu_info'
653 | struct cpmu_info *info = pmu_to_cpmu_info(event->pmu);
| ^~~~~~~~~~~~~~~~
drivers/cxl/cpmu.c:654:43: error: 'struct perf_event' has no member named 'hw'
654 | struct hw_perf_event *hwc = &event->hw;
| ^~
drivers/cxl/cpmu.c:658:45: error: 'struct hw_perf_event' has no member named 'prev_count'
658 | prev_cnt = local64_read(&hwc->prev_count);
| ^~
arch/alpha/include/asm/local.h:14:44: note: in definition of macro 'local_read'
14 | #define local_read(l) atomic_long_read(&(l)->a)
| ^
drivers/cxl/cpmu.c:658:28: note: in expansion of macro 'local64_read'
658 | prev_cnt = local64_read(&hwc->prev_count);
| ^~~~~~~~~~~~
In file included from include/linux/atomic.h:82,
from include/linux/rcupdate.h:25,
from include/linux/rculist.h:11:
drivers/cxl/cpmu.c:660:38: error: 'struct hw_perf_event' has no member named 'prev_count'
660 | } while (local64_cmpxchg(&hwc->prev_count, prev_cnt, new_cnt) != prev_cnt);
| ^~
include/linux/atomic/atomic-instrumented.h:2049:16: note: in definition of macro 'cmpxchg_local'
2049 | typeof(ptr) __ai_ptr = (ptr); \
| ^~~
include/asm-generic/local64.h:45:34: note: in expansion of macro 'local_cmpxchg'
45 | #define local64_cmpxchg(l, o, n) local_cmpxchg((&(l)->a), (o), (n))
| ^~~~~~~~~~~~~
drivers/cxl/cpmu.c:660:18: note: in expansion of macro 'local64_cmpxchg'
660 | } while (local64_cmpxchg(&hwc->prev_count, prev_cnt, new_cnt) != prev_cnt);
| ^~~~~~~~~~~~~~~
drivers/cxl/cpmu.c:660:38: error: 'struct hw_perf_event' has no member named 'prev_count'
660 | } while (local64_cmpxchg(&hwc->prev_count, prev_cnt, new_cnt) != prev_cnt);
| ^~
include/linux/atomic/atomic-instrumented.h:2049:33: note: in definition of macro 'cmpxchg_local'
2049 | typeof(ptr) __ai_ptr = (ptr); \
| ^~~
include/asm-generic/local64.h:45:34: note: in expansion of macro 'local_cmpxchg'
45 | #define local64_cmpxchg(l, o, n) local_cmpxchg((&(l)->a), (o), (n))
| ^~~~~~~~~~~~~
drivers/cxl/cpmu.c:660:18: note: in expansion of macro 'local64_cmpxchg'
660 | } while (local64_cmpxchg(&hwc->prev_count, prev_cnt, new_cnt) != prev_cnt);
| ^~~~~~~~~~~~~~~
>> include/linux/atomic/atomic-instrumented.h:2050:50: error: invalid type argument of unary '*' (have 'int')
2050 | instrument_atomic_write(__ai_ptr, sizeof(*__ai_ptr)); \
| ^~~~~~~~~
arch/alpha/include/asm/local.h:56:10: note: in expansion of macro 'cmpxchg_local'
56 | (cmpxchg_local(&((l)->a.counter), (o), (n)))
| ^~~~~~~~~~~~~
include/asm-generic/local64.h:45:34: note: in expansion of macro 'local_cmpxchg'
45 | #define local64_cmpxchg(l, o, n) local_cmpxchg((&(l)->a), (o), (n))
| ^~~~~~~~~~~~~
drivers/cxl/cpmu.c:660:18: note: in expansion of macro 'local64_cmpxchg'
660 | } while (local64_cmpxchg(&hwc->prev_count, prev_cnt, new_cnt) != prev_cnt);
| ^~~~~~~~~~~~~~~
>> include/linux/atomic/atomic-instrumented.h:2050:33: warning: passing argument 1 of 'instrument_atomic_write' makes pointer from integer without a cast [-Wint-conversion]
2050 | instrument_atomic_write(__ai_ptr, sizeof(*__ai_ptr)); \
| ^~~~~~~~
| |
| int
arch/alpha/include/asm/local.h:56:10: note: in expansion of macro 'cmpxchg_local'
56 | (cmpxchg_local(&((l)->a.counter), (o), (n)))
| ^~~~~~~~~~~~~
include/asm-generic/local64.h:45:34: note: in expansion of macro 'local_cmpxchg'
45 | #define local64_cmpxchg(l, o, n) local_cmpxchg((&(l)->a), (o), (n))
| ^~~~~~~~~~~~~
drivers/cxl/cpmu.c:660:18: note: in expansion of macro 'local64_cmpxchg'
660 | } while (local64_cmpxchg(&hwc->prev_count, prev_cnt, new_cnt) != prev_cnt);
| ^~~~~~~~~~~~~~~
In file included from include/linux/atomic/atomic-instrumented.h:22:
include/linux/instrumented.h:85:74: note: expected 'const volatile void *' but argument is of type 'int'
85 | static __always_inline void instrument_atomic_write(const volatile void *v, size_t size)
| ~~~~~~~~~~~~~~~~~~~~~^
In file included from arch/alpha/include/asm/atomic.h:7,
from include/linux/atomic.h:7:
>> arch/alpha/include/asm/cmpxchg.h:22:20: error: invalid type argument of unary '*' (have 'int')
22 | __typeof__(*(ptr)) _o_ = (o); \
| ^~~~~~
include/linux/atomic/atomic-instrumented.h:2051:9: note: in expansion of macro 'arch_cmpxchg_local'
2051 | arch_cmpxchg_local(__ai_ptr, __VA_ARGS__); \
| ^~~~~~~~~~~~~~~~~~
arch/alpha/include/asm/local.h:56:10: note: in expansion of macro 'cmpxchg_local'
56 | (cmpxchg_local(&((l)->a.counter), (o), (n)))
| ^~~~~~~~~~~~~
include/asm-generic/local64.h:45:34: note: in expansion of macro 'local_cmpxchg'
45 | #define local64_cmpxchg(l, o, n) local_cmpxchg((&(l)->a), (o), (n))
| ^~~~~~~~~~~~~
drivers/cxl/cpmu.c:660:18: note: in expansion of macro 'local64_cmpxchg'
660 | } while (local64_cmpxchg(&hwc->prev_count, prev_cnt, new_cnt) != prev_cnt);
| ^~~~~~~~~~~~~~~
arch/alpha/include/asm/cmpxchg.h:23:20: error: invalid type argument of unary '*' (have 'int')
23 | __typeof__(*(ptr)) _n_ = (n); \
| ^~~~~~
include/linux/atomic/atomic-instrumented.h:2051:9: note: in expansion of macro 'arch_cmpxchg_local'
2051 | arch_cmpxchg_local(__ai_ptr, __VA_ARGS__); \
| ^~~~~~~~~~~~~~~~~~
arch/alpha/include/asm/local.h:56:10: note: in expansion of macro 'cmpxchg_local'
56 | (cmpxchg_local(&((l)->a.counter), (o), (n)))
| ^~~~~~~~~~~~~
include/asm-generic/local64.h:45:34: note: in expansion of macro 'local_cmpxchg'
45 | #define local64_cmpxchg(l, o, n) local_cmpxchg((&(l)->a), (o), (n))
| ^~~~~~~~~~~~~
drivers/cxl/cpmu.c:660:18: note: in expansion of macro 'local64_cmpxchg'
660 | } while (local64_cmpxchg(&hwc->prev_count, prev_cnt, new_cnt) != prev_cnt);
| ^~~~~~~~~~~~~~~
arch/alpha/include/asm/cmpxchg.h:24:21: error: invalid type argument of unary '*' (have 'int')
24 | (__typeof__(*(ptr))) __cmpxchg_local((ptr), (unsigned long)_o_, \
| ^~~~~~
include/linux/atomic/atomic-instrumented.h:2051:9: note: in expansion of macro 'arch_cmpxchg_local'
2051 | arch_cmpxchg_local(__ai_ptr, __VA_ARGS__); \
| ^~~~~~~~~~~~~~~~~~
arch/alpha/include/asm/local.h:56:10: note: in expansion of macro 'cmpxchg_local'
56 | (cmpxchg_local(&((l)->a.counter), (o), (n)))
| ^~~~~~~~~~~~~
include/asm-generic/local64.h:45:34: note: in expansion of macro 'local_cmpxchg'
45 | #define local64_cmpxchg(l, o, n) local_cmpxchg((&(l)->a), (o), (n))
| ^~~~~~~~~~~~~
drivers/cxl/cpmu.c:660:18: note: in expansion of macro 'local64_cmpxchg'
660 | } while (local64_cmpxchg(&hwc->prev_count, prev_cnt, new_cnt) != prev_cnt);
| ^~~~~~~~~~~~~~~
arch/alpha/include/asm/cmpxchg.h:26:50: error: invalid type argument of unary '*' (have 'int')
26 | sizeof(*(ptr))); \
| ^~~~~~
include/linux/atomic/atomic-instrumented.h:2051:9: note: in expansion of macro 'arch_cmpxchg_local'
2051 | arch_cmpxchg_local(__ai_ptr, __VA_ARGS__); \
| ^~~~~~~~~~~~~~~~~~
arch/alpha/include/asm/local.h:56:10: note: in expansion of macro 'cmpxchg_local'
56 | (cmpxchg_local(&((l)->a.counter), (o), (n)))
| ^~~~~~~~~~~~~
include/asm-generic/local64.h:45:34: note: in expansion of macro 'local_cmpxchg'
45 | #define local64_cmpxchg(l, o, n) local_cmpxchg((&(l)->a), (o), (n))
| ^~~~~~~~~~~~~
drivers/cxl/cpmu.c:660:18: note: in expansion of macro 'local64_cmpxchg'
660 | } while (local64_cmpxchg(&hwc->prev_count, prev_cnt, new_cnt) != prev_cnt);
| ^~~~~~~~~~~~~~~
>> arch/alpha/include/asm/cmpxchg.h:24:46: warning: passing argument 1 of '__cmpxchg_local' makes pointer from integer without a cast [-Wint-conversion]
24 | (__typeof__(*(ptr))) __cmpxchg_local((ptr), (unsigned long)_o_, \
| ^~~~~
| |
| int
include/linux/atomic/atomic-instrumented.h:2051:9: note: in expansion of macro 'arch_cmpxchg_local'
2051 | arch_cmpxchg_local(__ai_ptr, __VA_ARGS__); \
| ^~~~~~~~~~~~~~~~~~
arch/alpha/include/asm/local.h:56:10: note: in expansion of macro 'cmpxchg_local'
56 | (cmpxchg_local(&((l)->a.counter), (o), (n)))
| ^~~~~~~~~~~~~
include/asm-generic/local64.h:45:34: note: in expansion of macro 'local_cmpxchg'
45 | #define local64_cmpxchg(l, o, n) local_cmpxchg((&(l)->a), (o), (n))
| ^~~~~~~~~~~~~
drivers/cxl/cpmu.c:660:18: note: in expansion of macro 'local64_cmpxchg'
660 | } while (local64_cmpxchg(&hwc->prev_count, prev_cnt, new_cnt) != prev_cnt);
| ^~~~~~~~~~~~~~~
arch/alpha/include/asm/xchg.h:229:30: note: expected 'volatile void *' but argument is of type 'int'
229 | ____cmpxchg(, volatile void *ptr, unsigned long old, unsigned long new,
| ~~~~~~~~~~~~~~~^~~
arch/alpha/include/asm/cmpxchg.h:10:69: note: in definition of macro '____cmpxchg'
10 | #define ____cmpxchg(type, args...) __cmpxchg ## type ## _local(args)
| ^~~~
drivers/cxl/cpmu.c:670:34: error: 'struct perf_event' has no member named 'count'
670 | local64_add(delta, &event->count);
| ^~
arch/alpha/include/asm/local.h:18:48: note: in definition of macro 'local_add'
18 | #define local_add(i,l) atomic_long_add((i),(&(l)->a))
| ^
drivers/cxl/cpmu.c:670:9: note: in expansion of macro 'local64_add'
670 | local64_add(delta, &event->count);
| ^~~~~~~~~~~
drivers/cxl/cpmu.c: In function 'cpmu_event_stop':
drivers/cxl/cpmu.c:680:56: error: 'struct perf_event' has no member named 'pmu'
680 | struct cpmu_info *info = pmu_to_cpmu_info(event->pmu);
| ^~
include/linux/container_of.h:19:33: note: in definition of macro 'container_of'
19 | void *__mptr = (void *)(ptr); \
| ^~~
drivers/cxl/cpmu.c:680:34: note: in expansion of macro 'pmu_to_cpmu_info'
680 | struct cpmu_info *info = pmu_to_cpmu_info(event->pmu);
| ^~~~~~~~~~~~~~~~
drivers/cxl/cpmu.c:680:56: error: 'struct perf_event' has no member named 'pmu'
680 | struct cpmu_info *info = pmu_to_cpmu_info(event->pmu);
| ^~
include/linux/build_bug.h:78:56: note: in definition of macro '__static_assert'
78 | #define __static_assert(expr, msg, ...) _Static_assert(expr, msg)
| ^~~~
include/linux/container_of.h:20:9: note: in expansion of macro 'static_assert'
20 | static_assert(__same_type(*(ptr), ((type *)0)->member) || \
| ^~~~~~~~~~~~~
include/linux/container_of.h:20:23: note: in expansion of macro '__same_type'
20 | static_assert(__same_type(*(ptr), ((type *)0)->member) || \
| ^~~~~~~~~~~
drivers/cxl/cpmu.c:82:32: note: in expansion of macro 'container_of'
82 | #define pmu_to_cpmu_info(_pmu) container_of(_pmu, struct cpmu_info, pmu)
| ^~~~~~~~~~~~
drivers/cxl/cpmu.c:680:34: note: in expansion of macro 'pmu_to_cpmu_info'
680 | struct cpmu_info *info = pmu_to_cpmu_info(event->pmu);
| ^~~~~~~~~~~~~~~~
drivers/cxl/cpmu.c:680:56: error: 'struct perf_event' has no member named 'pmu'
680 | struct cpmu_info *info = pmu_to_cpmu_info(event->pmu);
| ^~
include/linux/build_bug.h:78:56: note: in definition of macro '__static_assert'
78 | #define __static_assert(expr, msg, ...) _Static_assert(expr, msg)
| ^~~~
include/linux/container_of.h:20:9: note: in expansion of macro 'static_assert'
20 | static_assert(__same_type(*(ptr), ((type *)0)->member) || \
| ^~~~~~~~~~~~~
include/linux/container_of.h:21:23: note: in expansion of macro '__same_type'
21 | __same_type(*(ptr), void), \
| ^~~~~~~~~~~
drivers/cxl/cpmu.c:82:32: note: in expansion of macro 'container_of'
82 | #define pmu_to_cpmu_info(_pmu) container_of(_pmu, struct cpmu_info, pmu)
| ^~~~~~~~~~~~
drivers/cxl/cpmu.c:680:34: note: in expansion of macro 'pmu_to_cpmu_info'
680 | struct cpmu_info *info = pmu_to_cpmu_info(event->pmu);
| ^~~~~~~~~~~~~~~~
include/linux/compiler_types.h:338:27: error: expression in static assertion is not an integer
338 | #define __same_type(a, b) __builtin_types_compatible_p(typeof(a), typeof(b))
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/build_bug.h:78:56: note: in definition of macro '__static_assert'
78 | #define __static_assert(expr, msg, ...) _Static_assert(expr, msg)
| ^~~~
include/linux/container_of.h:20:9: note: in expansion of macro 'static_assert'
20 | static_assert(__same_type(*(ptr), ((type *)0)->member) || \
| ^~~~~~~~~~~~~
include/linux/container_of.h:20:23: note: in expansion of macro '__same_type'
20 | static_assert(__same_type(*(ptr), ((type *)0)->member) || \
| ^~~~~~~~~~~
drivers/cxl/cpmu.c:82:32: note: in expansion of macro 'container_of'
82 | #define pmu_to_cpmu_info(_pmu) container_of(_pmu, struct cpmu_info, pmu)
| ^~~~~~~~~~~~
drivers/cxl/cpmu.c:680:34: note: in expansion of macro 'pmu_to_cpmu_info'
680 | struct cpmu_info *info = pmu_to_cpmu_info(event->pmu);
| ^~~~~~~~~~~~~~~~
drivers/cxl/cpmu.c:682:43: error: 'struct perf_event' has no member named 'hw'
682 | struct hw_perf_event *hwc = &event->hw;
| ^~
drivers/cxl/cpmu.c:686:25: error: 'struct hw_perf_event' has no member named 'state'
686 | WARN_ON_ONCE(hwc->state & PERF_HES_STOPPED);


vim +2050 include/linux/atomic/atomic-instrumented.h

0aa7be05d83cc5 include/linux/atomic/atomic-instrumented.h Uros Bizjak 2022-05-15 2046
aa525d063851a9 include/asm-generic/atomic-instrumented.h Mark Rutland 2018-09-04 2047 #define cmpxchg_local(ptr, ...) \
b06ed71a624ba0 include/asm-generic/atomic-instrumented.h Dmitry Vyukov 2018-01-29 2048 ({ \
df79ed2c064363 include/asm-generic/atomic-instrumented.h Mark Rutland 2018-07-16 2049 typeof(ptr) __ai_ptr = (ptr); \
ed8af2e4d2a71b include/asm-generic/atomic-instrumented.h Marco Elver 2020-01-21 @2050 instrument_atomic_write(__ai_ptr, sizeof(*__ai_ptr)); \
aa525d063851a9 include/asm-generic/atomic-instrumented.h Mark Rutland 2018-09-04 2051 arch_cmpxchg_local(__ai_ptr, __VA_ARGS__); \
b06ed71a624ba0 include/asm-generic/atomic-instrumented.h Dmitry Vyukov 2018-01-29 2052 })
b06ed71a624ba0 include/asm-generic/atomic-instrumented.h Dmitry Vyukov 2018-01-29 2053

--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests