linux-kernel - Re: [PATCH] PCI: Try to find two continuous regions for child resource

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <202103300042.f3GFGCGr-lkp@intel.com>
Date:   Tue, 30 Mar 2021 00:56:35 +0800
From:   kernel test robot <lkp@...el.com>
To:     Kai-Heng Feng <kai.heng.feng@...onical.com>, bhelgaas@...gle.com
Cc:     kbuild-all@...ts.01.org, Qian Cai <cai@....pw>,
        Michal Simek <monstr@...str.eu>, Arnd Bergmann <arnd@...db.de>,
        "open list:SPARC + UltraSPARC sparc/sparc64" 
        <sparclinux@...r.kernel.org>, Alexey Kardashevskiy <aik@...abs.ru>,
        "open list:PCI SUBSYSTEM" <linux-pci@...r.kernel.org>,
        open list <linux-kernel@...r.kernel.org>,
        Dominik Brodowski <linux@...inikbrodowski.net>,
        Qinglang Miao <miaoqinglang@...wei.com>
Subject: Re: [PATCH] PCI: Try to find two continuous regions for child
 resource

Hi Kai-Heng,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on pci/next]
[also build test ERROR on powerpc/next sparc/master linus/master sparc-next/master v5.12-rc5]
[cannot apply to next-20210329]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]

url:    https://github.com/0day-ci/linux/commits/Kai-Heng-Feng/PCI-Try-to-find-two-continuous-regions-for-child-resource/20210329-165155
base:   https://git.kernel.org/pub/scm/linux/kernel/git/helgaas/pci.git next
config: microblaze-randconfig-p001-20210329 (attached as .config)
compiler: microblaze-linux-gcc (GCC) 9.3.0
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # https://github.com/0day-ci/linux/commit/94d03d680678c966738f767c5fb91305a3af5c9d
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review Kai-Heng-Feng/PCI-Try-to-find-two-continuous-regions-for-child-resource/20210329-165155
        git checkout 94d03d680678c966738f767c5fb91305a3af5c9d
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=microblaze 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@...el.com>

All errors (new ones prefixed by >>):

   arch/microblaze/pci/pci-common.c: In function 'alloc_resource':
>> arch/microblaze/pci/pci-common.c:749:7: error: too few arguments to function 'pci_find_parent_resource'
     749 |  pr = pci_find_parent_resource(dev, r);
         |       ^~~~~~~~~~~~~~~~~~~~~~~~
   In file included from arch/microblaze/pci/pci-common.c:16:
   include/linux/pci.h:1052:6: note: declared here
    1052 | void pci_find_parent_resource(const struct pci_dev *dev,
         |      ^~~~~~~~~~~~~~~~~~~~~~~~


vim +/pci_find_parent_resource +749 arch/microblaze/pci/pci-common.c

d3afa58c20b651 Michal Simek       2010-01-18  738  
b881bc469bdbdc Greg Kroah-Hartman 2012-12-21  739  static inline void alloc_resource(struct pci_dev *dev, int idx)
d3afa58c20b651 Michal Simek       2010-01-18  740  {
d3afa58c20b651 Michal Simek       2010-01-18  741  	struct resource *pr, *r = &dev->resource[idx];
d3afa58c20b651 Michal Simek       2010-01-18  742  
d3afa58c20b651 Michal Simek       2010-01-18  743  	pr_debug("PCI: Allocating %s: Resource %d: %016llx..%016llx [%x]\n",
d3afa58c20b651 Michal Simek       2010-01-18  744  		 pci_name(dev), idx,
d3afa58c20b651 Michal Simek       2010-01-18  745  		 (unsigned long long)r->start,
d3afa58c20b651 Michal Simek       2010-01-18  746  		 (unsigned long long)r->end,
d3afa58c20b651 Michal Simek       2010-01-18  747  		 (unsigned int)r->flags);
d3afa58c20b651 Michal Simek       2010-01-18  748  
d3afa58c20b651 Michal Simek       2010-01-18 @749  	pr = pci_find_parent_resource(dev, r);
d3afa58c20b651 Michal Simek       2010-01-18  750  	if (!pr || (pr->flags & IORESOURCE_UNSET) ||
d3afa58c20b651 Michal Simek       2010-01-18  751  	    request_resource(pr, r) < 0) {
6bd55f0bbaebb7 Michal Simek       2012-12-27  752  		pr_warn("PCI: Cannot allocate resource region %d ", idx);
6bd55f0bbaebb7 Michal Simek       2012-12-27  753  		pr_cont("of device %s, will remap\n", pci_name(dev));
d3afa58c20b651 Michal Simek       2010-01-18  754  		if (pr)
d3afa58c20b651 Michal Simek       2010-01-18  755  			pr_debug("PCI:  parent is %p: %016llx-%016llx [%x]\n",
d3afa58c20b651 Michal Simek       2010-01-18  756  				 pr,
d3afa58c20b651 Michal Simek       2010-01-18  757  				 (unsigned long long)pr->start,
d3afa58c20b651 Michal Simek       2010-01-18  758  				 (unsigned long long)pr->end,
d3afa58c20b651 Michal Simek       2010-01-18  759  				 (unsigned int)pr->flags);
d3afa58c20b651 Michal Simek       2010-01-18  760  		/* We'll assign a new address later */
d3afa58c20b651 Michal Simek       2010-01-18  761  		r->flags |= IORESOURCE_UNSET;
d3afa58c20b651 Michal Simek       2010-01-18  762  		r->end -= r->start;
d3afa58c20b651 Michal Simek       2010-01-18  763  		r->start = 0;
d3afa58c20b651 Michal Simek       2010-01-18  764  	}
d3afa58c20b651 Michal Simek       2010-01-18  765  }
d3afa58c20b651 Michal Simek       2010-01-18  766  

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org

Download attachment ".config.gz" of type "application/gzip" (42005 bytes)