Hi Xiangliang,

[auto build test WARNING on v4.4-rc5]
[also build test WARNING on next-20151217]

url:    
https://github.com/0day-ci/linux/commits/Xiangliang-Yu/NTB-Add-AMD-PCI-Express-NTB-driver/20151217-113608
config: x86_64-randconfig-n0-12171710 (attached as .config)
reproduce:
        # save the attached .config to linux build tree
        make ARCH=x86_64 

Note: it may well be a FALSE warning. FWIW you are at least aware of it now.
http://gcc.gnu.org/wiki/Better_Uninitialized_Warnings

All warnings (new ones prefixed by >>):

   drivers/ntb/hw/amd/ntb_hw_amd.c:486:12: warning: 'amd_flush_peer_requests' 
defined but not used [-Wunused-function]
    static int amd_flush_peer_requests(struct amd_ntb_dev *ndev)
               ^
   drivers/ntb/hw/amd/ntb_hw_amd.c:507:12: warning: 'amd_wakeup_peer_side' 
defined but not used [-Wunused-function]
    static int amd_wakeup_peer_side(struct amd_ntb_dev *ndev)
               ^
   drivers/ntb/hw/amd/ntb_hw_amd.c: In function 'amd_ntb_mw_set_trans':
>> drivers/ntb/hw/amd/ntb_hw_amd.c:187:6: warning: 'limit' may be used 
>> uninitialized in this function [-Wmaybe-uninitialized]
      if (reg_val != limit) {
         ^

vim +/limit +187 drivers/ntb/hw/amd/ntb_hw_amd.c

a635755a Xiangliang Yu 2015-12-17  171          if (bar != 1) {
a635755a Xiangliang Yu 2015-12-17  172                  /* Set the limit if 
supported */
a635755a Xiangliang Yu 2015-12-17  173                  if (limit_reg)
a635755a Xiangliang Yu 2015-12-17  174                          limit = 
base_addr + size;
a635755a Xiangliang Yu 2015-12-17  175  
a635755a Xiangliang Yu 2015-12-17  176                  /* set and verify 
setting the translation address */
a635755a Xiangliang Yu 2015-12-17  177                  iowrite64(addr, 
peer_mmio + xlat_reg);
a635755a Xiangliang Yu 2015-12-17  178                  reg_val = 
ioread64(peer_mmio + xlat_reg);
a635755a Xiangliang Yu 2015-12-17  179                  if (reg_val != addr) {
a635755a Xiangliang Yu 2015-12-17  180                          iowrite64(0, 
peer_mmio + xlat_reg);
a635755a Xiangliang Yu 2015-12-17  181                          return -EIO;
a635755a Xiangliang Yu 2015-12-17  182                  }
a635755a Xiangliang Yu 2015-12-17  183  
a635755a Xiangliang Yu 2015-12-17  184                  /* set and verify 
setting the limit */
a635755a Xiangliang Yu 2015-12-17  185                  iowrite64(limit, mmio + 
limit_reg);
a635755a Xiangliang Yu 2015-12-17  186                  reg_val = ioread64(mmio 
+ limit_reg);
a635755a Xiangliang Yu 2015-12-17 @187                  if (reg_val != limit) {
a635755a Xiangliang Yu 2015-12-17  188                          
iowrite64(base_addr, mmio + limit_reg);
a635755a Xiangliang Yu 2015-12-17  189                          iowrite64(0, 
peer_mmio + xlat_reg);
a635755a Xiangliang Yu 2015-12-17  190                          return -EIO;
a635755a Xiangliang Yu 2015-12-17  191                  }
a635755a Xiangliang Yu 2015-12-17  192          } else {
a635755a Xiangliang Yu 2015-12-17  193                  /* split bar addr range 
must all be 32 bit */
a635755a Xiangliang Yu 2015-12-17  194                  if (addr & (~0ull << 
32))
a635755a Xiangliang Yu 2015-12-17  195                          return -EINVAL;

:::::: The code at line 187 was first introduced by commit
:::::: a635755a710138a6f505cc7e62693741786fb19e NTB: Add AMD PCI-Express NTB 
driver

:::::: TO: Xiangliang Yu <[email protected]>
:::::: CC: 0day robot <[email protected]>

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

Attachment: .config.gz
Description: Binary data

Reply via email to