Re: [PATCH 06/15] net: thunderx: Enable mailbox interrupts on 81xx/83xx

From: kbuild test robot
Date: Mon Jul 11 2016 - 07:42:42 EST


Hi,

[auto build test WARNING on net-next/master]
[also build test WARNING on v4.7-rc7 next-20160711]
[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/sunil-kovvuri-gmail-com/net-thunderx-Add-support-for-81xx-and-83xx/20160711-181138
config: x86_64-randconfig-i0-201628 (attached as .config)
compiler: gcc-6 (Debian 6.1.1-1) 6.1.1 20160430
reproduce:
# save the attached .config to linux build tree
make ARCH=x86_64

All warnings (new ones prefixed by >>):

drivers/net/ethernet/cavium/thunder/nic_main.c: In function 'nic_enable_msix':
>> drivers/net/ethernet/cavium/thunder/nic_main.c:944:30: warning: iteration 10 invokes undefined behavior [-Waggressive-loop-optimizations]
nic->msix_entries[i].entry = i;
~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~
drivers/net/ethernet/cavium/thunder/nic_main.c:943:2: note: within this loop
for (i = 0; i < nic->num_vec; i++)
^~~

vim +944 drivers/net/ethernet/cavium/thunder/nic_main.c

4863dea3 Sunil Goutham 2015-05-26 928 vf + (mbx * vf_per_mbx_reg));
92dc8769 Sunil Goutham 2015-08-30 929
4863dea3 Sunil Goutham 2015-05-26 930 nic_handle_mbx_intr(nic, vf + (mbx * vf_per_mbx_reg));
4863dea3 Sunil Goutham 2015-05-26 931 nic_clear_mbx_intr(nic, vf, mbx);
4863dea3 Sunil Goutham 2015-05-26 932 }
4863dea3 Sunil Goutham 2015-05-26 933 }
4863dea3 Sunil Goutham 2015-05-26 934 return IRQ_HANDLED;
4863dea3 Sunil Goutham 2015-05-26 935 }
4863dea3 Sunil Goutham 2015-05-26 936
4863dea3 Sunil Goutham 2015-05-26 937 static int nic_enable_msix(struct nicpf *nic)
4863dea3 Sunil Goutham 2015-05-26 938 {
4863dea3 Sunil Goutham 2015-05-26 939 int i, ret;
4863dea3 Sunil Goutham 2015-05-26 940
45ff737f Sunil Goutham 2016-07-11 941 nic->num_vec = pci_msix_vec_count(nic->pdev);
4863dea3 Sunil Goutham 2015-05-26 942
4863dea3 Sunil Goutham 2015-05-26 943 for (i = 0; i < nic->num_vec; i++)
4863dea3 Sunil Goutham 2015-05-26 @944 nic->msix_entries[i].entry = i;
4863dea3 Sunil Goutham 2015-05-26 945
4863dea3 Sunil Goutham 2015-05-26 946 ret = pci_enable_msix(nic->pdev, nic->msix_entries, nic->num_vec);
4863dea3 Sunil Goutham 2015-05-26 947 if (ret) {
4863dea3 Sunil Goutham 2015-05-26 948 dev_err(&nic->pdev->dev,
45ff737f Sunil Goutham 2016-07-11 949 "Request for #%d msix vectors failed, returned %d\n",
45ff737f Sunil Goutham 2016-07-11 950 nic->num_vec, ret);
4863dea3 Sunil Goutham 2015-05-26 951 return ret;
4863dea3 Sunil Goutham 2015-05-26 952 }

:::::: The code at line 944 was first introduced by commit
:::::: 4863dea3fab01734768c9f7fc2c1590a8f1f6266 net: Adding support for Cavium ThunderX network controller

:::::: TO: Sunil Goutham <sgoutham@xxxxxxxxxx>
:::::: CC: David S. Miller <davem@xxxxxxxxxxxxx>

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

Attachment: .config.gz
Description: Binary data