On 2024-07-09 at 03:44:14, Tony Nguyen (anthony.l.nguyen@xxxxxxxxx) wrote: > From: Maciej Fijalkowski <maciej.fijalkowski@xxxxxxxxx> > > { > u16 reg_idx = q_vector->reg_idx; > struct ice_pf *pf = vsi->back; > struct ice_hw *hw = &pf->hw; > - struct ice_tx_ring *tx_ring; > - struct ice_rx_ring *rx_ring; > + int q, _qid = qid; > > ice_cfg_itr(hw, q_vector); > > - ice_for_each_tx_ring(tx_ring, q_vector->tx) > - ice_cfg_txq_interrupt(vsi, tx_ring->reg_idx, reg_idx, > - q_vector->tx.itr_idx); > + for (q = 0; q < q_vector->num_ring_tx; q++) { > + ice_cfg_txq_interrupt(vsi, _qid, reg_idx, q_vector->tx.itr_idx); > + _qid++; > + } nit: why we need a new variable just for "for" loop ? qid + q wont suffice ? >