Hi Ze,
FYI, the error/warning still remains.
tree: https://gitee.com/openeuler/kernel.git OLK-5.10
head: b61a367ade32ff42d5f3066b9750e855dc98dd29
commit: 0695e18bbaebbfbe22b049104bbb9a4300a9f30c [2439/2439] mm: Add PMU based memory sampling abstract layer
config: arm64-randconfig-004-20241117 (https://download.01.org/0day-ci/archive/20241204/202412041721.uYCyd8sz-lkp@…)
compiler: aarch64-linux-gcc (GCC) 14.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20241204/202412041721.uYCyd8sz-lkp@…)
If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp(a)intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202412041721.uYCyd8sz-lkp@intel.com/
All warnings (new ones prefixed by >>):
In file included from include/linux/compiler_types.h:65,
from <command-line>:
drivers/arm/spe/spe.c: In function '__arm_spe_dev_probe':
>> include/linux/compiler_attributes.h:221:41: warning: attribute 'fallthrough' not preceding a case label or default label
221 | # define fallthrough __attribute__((__fallthrough__))
| ^~~~~~~~~~~~~
drivers/arm/spe/spe.c:492:17: note: in expansion of macro 'fallthrough'
492 | fallthrough;
| ^~~~~~~~~~~
>> include/linux/compiler_attributes.h:221:41: warning: attribute 'fallthrough' not preceding a case label or default label
221 | # define fallthrough __attribute__((__fallthrough__))
| ^~~~~~~~~~~~~
drivers/arm/spe/spe.c:514:17: note: in expansion of macro 'fallthrough'
514 | fallthrough;
| ^~~~~~~~~~~
Kconfig warnings: (for reference only)
WARNING: unmet direct dependencies detected for ARM_SPE_MEM_SAMPLING
Depends on [n]: ARM64 [=y] && !ARM_SPE_PMU [=y]
Selected by [y]:
- MEM_SAMPLING [=y] && ARM64 [=y]
vim +/fallthrough +221 include/linux/compiler_attributes.h
294f69e662d157 Joe Perches 2019-10-05 209
294f69e662d157 Joe Perches 2019-10-05 210 /*
294f69e662d157 Joe Perches 2019-10-05 211 * Add the pseudo keyword 'fallthrough' so case statement blocks
294f69e662d157 Joe Perches 2019-10-05 212 * must end with any of these keywords:
294f69e662d157 Joe Perches 2019-10-05 213 * break;
294f69e662d157 Joe Perches 2019-10-05 214 * fallthrough;
294f69e662d157 Joe Perches 2019-10-05 215 * goto <label>;
294f69e662d157 Joe Perches 2019-10-05 216 * return [expression];
294f69e662d157 Joe Perches 2019-10-05 217 *
294f69e662d157 Joe Perches 2019-10-05 218 * gcc: https://gcc.gnu.org/onlinedocs/gcc/Statement-Attributes.html#Statement-Attr…
294f69e662d157 Joe Perches 2019-10-05 219 */
294f69e662d157 Joe Perches 2019-10-05 220 #if __has_attribute(__fallthrough__)
294f69e662d157 Joe Perches 2019-10-05 @221 # define fallthrough __attribute__((__fallthrough__))
294f69e662d157 Joe Perches 2019-10-05 222 #else
294f69e662d157 Joe Perches 2019-10-05 223 # define fallthrough do {} while (0) /* fallthrough */
a3f8a30f3f0079 Miguel Ojeda 2018-08-30 224 #endif
a3f8a30f3f0079 Miguel Ojeda 2018-08-30 225
:::::: The code at line 221 was first introduced by commit
:::::: 294f69e662d1570703e9b56e95be37a9fd3afba5 compiler_attributes.h: Add 'fallthrough' pseudo keyword for switch/case use
:::::: TO: Joe Perches <joe(a)perches.com>
:::::: CC: Linus Torvalds <torvalds(a)linux-foundation.org>
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki