Messages in this thread | | | Date | Wed, 1 Mar 2023 08:00:07 +0300 | From | Dan Carpenter <> | Subject | drivers/net/ethernet/intel/ice/ice_txrx.c:1304 ice_clean_rx_irq() error: uninitialized symbol 'cached_ntu'. |
| |
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: c0927a7a5391f7d8e593e5e50ead7505a23cadf9 commit: 3246a10752a7120878740f345f3956997777b3d6 ice: Add support for XDP multi-buffer on Tx side config: x86_64-randconfig-m001-20230227 (https://download.01.org/0day-ci/archive/20230301/202303011231.SEfXSwLT-lkp@intel.com/config) compiler: gcc-11 (Debian 11.3.0-8) 11.3.0
If you fix the issue, kindly add following tag where applicable | Reported-by: kernel test robot <lkp@intel.com> | Reported-by: Dan Carpenter <error27@gmail.com> | Link: https://lore.kernel.org/r/202303011231.SEfXSwLT-lkp@intel.com/
New smatch warnings: drivers/net/ethernet/intel/ice/ice_txrx.c:1304 ice_clean_rx_irq() error: uninitialized symbol 'cached_ntu'.
vim +/cached_ntu +1304 drivers/net/ethernet/intel/ice/ice_txrx.c
efc2214b6047b6 Maciej Fijalkowski 2019-11-04 1150 eb087cd828648d Maciej Fijalkowski 2021-08-19 1151 xdp_prog = READ_ONCE(rx_ring->xdp_prog); 3246a10752a712 Maciej Fijalkowski 2023-01-31 1152 if (xdp_prog) { eb087cd828648d Maciej Fijalkowski 2021-08-19 1153 xdp_ring = rx_ring->xdp_ring; 3246a10752a712 Maciej Fijalkowski 2023-01-31 1154 cached_ntu = xdp_ring->next_to_use; 3246a10752a712 Maciej Fijalkowski 2023-01-31 1155 }
Uninitialized on else path.
eb087cd828648d Maciej Fijalkowski 2021-08-19 1156 f9867df6d96593 Anirudh Venkataramanan 2019-02-19 1157 /* start the loop to process Rx packets bounded by 'budget' */ 2b245cb29421ab Anirudh Venkataramanan 2018-03-20 1158 while (likely(total_rx_pkts < (unsigned int)budget)) { 2b245cb29421ab Anirudh Venkataramanan 2018-03-20 1159 union ice_32b_rx_flex_desc *rx_desc; 6c869cb7a8f02b Maciej Fijalkowski 2019-02-13 1160 struct ice_rx_buf *rx_buf; 2fba7dc5157b6f Maciej Fijalkowski 2023-01-31 1161 struct sk_buff *skb; 6c869cb7a8f02b Maciej Fijalkowski 2019-02-13 1162 unsigned int size; 2b245cb29421ab Anirudh Venkataramanan 2018-03-20 1163 u16 stat_err_bits; 2b245cb29421ab Anirudh Venkataramanan 2018-03-20 1164 u16 vlan_tag = 0; dda90cb90a5ced Jesse Brandeburg 2021-02-23 1165 u16 rx_ptype; 2b245cb29421ab Anirudh Venkataramanan 2018-03-20 1166 f9867df6d96593 Anirudh Venkataramanan 2019-02-19 1167 /* get the Rx desc from Rx ring based on 'next_to_clean' */ d7956d81f1502d Maciej Fijalkowski 2023-01-31 1168 rx_desc = ICE_RX_DESC(rx_ring, ntc); 2b245cb29421ab Anirudh Venkataramanan 2018-03-20 1169 2b245cb29421ab Anirudh Venkataramanan 2018-03-20 1170 /* status_error_len will always be zero for unused descriptors 2b245cb29421ab Anirudh Venkataramanan 2018-03-20 1171 * because it's cleared in cleanup, and overlaps with hdr_addr 2b245cb29421ab Anirudh Venkataramanan 2018-03-20 1172 * which is always zero because packet split isn't used, if the 2b245cb29421ab Anirudh Venkataramanan 2018-03-20 1173 * hardware wrote DD then it will be non-zero 2b245cb29421ab Anirudh Venkataramanan 2018-03-20 1174 */ 2b245cb29421ab Anirudh Venkataramanan 2018-03-20 1175 stat_err_bits = BIT(ICE_RX_FLEX_DESC_STATUS0_DD_S); 0d54d8f7a16de0 Brett Creeley 2021-12-02 1176 if (!ice_test_staterr(rx_desc->wb.status_error0, stat_err_bits)) 2b245cb29421ab Anirudh Venkataramanan 2018-03-20 1177 break; 2b245cb29421ab Anirudh Venkataramanan 2018-03-20 1178 2b245cb29421ab Anirudh Venkataramanan 2018-03-20 1179 /* This memory barrier is needed to keep us from reading 2b245cb29421ab Anirudh Venkataramanan 2018-03-20 1180 * any other fields out of the rx_desc until we know the 2b245cb29421ab Anirudh Venkataramanan 2018-03-20 1181 * DD bit is set. 2b245cb29421ab Anirudh Venkataramanan 2018-03-20 1182 */ 2b245cb29421ab Anirudh Venkataramanan 2018-03-20 1183 dma_rmb(); 2b245cb29421ab Anirudh Venkataramanan 2018-03-20 1184 3089cf6d3caa1e Jesse Brandeburg 2021-06-08 1185 ice_trace(clean_rx_irq, rx_ring, rx_desc); 148beb61203125 Henry Tieman 2020-05-11 1186 if (rx_desc->wb.rxdid == FDIR_DESC_RXDID || !rx_ring->netdev) { d6218317e2eff8 Qi Zhang 2021-03-09 1187 struct ice_vsi *ctrl_vsi = rx_ring->vsi; d6218317e2eff8 Qi Zhang 2021-03-09 1188 d6218317e2eff8 Qi Zhang 2021-03-09 1189 if (rx_desc->wb.rxdid == FDIR_DESC_RXDID && b03d519d3460f3 Jacob Keller 2022-02-16 1190 ctrl_vsi->vf) d6218317e2eff8 Qi Zhang 2021-03-09 1191 ice_vc_fdir_irq_handler(ctrl_vsi, rx_desc); d7956d81f1502d Maciej Fijalkowski 2023-01-31 1192 if (++ntc == cnt) d7956d81f1502d Maciej Fijalkowski 2023-01-31 1193 ntc = 0; 148beb61203125 Henry Tieman 2020-05-11 1194 continue; 148beb61203125 Henry Tieman 2020-05-11 1195 } 148beb61203125 Henry Tieman 2020-05-11 1196 6c869cb7a8f02b Maciej Fijalkowski 2019-02-13 1197 size = le16_to_cpu(rx_desc->wb.pkt_len) & 6c869cb7a8f02b Maciej Fijalkowski 2019-02-13 1198 ICE_RX_FLX_DESC_PKT_LEN_M; 6c869cb7a8f02b Maciej Fijalkowski 2019-02-13 1199 ac6f733a7bd5e2 Mitch Williams 2019-07-25 1200 /* retrieve a buffer from the ring */ d7956d81f1502d Maciej Fijalkowski 2023-01-31 1201 rx_buf = ice_get_rx_buf(rx_ring, size, ntc); ac6f733a7bd5e2 Mitch Williams 2019-07-25 1202 2fba7dc5157b6f Maciej Fijalkowski 2023-01-31 1203 if (!xdp->data) { 2fba7dc5157b6f Maciej Fijalkowski 2023-01-31 1204 void *hard_start; efc2214b6047b6 Maciej Fijalkowski 2019-11-04 1205 be9df4aff65f18 Lorenzo Bianconi 2020-12-22 1206 hard_start = page_address(rx_buf->page) + rx_buf->page_offset - be9df4aff65f18 Lorenzo Bianconi 2020-12-22 1207 offset; cb0473e0e9dcca Maciej Fijalkowski 2023-01-31 1208 xdp_prepare_buff(xdp, hard_start, offset, size, !!offset); d4ecdbf7aa2fa4 Jesper Dangaard Brouer 2020-05-14 1209 #if (PAGE_SIZE > 4096) d4ecdbf7aa2fa4 Jesper Dangaard Brouer 2020-05-14 1210 /* At larger PAGE_SIZE, frame_sz depend on len size */ cb0473e0e9dcca Maciej Fijalkowski 2023-01-31 1211 xdp->frame_sz = ice_rx_frame_truesize(rx_ring, size); d4ecdbf7aa2fa4 Jesper Dangaard Brouer 2020-05-14 1212 #endif 2fba7dc5157b6f Maciej Fijalkowski 2023-01-31 1213 xdp_buff_clear_frags_flag(xdp); 2fba7dc5157b6f Maciej Fijalkowski 2023-01-31 1214 } else if (ice_add_xdp_frag(rx_ring, xdp, rx_buf, size)) { 2fba7dc5157b6f Maciej Fijalkowski 2023-01-31 1215 break; 2fba7dc5157b6f Maciej Fijalkowski 2023-01-31 1216 } 2fba7dc5157b6f Maciej Fijalkowski 2023-01-31 1217 if (++ntc == cnt) 2fba7dc5157b6f Maciej Fijalkowski 2023-01-31 1218 ntc = 0; 2fba7dc5157b6f Maciej Fijalkowski 2023-01-31 1219 2fba7dc5157b6f Maciej Fijalkowski 2023-01-31 1220 /* skip if it is NOP desc */ 2fba7dc5157b6f Maciej Fijalkowski 2023-01-31 1221 if (ice_is_non_eop(rx_ring, rx_desc)) 2fba7dc5157b6f Maciej Fijalkowski 2023-01-31 1222 continue; efc2214b6047b6 Maciej Fijalkowski 2019-11-04 1223 1dc1a7e7f4108b Maciej Fijalkowski 2023-01-31 1224 ice_run_xdp(rx_ring, xdp, xdp_prog, xdp_ring, rx_buf); 1dc1a7e7f4108b Maciej Fijalkowski 2023-01-31 1225 if (rx_buf->act == ICE_XDP_PASS) 59bb0808055758 Maciej Fijalkowski 2019-10-24 1226 goto construct_skb; 2fba7dc5157b6f Maciej Fijalkowski 2023-01-31 1227 total_rx_bytes += xdp_get_buff_len(xdp); efc2214b6047b6 Maciej Fijalkowski 2019-11-04 1228 total_rx_pkts++; efc2214b6047b6 Maciej Fijalkowski 2019-11-04 1229 2fba7dc5157b6f Maciej Fijalkowski 2023-01-31 1230 xdp->data = NULL; 2fba7dc5157b6f Maciej Fijalkowski 2023-01-31 1231 rx_ring->first_desc = ntc; efc2214b6047b6 Maciej Fijalkowski 2019-11-04 1232 continue; efc2214b6047b6 Maciej Fijalkowski 2019-11-04 1233 construct_skb: 2fba7dc5157b6f Maciej Fijalkowski 2023-01-31 1234 if (likely(ice_ring_uses_build_skb(rx_ring))) 2fba7dc5157b6f Maciej Fijalkowski 2023-01-31 1235 skb = ice_build_skb(rx_ring, xdp); 712edbbb67d404 Maciej Fijalkowski 2019-02-13 1236 else 2fba7dc5157b6f Maciej Fijalkowski 2023-01-31 1237 skb = ice_construct_skb(rx_ring, xdp); 712edbbb67d404 Maciej Fijalkowski 2019-02-13 1238 /* exit if we failed to retrieve a buffer */ 712edbbb67d404 Maciej Fijalkowski 2019-02-13 1239 if (!skb) { 2fba7dc5157b6f Maciej Fijalkowski 2023-01-31 1240 rx_ring->ring_stats->rx_stats.alloc_page_failed++; 2fba7dc5157b6f Maciej Fijalkowski 2023-01-31 1241 rx_buf->act = ICE_XDP_CONSUMED; 2fba7dc5157b6f Maciej Fijalkowski 2023-01-31 1242 if (unlikely(xdp_buff_has_frags(xdp))) 2fba7dc5157b6f Maciej Fijalkowski 2023-01-31 1243 ice_set_rx_bufs_act(xdp, rx_ring, 2fba7dc5157b6f Maciej Fijalkowski 2023-01-31 1244 ICE_XDP_CONSUMED); 2fba7dc5157b6f Maciej Fijalkowski 2023-01-31 1245 xdp->data = NULL; 2fba7dc5157b6f Maciej Fijalkowski 2023-01-31 1246 rx_ring->first_desc = ntc; 2b245cb29421ab Anirudh Venkataramanan 2018-03-20 1247 break; 712edbbb67d404 Maciej Fijalkowski 2019-02-13 1248 } 2fba7dc5157b6f Maciej Fijalkowski 2023-01-31 1249 xdp->data = NULL; 2fba7dc5157b6f Maciej Fijalkowski 2023-01-31 1250 rx_ring->first_desc = ntc; 2b245cb29421ab Anirudh Venkataramanan 2018-03-20 1251 2b245cb29421ab Anirudh Venkataramanan 2018-03-20 1252 stat_err_bits = BIT(ICE_RX_FLEX_DESC_STATUS0_RXE_S); 0d54d8f7a16de0 Brett Creeley 2021-12-02 1253 if (unlikely(ice_test_staterr(rx_desc->wb.status_error0, 0d54d8f7a16de0 Brett Creeley 2021-12-02 1254 stat_err_bits))) { 2b245cb29421ab Anirudh Venkataramanan 2018-03-20 1255 dev_kfree_skb_any(skb); 2b245cb29421ab Anirudh Venkataramanan 2018-03-20 1256 continue; 2b245cb29421ab Anirudh Venkataramanan 2018-03-20 1257 } 2b245cb29421ab Anirudh Venkataramanan 2018-03-20 1258 0d54d8f7a16de0 Brett Creeley 2021-12-02 1259 vlan_tag = ice_get_vlan_tag_from_rx_desc(rx_desc); 2b245cb29421ab Anirudh Venkataramanan 2018-03-20 1260 133f4883f90668 Krzysztof Kazimierczak 2019-10-09 1261 /* pad the skb if needed, to make a valid ethernet frame */ 2fba7dc5157b6f Maciej Fijalkowski 2023-01-31 1262 if (eth_skb_pad(skb)) 2b245cb29421ab Anirudh Venkataramanan 2018-03-20 1263 continue; 2b245cb29421ab Anirudh Venkataramanan 2018-03-20 1264 2b245cb29421ab Anirudh Venkataramanan 2018-03-20 1265 /* probably a little skewed due to removing CRC */ 2b245cb29421ab Anirudh Venkataramanan 2018-03-20 1266 total_rx_bytes += skb->len; 2b245cb29421ab Anirudh Venkataramanan 2018-03-20 1267 d76a60ba7afb89 Anirudh Venkataramanan 2018-03-20 1268 /* populate checksum, VLAN, and protocol */ 6503b659302893 Jesse Brandeburg 2019-08-08 1269 rx_ptype = le16_to_cpu(rx_desc->wb.ptype_flex_flags0) & 6503b659302893 Jesse Brandeburg 2019-08-08 1270 ICE_RX_FLEX_DESC_PTYPE_M; 6503b659302893 Jesse Brandeburg 2019-08-08 1271 d76a60ba7afb89 Anirudh Venkataramanan 2018-03-20 1272 ice_process_skb_fields(rx_ring, rx_desc, skb, rx_ptype); d76a60ba7afb89 Anirudh Venkataramanan 2018-03-20 1273 3089cf6d3caa1e Jesse Brandeburg 2021-06-08 1274 ice_trace(clean_rx_irq_indicate, rx_ring, rx_desc, skb); 2b245cb29421ab Anirudh Venkataramanan 2018-03-20 1275 /* send completed skb up the stack */ 2b245cb29421ab Anirudh Venkataramanan 2018-03-20 1276 ice_receive_skb(rx_ring, skb, vlan_tag); 2b245cb29421ab Anirudh Venkataramanan 2018-03-20 1277 2b245cb29421ab Anirudh Venkataramanan 2018-03-20 1278 /* update budget accounting */ 2b245cb29421ab Anirudh Venkataramanan 2018-03-20 1279 total_rx_pkts++; 2b245cb29421ab Anirudh Venkataramanan 2018-03-20 1280 } 2b245cb29421ab Anirudh Venkataramanan 2018-03-20 1281 2fba7dc5157b6f Maciej Fijalkowski 2023-01-31 1282 first = rx_ring->first_desc; 2fba7dc5157b6f Maciej Fijalkowski 2023-01-31 1283 while (cached_ntc != first) { 1dc1a7e7f4108b Maciej Fijalkowski 2023-01-31 1284 struct ice_rx_buf *buf = &rx_ring->rx_buf[cached_ntc]; 1dc1a7e7f4108b Maciej Fijalkowski 2023-01-31 1285 1dc1a7e7f4108b Maciej Fijalkowski 2023-01-31 1286 if (buf->act & (ICE_XDP_TX | ICE_XDP_REDIR)) { 1dc1a7e7f4108b Maciej Fijalkowski 2023-01-31 1287 ice_rx_buf_adjust_pg_offset(buf, xdp->frame_sz); 1dc1a7e7f4108b Maciej Fijalkowski 2023-01-31 1288 xdp_xmit |= buf->act; 1dc1a7e7f4108b Maciej Fijalkowski 2023-01-31 1289 } else if (buf->act & ICE_XDP_CONSUMED) { 1dc1a7e7f4108b Maciej Fijalkowski 2023-01-31 1290 buf->pagecnt_bias++; 1dc1a7e7f4108b Maciej Fijalkowski 2023-01-31 1291 } else if (buf->act == ICE_XDP_PASS) { 1dc1a7e7f4108b Maciej Fijalkowski 2023-01-31 1292 ice_rx_buf_adjust_pg_offset(buf, xdp->frame_sz); 1dc1a7e7f4108b Maciej Fijalkowski 2023-01-31 1293 } 1dc1a7e7f4108b Maciej Fijalkowski 2023-01-31 1294 1dc1a7e7f4108b Maciej Fijalkowski 2023-01-31 1295 ice_put_rx_buf(rx_ring, buf); 1dc1a7e7f4108b Maciej Fijalkowski 2023-01-31 1296 if (++cached_ntc >= cnt) 1dc1a7e7f4108b Maciej Fijalkowski 2023-01-31 1297 cached_ntc = 0; 1dc1a7e7f4108b Maciej Fijalkowski 2023-01-31 1298 } d7956d81f1502d Maciej Fijalkowski 2023-01-31 1299 rx_ring->next_to_clean = ntc; cb7db35641c9a5 Brett Creeley 2019-06-26 1300 /* return up to cleaned_count buffers to hardware */ 2fba7dc5157b6f Maciej Fijalkowski 2023-01-31 1301 failure = ice_alloc_rx_bufs(rx_ring, ICE_RX_DESC_UNUSED(rx_ring)); cb7db35641c9a5 Brett Creeley 2019-06-26 1302 9070fe3da0b126 Maciej Fijalkowski 2023-01-31 1303 if (xdp_xmit) 3246a10752a712 Maciej Fijalkowski 2023-01-31 @1304 ice_finalize_xdp_rx(xdp_ring, xdp_xmit, cached_ntu); ^^^^^^^^^^
efc2214b6047b6 Maciej Fijalkowski 2019-11-04 1305 288ecf491b1654 Benjamin Mikailenko 2022-11-18 1306 if (rx_ring->ring_stats) 288ecf491b1654 Benjamin Mikailenko 2022-11-18 1307 ice_update_rx_ring_stats(rx_ring, total_rx_pkts, 288ecf491b1654 Benjamin Mikailenko 2022-11-18 1308 total_rx_bytes); 2b245cb29421ab Anirudh Venkataramanan 2018-03-20 1309 2b245cb29421ab Anirudh Venkataramanan 2018-03-20 1310 /* guarantee a trip back through this routine if there was a failure */ 2b245cb29421ab Anirudh Venkataramanan 2018-03-20 1311 return failure ? budget : (int)total_rx_pkts; 2b245cb29421ab Anirudh Venkataramanan 2018-03-20 1312 }
-- 0-DAY CI Kernel Test Service https://github.com/intel/lkp-tests
| |