[kbuild-all] [ti:ti-linux-5.4.y 2795/9671] drivers/pci/endpoint/pci-epf-core.c:133 pci_epf_add_vepf() warn: inconsistent returns 'epf_pf->lock'.

tree: git://git.ti.com/ti-linux-kernel/ti-linux-kernel.git ti-linux-5.4.y head: 5fd9bb3f78d32f23d809e9c2a689925c7bd774f9 commit: 46241066755af938ddbf829f86ae15b2eebaa3f5 [2795/9671] PCI: endpoint: Add support to add virtual function in endpoint core config: x86_64-randconfig-m001-20200913 (attached as .config) compiler: gcc-9 (Debian 9.3.0-15) 9.3.0 If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <lkp(a)intel.com&gt; Reported-by: Dan Carpenter <dan.carpenter(a)oracle.com&gt; smatch warnings: drivers/pci/endpoint/pci-epf-core.c:133 pci_epf_add_vepf() warn: inconsistent returns 'epf_pf->lock'. git remote add ti git://git.ti.com/ti-linux-kernel/ti-linux-kernel.git git fetch --no-tags ti ti-linux-5.4.y git checkout 46241066755af938ddbf829f86ae15b2eebaa3f5 vim +133 drivers/pci/endpoint/pci-epf-core.c 46241066755af93 Kishon Vijay Abraham I 2020-01-23 108 int pci_epf_add_vepf(struct pci_epf *epf_pf, struct pci_epf *epf_vf) 46241066755af93 Kishon Vijay Abraham I 2020-01-23 109 { 46241066755af93 Kishon Vijay Abraham I 2020-01-23 110 u32 vfunc_no; 46241066755af93 Kishon Vijay Abraham I 2020-01-23 111 46241066755af93 Kishon Vijay Abraham I 2020-01-23 112 if (IS_ERR_OR_NULL(epf_pf) || IS_ERR_OR_NULL(epf_vf)) 46241066755af93 Kishon Vijay Abraham I 2020-01-23 113 return -EINVAL; 46241066755af93 Kishon Vijay Abraham I 2020-01-23 114 46241066755af93 Kishon Vijay Abraham I 2020-01-23 115 if (epf_pf->epc || epf_vf->epc || epf_vf->epf_pf) 46241066755af93 Kishon Vijay Abraham I 2020-01-23 116 return -EBUSY; 46241066755af93 Kishon Vijay Abraham I 2020-01-23 117 46241066755af93 Kishon Vijay Abraham I 2020-01-23 118 mutex_lock(&epf_pf->lock); ^^^^^^^^^^^^^^^^^^^^^^^^^ 46241066755af93 Kishon Vijay Abraham I 2020-01-23 119 vfunc_no = find_first_zero_bit(&epf_pf->vfunction_num_map, 46241066755af93 Kishon Vijay Abraham I 2020-01-23 120 BITS_PER_LONG); 46241066755af93 Kishon Vijay Abraham I 2020-01-23 121 if (vfunc_no >= BITS_PER_LONG) 46241066755af93 Kishon Vijay Abraham I 2020-01-23 122 return -EINVAL; ^^^^^^^^^^^^^^ ret = -EINVAL; goto unlock? 46241066755af93 Kishon Vijay Abraham I 2020-01-23 123 46241066755af93 Kishon Vijay Abraham I 2020-01-23 124 set_bit(vfunc_no, &epf_pf->vfunction_num_map); 46241066755af93 Kishon Vijay Abraham I 2020-01-23 125 epf_vf->vfunc_no = vfunc_no; 46241066755af93 Kishon Vijay Abraham I 2020-01-23 126 46241066755af93 Kishon Vijay Abraham I 2020-01-23 127 epf_vf->epf_pf = epf_pf; 46241066755af93 Kishon Vijay Abraham I 2020-01-23 128 epf_vf->is_vf = true; 46241066755af93 Kishon Vijay Abraham I 2020-01-23 129 46241066755af93 Kishon Vijay Abraham I 2020-01-23 130 list_add_tail(&epf_vf->list, &epf_pf->pci_vepf); 46241066755af93 Kishon Vijay Abraham I 2020-01-23 131 mutex_unlock(&epf_pf->lock); 46241066755af93 Kishon Vijay Abraham I 2020-01-23 132 46241066755af93 Kishon Vijay Abraham I 2020-01-23 @133 return 0; 46241066755af93 Kishon Vijay Abraham I 2020-01-23 134 } --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org