drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c:1240 i40e_set_vsi_promisc() error: uninitialized symbol 'aq_ret'.

From: kernel test robot
Date: Wed Aug 12 2020 - 16:52:32 EST

tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: fb893de323e2d39f7a1f6df425703a2edbdf56ea
commit: 37d318d7805f25b672bfd74fc694f19a2ee9665d i40e: Remove scheduling while atomic possibility
date: 7 weeks ago
config: parisc-randconfig-m031-20200811 (attached as .config)
compiler: hppa-linux-gcc (GCC) 9.3.0

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

smatch warnings:
drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c:1240 i40e_set_vsi_promisc() error: uninitialized symbol 'aq_ret'.

vim +/aq_ret +1240 drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c

37d318d7805f25 Aleksandr Loktionov 2020-05-29 1158
37d318d7805f25 Aleksandr Loktionov 2020-05-29 1159 /**
37d318d7805f25 Aleksandr Loktionov 2020-05-29 1160 * i40e_set_vsi_promisc
37d318d7805f25 Aleksandr Loktionov 2020-05-29 1161 * @vf: pointer to the VF struct
37d318d7805f25 Aleksandr Loktionov 2020-05-29 1162 * @seid: VSI number
37d318d7805f25 Aleksandr Loktionov 2020-05-29 1163 * @multi_enable: set MAC L2 layer multicast promiscuous enable/disable
37d318d7805f25 Aleksandr Loktionov 2020-05-29 1164 * for a given VLAN
37d318d7805f25 Aleksandr Loktionov 2020-05-29 1165 * @unicast_enable: set MAC L2 layer unicast promiscuous enable/disable
37d318d7805f25 Aleksandr Loktionov 2020-05-29 1166 * for a given VLAN
37d318d7805f25 Aleksandr Loktionov 2020-05-29 1167 * @vl: List of VLANs - apply filter for given VLANs
37d318d7805f25 Aleksandr Loktionov 2020-05-29 1168 * @num_vlans: Number of elements in @vl
37d318d7805f25 Aleksandr Loktionov 2020-05-29 1169 **/
37d318d7805f25 Aleksandr Loktionov 2020-05-29 1170 static i40e_status
37d318d7805f25 Aleksandr Loktionov 2020-05-29 1171 i40e_set_vsi_promisc(struct i40e_vf *vf, u16 seid, bool multi_enable,
37d318d7805f25 Aleksandr Loktionov 2020-05-29 1172 bool unicast_enable, s16 *vl, int num_vlans)
37d318d7805f25 Aleksandr Loktionov 2020-05-29 1173 {
37d318d7805f25 Aleksandr Loktionov 2020-05-29 1174 struct i40e_pf *pf = vf->pf;
37d318d7805f25 Aleksandr Loktionov 2020-05-29 1175 struct i40e_hw *hw = &pf->hw;
37d318d7805f25 Aleksandr Loktionov 2020-05-29 1176 i40e_status aq_ret;
37d318d7805f25 Aleksandr Loktionov 2020-05-29 1177 int i;
37d318d7805f25 Aleksandr Loktionov 2020-05-29 1178
37d318d7805f25 Aleksandr Loktionov 2020-05-29 1179 /* No VLAN to set promisc on, set on VSI */
37d318d7805f25 Aleksandr Loktionov 2020-05-29 1180 if (!num_vlans || !vl) {
37d318d7805f25 Aleksandr Loktionov 2020-05-29 1181 aq_ret = i40e_aq_set_vsi_multicast_promiscuous(hw, seid,
37d318d7805f25 Aleksandr Loktionov 2020-05-29 1182 multi_enable,
0ce5233e6c2afe Mariusz Stachura 2018-08-20 1183 NULL);
0ce5233e6c2afe Mariusz Stachura 2018-08-20 1184 if (aq_ret) {
0ce5233e6c2afe Mariusz Stachura 2018-08-20 1185 int aq_err = pf->hw.aq.asq_last_status;
0ce5233e6c2afe Mariusz Stachura 2018-08-20 1186
0ce5233e6c2afe Mariusz Stachura 2018-08-20 1187 dev_err(&pf->pdev->dev,
0ce5233e6c2afe Mariusz Stachura 2018-08-20 1188 "VF %d failed to set multicast promiscuous mode err %s aq_err %s\n",
0ce5233e6c2afe Mariusz Stachura 2018-08-20 1189 vf->vf_id,
0ce5233e6c2afe Mariusz Stachura 2018-08-20 1190 i40e_stat_str(&pf->hw, aq_ret),
0ce5233e6c2afe Mariusz Stachura 2018-08-20 1191 i40e_aq_str(&pf->hw, aq_err));
37d318d7805f25 Aleksandr Loktionov 2020-05-29 1192
0ce5233e6c2afe Mariusz Stachura 2018-08-20 1193 return aq_ret;
0ce5233e6c2afe Mariusz Stachura 2018-08-20 1194 }
0ce5233e6c2afe Mariusz Stachura 2018-08-20 1195
37d318d7805f25 Aleksandr Loktionov 2020-05-29 1196 aq_ret = i40e_aq_set_vsi_unicast_promiscuous(hw, seid,
37d318d7805f25 Aleksandr Loktionov 2020-05-29 1197 unicast_enable,
37d318d7805f25 Aleksandr Loktionov 2020-05-29 1198 NULL, true);
37d318d7805f25 Aleksandr Loktionov 2020-05-29 1199
0ce5233e6c2afe Mariusz Stachura 2018-08-20 1200 if (aq_ret) {
0ce5233e6c2afe Mariusz Stachura 2018-08-20 1201 int aq_err = pf->hw.aq.asq_last_status;
0ce5233e6c2afe Mariusz Stachura 2018-08-20 1202
0ce5233e6c2afe Mariusz Stachura 2018-08-20 1203 dev_err(&pf->pdev->dev,
0ce5233e6c2afe Mariusz Stachura 2018-08-20 1204 "VF %d failed to set unicast promiscuous mode err %s aq_err %s\n",
0ce5233e6c2afe Mariusz Stachura 2018-08-20 1205 vf->vf_id,
0ce5233e6c2afe Mariusz Stachura 2018-08-20 1206 i40e_stat_str(&pf->hw, aq_ret),
0ce5233e6c2afe Mariusz Stachura 2018-08-20 1207 i40e_aq_str(&pf->hw, aq_err));
0ce5233e6c2afe Mariusz Stachura 2018-08-20 1208 }
37d318d7805f25 Aleksandr Loktionov 2020-05-29 1209
0ce5233e6c2afe Mariusz Stachura 2018-08-20 1210 return aq_ret;
37d318d7805f25 Aleksandr Loktionov 2020-05-29 1211 }
37d318d7805f25 Aleksandr Loktionov 2020-05-29 1212
37d318d7805f25 Aleksandr Loktionov 2020-05-29 1213 for (i = 0; i < num_vlans; i++) {
37d318d7805f25 Aleksandr Loktionov 2020-05-29 1214 aq_ret = i40e_aq_set_vsi_mc_promisc_on_vlan(hw, seid,
37d318d7805f25 Aleksandr Loktionov 2020-05-29 1215 multi_enable,
37d318d7805f25 Aleksandr Loktionov 2020-05-29 1216 vl[i], NULL);
0ce5233e6c2afe Mariusz Stachura 2018-08-20 1217 if (aq_ret) {
0ce5233e6c2afe Mariusz Stachura 2018-08-20 1218 int aq_err = pf->hw.aq.asq_last_status;
0ce5233e6c2afe Mariusz Stachura 2018-08-20 1219
0ce5233e6c2afe Mariusz Stachura 2018-08-20 1220 dev_err(&pf->pdev->dev,
37d318d7805f25 Aleksandr Loktionov 2020-05-29 1221 "VF %d failed to set multicast promiscuous mode err %s aq_err %s\n",
37d318d7805f25 Aleksandr Loktionov 2020-05-29 1222 vf->vf_id,
0ce5233e6c2afe Mariusz Stachura 2018-08-20 1223 i40e_stat_str(&pf->hw, aq_ret),
0ce5233e6c2afe Mariusz Stachura 2018-08-20 1224 i40e_aq_str(&pf->hw, aq_err));
0ce5233e6c2afe Mariusz Stachura 2018-08-20 1225 }
0ce5233e6c2afe Mariusz Stachura 2018-08-20 1226
37d318d7805f25 Aleksandr Loktionov 2020-05-29 1227 aq_ret = i40e_aq_set_vsi_uc_promisc_on_vlan(hw, seid,
37d318d7805f25 Aleksandr Loktionov 2020-05-29 1228 unicast_enable,
37d318d7805f25 Aleksandr Loktionov 2020-05-29 1229 vl[i], NULL);
0ce5233e6c2afe Mariusz Stachura 2018-08-20 1230 if (aq_ret) {
0ce5233e6c2afe Mariusz Stachura 2018-08-20 1231 int aq_err = pf->hw.aq.asq_last_status;
0ce5233e6c2afe Mariusz Stachura 2018-08-20 1232
0ce5233e6c2afe Mariusz Stachura 2018-08-20 1233 dev_err(&pf->pdev->dev,
37d318d7805f25 Aleksandr Loktionov 2020-05-29 1234 "VF %d failed to set unicast promiscuous mode err %s aq_err %s\n",
37d318d7805f25 Aleksandr Loktionov 2020-05-29 1235 vf->vf_id,
0ce5233e6c2afe Mariusz Stachura 2018-08-20 1236 i40e_stat_str(&pf->hw, aq_ret),
0ce5233e6c2afe Mariusz Stachura 2018-08-20 1237 i40e_aq_str(&pf->hw, aq_err));
0ce5233e6c2afe Mariusz Stachura 2018-08-20 1238 }
0ce5233e6c2afe Mariusz Stachura 2018-08-20 1239 }
0ce5233e6c2afe Mariusz Stachura 2018-08-20 @1240 return aq_ret;
0ce5233e6c2afe Mariusz Stachura 2018-08-20 1241 }
0ce5233e6c2afe Mariusz Stachura 2018-08-20 1242

:::::: The code at line 1240 was first introduced by commit
:::::: 0ce5233e6c2afeb39b9631961978a993a6d4e500 i40e: Unset promiscuous settings on VF reset

:::::: TO: Mariusz Stachura <mariusz.stachura@xxxxxxxxx>
:::::: CC: Jeff Kirsher <jeffrey.t.kirsher@xxxxxxxxx>

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

Attachment: .config.gz
Description: application/gzip