samsung_sm8750/android_kernel_samsung_sm8750
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

ice: create new Tx scheduler nodes for new queues only

Browse Source 
[ Upstream commit 6fa2942578472c9cab13a8fc1dae0d830193e0a1 ]

The current implementation of the Tx scheduler tree attempts
to create nodes for all Tx queues, ignoring the fact that some
queues may already exist in the tree. For example, if the VSI
already has 128 Tx queues and the user requests for 16 new queues,
the Tx scheduler will compute the tree for 272 queues (128 existing
queues + 144 new queues), instead of 144 queues (128 existing queues
and 16 new queues).
Fix that by modifying the node count calculation algorithm to skip
the queues that already exist in the tree.

Fixes: 5513b920a4 ("ice: Update Tx scheduler tree for VSI multi-Tx queue support")
Reviewed-by: Dawid Osuchowski <dawid.osuchowski@linux.intel.com>
Reviewed-by: Przemek Kitszel <przemyslaw.kitszel@intel.com>
Reviewed-by: Jacob Keller <jacob.e.keller@intel.com>
Signed-off-by: Michal Kubiak <michal.kubiak@intel.com>
Reviewed-by: Simon Horman <horms@kernel.org>
Tested-by: Jesse Brandeburg <jbrandeburg@cloudflare.com>
Tested-by: Saritha Sanigani <sarithax.sanigani@intel.com> (A Contingent Worker at Intel)
Signed-off-by: Tony Nguyen <anthony.l.nguyen@intel.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
Michal Kubiak
2025-05-13 12:55:28 +02:00
committed by Greg Kroah-Hartman
parent 1d3c5d0dec
commit d78e538b65
1 changed files with 6 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
drivers/net/ethernet/intel/ice/ice_sched.c
View File

@@ -1614,16 +1614,16 @@ ice_sched_get_agg_node(struct ice_port_info *pi, struct ice_sched_node *tc_node,
/** /**
* ice_sched_calc_vsi_child_nodes - calculate number of VSI child nodes * ice_sched_calc_vsi_child_nodes - calculate number of VSI child nodes
* @hw: pointer to the HW struct * @hw: pointer to the HW struct
* @num_qs: number of queues * @num_new_qs: number of new queues that will be added to the tree
* @num_nodes: num nodes array * @num_nodes: num nodes array
* *
* This function calculates the number of VSI child nodes based on the * This function calculates the number of VSI child nodes based on the
* number of queues. * number of queues.
*/ */
static void static void
ice_sched_calc_vsi_child_nodes(struct ice_hw *hw, u16 num_qs, u16 *num_nodes) ice_sched_calc_vsi_child_nodes(struct ice_hw *hw, u16 num_new_qs, u16 *num_nodes)
{ {
u16 num = num_qs; u16 num = num_new_qs;
u8 i, qgl, vsil; u8 i, qgl, vsil;
qgl = ice_sched_get_qgrp_layer(hw); qgl = ice_sched_get_qgrp_layer(hw);
@@ -1873,8 +1873,9 @@ ice_sched_update_vsi_child_nodes(struct ice_port_info *pi, u16 vsi_handle,
return status; return status;
} }
if (new_numqs) ice_sched_calc_vsi_child_nodes(hw, new_numqs - prev_numqs,
ice_sched_calc_vsi_child_nodes(hw, new_numqs, new_num_nodes); new_num_nodes);
/* Keep the max number of queue configuration all the time. Update the /* Keep the max number of queue configuration all the time. Update the
* tree only if number of queues > previous number of queues. This may * tree only if number of queues > previous number of queues. This may
* leave some extra nodes in the tree if number of queues < previous * leave some extra nodes in the tree if number of queues < previous