Hi Sudarsana, Thank you for the patch! Yet something to improve:
[auto build test ERROR on net-next/master] url: https://github.com/0day-ci/linux/commits/Sudarsana-Reddy-Kalluru/qedr-Avoid-DB-recover-entry-deletion-when-device-is-not-active/20200501-033426 base: https://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git 323e395f197fd0bc1bf7a7a7e4c67d3d7410f24f config: riscv-allyesconfig (attached as .config) compiler: riscv64-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 # save the attached .config to linux build tree COMPILER_INSTALL_PATH=$HOME/0day GCC_VERSION=9.3.0 make.cross ARCH=riscv If you fix the issue, kindly add following tag as appropriate Reported-by: kbuild test robot <l...@intel.com> All error/warnings (new ones prefixed by >>): drivers/infiniband/hw/qedr/verbs.c: In function 'qedr_db_recovery_del': >> drivers/infiniband/hw/qedr/verbs.c:675:6: error: implicit declaration of >> function 'QEDR_DEAD' [-Werror=implicit-function-declaration] 675 | if (QEDR_DEAD(dev)) { | ^~~~~~~~~ In file included from include/linux/string.h:6, from include/linux/dma-mapping.h:6, from drivers/infiniband/hw/qedr/verbs.c:32: >> drivers/infiniband/hw/qedr/verbs.c:676:19: error: 'QEDR_MSG_FAIL' undeclared >> (first use in this function); did you mean 'QEDR_MSG_INIT'? 676 | DP_VERBOSE(dev, QEDR_MSG_FAIL, | ^~~~~~~~~~~~~ include/linux/compiler.h:78:42: note: in definition of macro 'unlikely' 78 | # define unlikely(x) __builtin_expect(!!(x), 0) | ^ >> drivers/infiniband/hw/qedr/verbs.c:676:3: note: in expansion of macro >> 'DP_VERBOSE' 676 | DP_VERBOSE(dev, QEDR_MSG_FAIL, | ^~~~~~~~~~ drivers/infiniband/hw/qedr/verbs.c:676:19: note: each undeclared identifier is reported only once for each function it appears in 676 | DP_VERBOSE(dev, QEDR_MSG_FAIL, | ^~~~~~~~~~~~~ include/linux/compiler.h:78:42: note: in definition of macro 'unlikely' 78 | # define unlikely(x) __builtin_expect(!!(x), 0) | ^ >> drivers/infiniband/hw/qedr/verbs.c:676:3: note: in expansion of macro >> 'DP_VERBOSE' 676 | DP_VERBOSE(dev, QEDR_MSG_FAIL, | ^~~~~~~~~~ >> drivers/infiniband/hw/qedr/verbs.c:678:10: warning: 'return' with a value, >> in function returning void [-Wreturn-type] 678 | return 0; | ^ drivers/infiniband/hw/qedr/verbs.c:671:13: note: declared here 671 | static void qedr_db_recovery_del(struct qedr_dev *dev, | ^~~~~~~~~~~~~~~~~~~~ cc1: some warnings being treated as errors vim +/QEDR_DEAD +675 drivers/infiniband/hw/qedr/verbs.c 670 671 static void qedr_db_recovery_del(struct qedr_dev *dev, 672 void __iomem *db_addr, 673 void *db_data) 674 { > 675 if (QEDR_DEAD(dev)) { > 676 DP_VERBOSE(dev, QEDR_MSG_FAIL, 677 "avoiding db rec since device is dead\n"); > 678 return 0; 679 } 680 681 if (!db_data) { 682 DP_DEBUG(dev, QEDR_MSG_INIT, "avoiding db rec since old lib\n"); 683 return; 684 } 685 686 /* Ignore return code as there is not much we can do about it. Error 687 * log will be printed inside. 688 */ 689 dev->ops->common->db_recovery_del(dev->cdev, db_addr, db_data); 690 } 691 --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-...@lists.01.org
.config.gz
Description: application/gzip
