:::::: :::::: Manual check reason: "low confidence static check warning: block/holder.c:10:72: warning: Parameter 'bdev' can be declared with const [constParameter]" ::::::
CC: [email protected] BCC: [email protected] CC: [email protected] TO: Christoph Hellwig <[email protected]> CC: Jens Axboe <[email protected]> tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: b047602d579b4fb028128a525f056bbdc890e7f0 commit: 0dbcfe247f22a6d73302dfa691c48b3c14d31c4c block: look up holders by bdev date: 11 months ago :::::: branch date: 19 hours ago :::::: commit date: 11 months ago compiler: arceb-elf-gcc (GCC) 11.3.0 reproduce (cppcheck warning): # apt-get install cppcheck git checkout 0dbcfe247f22a6d73302dfa691c48b3c14d31c4c cppcheck --quiet --enable=style,performance,portability --template=gcc FILE If you fix the issue, kindly add following tag where applicable Reported-by: kernel test robot <[email protected]> cppcheck possible warnings: (new ones prefixed by >>, may not real problems) >> block/holder.c:10:72: warning: Parameter 'bdev' can be declared with const >> [constParameter] static struct bd_holder_disk *bd_find_holder_disk(struct block_device *bdev, ^ vim +/bdev +10 block/holder.c c66fd019713e9c Christoph Hellwig 2021-08-04 9 c66fd019713e9c Christoph Hellwig 2021-08-04 @10 static struct bd_holder_disk *bd_find_holder_disk(struct block_device *bdev, c66fd019713e9c Christoph Hellwig 2021-08-04 11 struct gendisk *disk) c66fd019713e9c Christoph Hellwig 2021-08-04 12 { c66fd019713e9c Christoph Hellwig 2021-08-04 13 struct bd_holder_disk *holder; c66fd019713e9c Christoph Hellwig 2021-08-04 14 0dbcfe247f22a6 Christoph Hellwig 2021-08-04 15 list_for_each_entry(holder, &disk->slave_bdevs, list) 0dbcfe247f22a6 Christoph Hellwig 2021-08-04 16 if (holder->bdev == bdev) c66fd019713e9c Christoph Hellwig 2021-08-04 17 return holder; c66fd019713e9c Christoph Hellwig 2021-08-04 18 return NULL; c66fd019713e9c Christoph Hellwig 2021-08-04 19 } c66fd019713e9c Christoph Hellwig 2021-08-04 20 :::::: The code at line 10 was first introduced by commit :::::: c66fd019713e9cf7d6f1243c378cd177d01fe18a block: make the block holder code optional :::::: TO: Christoph Hellwig <[email protected]> :::::: CC: Jens Axboe <[email protected]> -- 0-DAY CI Kernel Test Service https://01.org/lkp _______________________________________________ kbuild mailing list -- [email protected] To unsubscribe send an email to [email protected]
