CC: [email protected]
CC: Bjorn Helgaas <[email protected]>
CC: [email protected]
TO: Kishon Vijay Abraham I <[email protected]>
CC: Lorenzo Pieralisi <[email protected]>

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/lpieralisi/pci.git 
pci/endpoint
head:   805ff686a143d77d44ffbb8e34ec3c08e65cb6c4
commit: b64215ff2b5e9fef9208ba652590c51631a0ac1e [2/8] PCI: endpoint: Add 
support to add virtual function in endpoint core
:::::: branch date: 2 days ago
:::::: commit date: 2 days ago
config: i386-randconfig-m021-20210818 (attached as .config)
compiler: gcc-9 (Debian 9.3.0-22) 9.3.0

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <[email protected]>
Reported-by: Dan Carpenter <[email protected]>

smatch warnings:
drivers/pci/endpoint/pci-epf-core.c:169 pci_epf_add_vepf() warn: inconsistent 
returns '&epf_pf->lock'.

vim +169 drivers/pci/endpoint/pci-epf-core.c

5e8cb4033807e3 Kishon Vijay Abraham I 2017-04-10  130  
b64215ff2b5e9f Kishon Vijay Abraham I 2021-08-11  131  /**
b64215ff2b5e9f Kishon Vijay Abraham I 2021-08-11  132   * pci_epf_add_vepf() - 
associate virtual EP function to physical EP function
b64215ff2b5e9f Kishon Vijay Abraham I 2021-08-11  133   * @epf_pf: the physical 
EP function to which the virtual EP function should be
b64215ff2b5e9f Kishon Vijay Abraham I 2021-08-11  134   *   associated
b64215ff2b5e9f Kishon Vijay Abraham I 2021-08-11  135   * @epf_vf: the virtual 
EP function to be added
b64215ff2b5e9f Kishon Vijay Abraham I 2021-08-11  136   *
b64215ff2b5e9f Kishon Vijay Abraham I 2021-08-11  137   * A physical endpoint 
function can be associated with multiple virtual
b64215ff2b5e9f Kishon Vijay Abraham I 2021-08-11  138   * endpoint functions. 
Invoke pci_epf_add_epf() to add a virtual PCI endpoint
b64215ff2b5e9f Kishon Vijay Abraham I 2021-08-11  139   * function to a 
physical PCI endpoint function.
b64215ff2b5e9f Kishon Vijay Abraham I 2021-08-11  140   */
b64215ff2b5e9f Kishon Vijay Abraham I 2021-08-11  141  int 
pci_epf_add_vepf(struct pci_epf *epf_pf, struct pci_epf *epf_vf)
b64215ff2b5e9f Kishon Vijay Abraham I 2021-08-11  142  {
b64215ff2b5e9f Kishon Vijay Abraham I 2021-08-11  143   u32 vfunc_no;
b64215ff2b5e9f Kishon Vijay Abraham I 2021-08-11  144  
b64215ff2b5e9f Kishon Vijay Abraham I 2021-08-11  145   if 
(IS_ERR_OR_NULL(epf_pf) || IS_ERR_OR_NULL(epf_vf))
b64215ff2b5e9f Kishon Vijay Abraham I 2021-08-11  146           return -EINVAL;
b64215ff2b5e9f Kishon Vijay Abraham I 2021-08-11  147  
b64215ff2b5e9f Kishon Vijay Abraham I 2021-08-11  148   if (epf_pf->epc || 
epf_vf->epc || epf_vf->epf_pf)
b64215ff2b5e9f Kishon Vijay Abraham I 2021-08-11  149           return -EBUSY;
b64215ff2b5e9f Kishon Vijay Abraham I 2021-08-11  150  
b64215ff2b5e9f Kishon Vijay Abraham I 2021-08-11  151   if (epf_pf->sec_epc || 
epf_vf->sec_epc)
b64215ff2b5e9f Kishon Vijay Abraham I 2021-08-11  152           return -EBUSY;
b64215ff2b5e9f Kishon Vijay Abraham I 2021-08-11  153  
b64215ff2b5e9f Kishon Vijay Abraham I 2021-08-11  154   
mutex_lock(&epf_pf->lock);
b64215ff2b5e9f Kishon Vijay Abraham I 2021-08-11  155   vfunc_no = 
find_first_zero_bit(&epf_pf->vfunction_num_map,
b64215ff2b5e9f Kishon Vijay Abraham I 2021-08-11  156                           
       BITS_PER_LONG);
b64215ff2b5e9f Kishon Vijay Abraham I 2021-08-11  157   if (vfunc_no >= 
BITS_PER_LONG)
b64215ff2b5e9f Kishon Vijay Abraham I 2021-08-11  158           return -EINVAL;
b64215ff2b5e9f Kishon Vijay Abraham I 2021-08-11  159  
b64215ff2b5e9f Kishon Vijay Abraham I 2021-08-11  160   set_bit(vfunc_no, 
&epf_pf->vfunction_num_map);
b64215ff2b5e9f Kishon Vijay Abraham I 2021-08-11  161   epf_vf->vfunc_no = 
vfunc_no;
b64215ff2b5e9f Kishon Vijay Abraham I 2021-08-11  162  
b64215ff2b5e9f Kishon Vijay Abraham I 2021-08-11  163   epf_vf->epf_pf = epf_pf;
b64215ff2b5e9f Kishon Vijay Abraham I 2021-08-11  164   epf_vf->is_vf = true;
b64215ff2b5e9f Kishon Vijay Abraham I 2021-08-11  165  
b64215ff2b5e9f Kishon Vijay Abraham I 2021-08-11  166   
list_add_tail(&epf_vf->list, &epf_pf->pci_vepf);
b64215ff2b5e9f Kishon Vijay Abraham I 2021-08-11  167   
mutex_unlock(&epf_pf->lock);
b64215ff2b5e9f Kishon Vijay Abraham I 2021-08-11  168  
b64215ff2b5e9f Kishon Vijay Abraham I 2021-08-11 @169   return 0;
b64215ff2b5e9f Kishon Vijay Abraham I 2021-08-11  170  }
b64215ff2b5e9f Kishon Vijay Abraham I 2021-08-11  171  
EXPORT_SYMBOL_GPL(pci_epf_add_vepf);
b64215ff2b5e9f Kishon Vijay Abraham I 2021-08-11  172  

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/[email protected]

Attachment: .config.gz
Description: application/gzip

_______________________________________________
kbuild mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to