Adjust the VSI/Aggregator layers based on the number of logical
layers supported by the FW. Currently the VSI and aggregator layers are
fixed based on the 9 layer scheduler tree layout. Due to performance
reasons the number of layers of the scheduler tree is changing from
9 to 5. It requires a readjument of these VSI/Aggregator layer
values.

Signed-off-by: Victor Raj <victor....@intel.com>
Signed-off-by: Qi Zhang <qi.z.zh...@intel.com>
---
 drivers/net/ice/base/ice_sched.c | 34 ++++++++++++++++----------------
 drivers/net/ice/base/ice_sched.h |  3 +++
 2 files changed, 20 insertions(+), 17 deletions(-)

diff --git a/drivers/net/ice/base/ice_sched.c b/drivers/net/ice/base/ice_sched.c
index 6f938d71a1..4d31e96fd0 100644
--- a/drivers/net/ice/base/ice_sched.c
+++ b/drivers/net/ice/base/ice_sched.c
@@ -1130,12 +1130,11 @@ static u8 ice_sched_get_vsi_layer(struct ice_hw *hw)
         *     5 or less       sw_entry_point_layer
         */
        /* calculate the VSI layer based on number of layers. */
-       if (hw->num_tx_sched_layers > ICE_VSI_LAYER_OFFSET + 1) {
-               u8 layer = hw->num_tx_sched_layers - ICE_VSI_LAYER_OFFSET;
-
-               if (layer > hw->sw_entry_point_layer)
-                       return layer;
-       }
+       if (hw->num_tx_sched_layers == ICE_SCHED_9_LAYERS)
+               return hw->num_tx_sched_layers - ICE_VSI_LAYER_OFFSET;
+       else if (hw->num_tx_sched_layers == ICE_SCHED_5_LAYERS)
+               /* qgroup and VSI layers are same */
+               return hw->num_tx_sched_layers - ICE_QGRP_LAYER_OFFSET;
        return hw->sw_entry_point_layer;
 }
 
@@ -1152,12 +1151,8 @@ static u8 ice_sched_get_agg_layer(struct ice_hw *hw)
         *     7 or less       sw_entry_point_layer
         */
        /* calculate the aggregator layer based on number of layers. */
-       if (hw->num_tx_sched_layers > ICE_AGG_LAYER_OFFSET + 1) {
-               u8 layer = hw->num_tx_sched_layers - ICE_AGG_LAYER_OFFSET;
-
-               if (layer > hw->sw_entry_point_layer)
-                       return layer;
-       }
+       if (hw->num_tx_sched_layers == ICE_SCHED_9_LAYERS)
+               return hw->num_tx_sched_layers - ICE_AGG_LAYER_OFFSET;
        return hw->sw_entry_point_layer;
 }
 
@@ -1542,10 +1537,11 @@ ice_sched_get_free_qparent(struct ice_port_info *pi, 
u16 vsi_handle, u8 tc,
 {
        struct ice_sched_node *vsi_node, *qgrp_node;
        struct ice_vsi_ctx *vsi_ctx;
+       u8 qgrp_layer, vsi_layer;
        u16 max_children;
-       u8 qgrp_layer;
 
        qgrp_layer = ice_sched_get_qgrp_layer(pi->hw);
+       vsi_layer = ice_sched_get_vsi_layer(pi->hw);
        max_children = pi->hw->max_children[qgrp_layer];
 
        vsi_ctx = ice_get_vsi_ctx(pi->hw, vsi_handle);
@@ -1556,6 +1552,12 @@ ice_sched_get_free_qparent(struct ice_port_info *pi, u16 
vsi_handle, u8 tc,
        if (!vsi_node)
                return NULL;
 
+       /* If the queue group and vsi layer are same then queues
+        * are all attached directly to VSI
+        */
+       if (qgrp_layer == vsi_layer)
+               return vsi_node;
+
        /* get the first queue group node from VSI sub-tree */
        qgrp_node = ice_sched_get_first_node(pi, vsi_node, qgrp_layer);
        while (qgrp_node) {
@@ -4060,7 +4062,7 @@ ice_sched_add_rl_profile(struct ice_hw *hw, enum 
ice_rl_type rl_type,
        enum ice_status status;
        u8 profile_type;
 
-       if (layer_num >= ICE_AQC_TOPO_MAX_LEVEL_NUM)
+       if (!hw || layer_num >= hw->num_tx_sched_layers)
                return NULL;
        switch (rl_type) {
        case ICE_MIN_BW:
@@ -4076,8 +4078,6 @@ ice_sched_add_rl_profile(struct ice_hw *hw, enum 
ice_rl_type rl_type,
                return NULL;
        }
 
-       if (!hw)
-               return NULL;
        LIST_FOR_EACH_ENTRY(rl_prof_elem, &hw->rl_prof_list[layer_num],
                            ice_aqc_rl_profile_info, list_entry)
                if ((rl_prof_elem->profile.flags & ICE_AQC_RL_PROFILE_TYPE_M) ==
@@ -4279,7 +4279,7 @@ ice_sched_rm_rl_profile(struct ice_hw *hw, u8 layer_num, 
u8 profile_type,
        struct ice_aqc_rl_profile_info *rl_prof_elem;
        enum ice_status status = ICE_SUCCESS;
 
-       if (layer_num >= ICE_AQC_TOPO_MAX_LEVEL_NUM)
+       if (!hw || layer_num >= hw->num_tx_sched_layers)
                return ICE_ERR_PARAM;
        /* Check the existing list for RL profile */
        LIST_FOR_EACH_ENTRY(rl_prof_elem, &hw->rl_prof_list[layer_num],
diff --git a/drivers/net/ice/base/ice_sched.h b/drivers/net/ice/base/ice_sched.h
index 6b12a0688a..53a68dbe51 100644
--- a/drivers/net/ice/base/ice_sched.h
+++ b/drivers/net/ice/base/ice_sched.h
@@ -7,6 +7,9 @@
 
 #include "ice_common.h"
 
+#define ICE_SCHED_5_LAYERS     5
+#define ICE_SCHED_9_LAYERS     9
+
 #define ICE_QGRP_LAYER_OFFSET  2
 #define ICE_VSI_LAYER_OFFSET   4
 #define ICE_AGG_LAYER_OFFSET   6
-- 
2.31.1

Reply via email to