From: kbuild test robot <lkp@intel.com>
To: Yongji Xie <xyjxie@linux.vnet.ibm.com>
Cc: kbuild-all@01.org, bhelgaas@google.com, zhong@linux.vnet.ibm.com,
aik@ozlabs.ru, linux-pci@vger.kernel.org,
gwshan@linux.vnet.ibm.com, alex.williamson@redhat.com,
paulus@samba.org, linuxppc-dev@lists.ozlabs.org
Subject: Re: [PATCH v8 2/3] PCI: Make sure the driver could get correct BAR size from pci_resource_len()
Date: Sat, 21 Jan 2017 03:53:11 +0800 [thread overview]
Message-ID: <201701210327.c1ultHEI%fengguang.wu@intel.com> (raw)
In-Reply-To: <1484898113-24939-3-git-send-email-xyjxie@linux.vnet.ibm.com>
[-- Attachment #1: Type: text/plain, Size: 2838 bytes --]
Hi Yongji,
[auto build test WARNING on pci/next]
[also build test WARNING on v4.10-rc4 next-20170120]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]
url: https://github.com/0day-ci/linux/commits/Yongji-Xie/PCI-Ignore-requested-alignment-for-IOV-BARs/20170121-031322
base: https://git.kernel.org/pub/scm/linux/kernel/git/helgaas/pci.git next
config: i386-defconfig (attached as .config)
compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
reproduce:
# save the attached .config to linux build tree
make ARCH=i386
All warnings (new ones prefixed by >>):
In file included from drivers/ata/ata_piix.c:88:0:
drivers/ata/ata_piix.c: In function 'piix_init_sidpr':
>> include/linux/pci.h:1648:44: warning: comparison between 'enum <anonymous>' and 'enum <anonymous>' [-Wenum-compare]
#define pci_resource_end(dev, bar) (((bar) > PCI_ROM_RESOURCE) ? \
^
>> include/linux/pci.h:1655:4: note: in expansion of macro 'pci_resource_end'
pci_resource_end((dev), (bar)) == \
^~~~~~~~~~~~~~~~
>> drivers/ata/ata_piix.c:1471:6: note: in expansion of macro 'pci_resource_len'
pci_resource_len(pdev, PIIX_SIDPR_BAR) != PIIX_SIDPR_LEN)
^~~~~~~~~~~~~~~~
>> include/linux/pci.h:1648:44: warning: comparison between 'enum <anonymous>' and 'enum <anonymous>' [-Wenum-compare]
#define pci_resource_end(dev, bar) (((bar) > PCI_ROM_RESOURCE) ? \
^
include/linux/pci.h:1658:4: note: in expansion of macro 'pci_resource_end'
(pci_resource_end((dev), (bar)) - \
^~~~~~~~~~~~~~~~
>> drivers/ata/ata_piix.c:1471:6: note: in expansion of macro 'pci_resource_len'
pci_resource_len(pdev, PIIX_SIDPR_BAR) != PIIX_SIDPR_LEN)
^~~~~~~~~~~~~~~~
vim +1648 include/linux/pci.h
1642 #define pci_root_bus_fwnode(bus) NULL
1643 #endif
1644
1645 /* these helpers provide future and backwards compatibility
1646 * for accessing popular PCI BAR info */
1647 #define pci_resource_start(dev, bar) ((dev)->resource[(bar)].start)
> 1648 #define pci_resource_end(dev, bar) (((bar) > PCI_ROM_RESOURCE) ? \
1649 (dev)->resource[(bar)].end : \
1650 ((dev)->resource[(bar)].end - \
1651 (dev)->res_addsize[(bar)]))
1652 #define pci_resource_flags(dev, bar) ((dev)->resource[(bar)].flags)
1653 #define pci_resource_len(dev,bar) \
1654 ((pci_resource_start((dev), (bar)) == 0 && \
> 1655 pci_resource_end((dev), (bar)) == \
1656 pci_resource_start((dev), (bar))) ? 0 : \
1657 \
1658 (pci_resource_end((dev), (bar)) - \
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 25441 bytes --]
next prev parent reply other threads:[~2017-01-20 19:53 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-01-20 7:41 [PATCH v8 0/3] PCI: Introduce a way to enforce all MMIO BARs not to share PAGE_SIZE Yongji Xie
2017-01-20 7:41 ` [PATCH v8 1/3] PCI: Ignore requested alignment for IOV BARs Yongji Xie
2017-01-20 7:41 ` [PATCH v8 2/3] PCI: Make sure the driver could get correct BAR size from pci_resource_len() Yongji Xie
2017-01-20 19:53 ` kbuild test robot [this message]
2017-01-20 20:41 ` kbuild test robot
2017-01-20 7:41 ` [PATCH v8 3/3] PCI: Add a macro to set default alignment for all PCI devices Yongji Xie
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=201701210327.c1ultHEI%fengguang.wu@intel.com \
--to=lkp@intel.com \
--cc=aik@ozlabs.ru \
--cc=alex.williamson@redhat.com \
--cc=bhelgaas@google.com \
--cc=gwshan@linux.vnet.ibm.com \
--cc=kbuild-all@01.org \
--cc=linux-pci@vger.kernel.org \
--cc=linuxppc-dev@lists.ozlabs.org \
--cc=paulus@samba.org \
--cc=xyjxie@linux.vnet.ibm.com \
--cc=zhong@linux.vnet.ibm.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).