Commit 04fb71cc authored by Hariprasad Kelam's avatar Hariprasad Kelam Committed by Jakub Kicinski

octeontx2-pf: Reuse Transmit queue/Send queue index of HTB class

Real number of Transmit queues are incremented when user enables HTB
class and vice versa. Depending on SKB priority driver returns transmit
queue (Txq). Transmit queues and Send queues are one-to-one mapped.

In few scenarios, Driver is returning transmit queue value which is
greater than real number of transmit queue and Stack detects this as
error and overwrites transmit queue value.

For example
user has added two classes and real number of queues are incremented
accordingly
- tc class add dev eth1 parent 1: classid 1:1 htb
      rate 100Mbit ceil 100Mbit prio 1 quantum 1024
- tc class add dev eth1 parent 1: classid 1:2 htb
      rate 100Mbit ceil 200Mbit prio 7 quantum 1024

now if user deletes the class with id 1:1, driver decrements the real
number of queues
- tc class del dev eth1 classid 1:1

But for the class with id 1:2, driver is returning transmit queue
value which is higher than real number of transmit queue leading
to below error

eth1 selects TX queue x, but real number of TX queues is x

This patch solves the problem by assigning deleted class transmit
queue/send queue to active class.
Signed-off-by: default avatarHariprasad Kelam <hkelam@marvell.com>
Reviewed-by: default avatarSimon Horman <horms@kernel.org>
Link: https://lore.kernel.org/r/20240508070935.11501-1-hkelam@marvell.comSigned-off-by: default avatarJakub Kicinski <kuba@kernel.org>
parent 9c1bbc7e
...@@ -545,6 +545,20 @@ otx2_qos_sw_create_leaf_node(struct otx2_nic *pfvf, ...@@ -545,6 +545,20 @@ otx2_qos_sw_create_leaf_node(struct otx2_nic *pfvf,
return node; return node;
} }
static struct otx2_qos_node
*otx2_sw_node_find_by_qid(struct otx2_nic *pfvf, u16 qid)
{
struct otx2_qos_node *node = NULL;
int bkt;
hash_for_each(pfvf->qos.qos_hlist, bkt, node, hlist) {
if (node->qid == qid)
break;
}
return node;
}
static struct otx2_qos_node * static struct otx2_qos_node *
otx2_sw_node_find(struct otx2_nic *pfvf, u32 classid) otx2_sw_node_find(struct otx2_nic *pfvf, u32 classid)
{ {
...@@ -917,6 +931,7 @@ static void otx2_qos_enadis_sq(struct otx2_nic *pfvf, ...@@ -917,6 +931,7 @@ static void otx2_qos_enadis_sq(struct otx2_nic *pfvf,
otx2_qos_disable_sq(pfvf, qid); otx2_qos_disable_sq(pfvf, qid);
pfvf->qos.qid_to_sqmap[qid] = node->schq; pfvf->qos.qid_to_sqmap[qid] = node->schq;
otx2_qos_txschq_config(pfvf, node);
otx2_qos_enable_sq(pfvf, qid); otx2_qos_enable_sq(pfvf, qid);
} }
...@@ -1475,13 +1490,45 @@ static int otx2_qos_leaf_to_inner(struct otx2_nic *pfvf, u16 classid, ...@@ -1475,13 +1490,45 @@ static int otx2_qos_leaf_to_inner(struct otx2_nic *pfvf, u16 classid,
return ret; return ret;
} }
static int otx2_qos_cur_leaf_nodes(struct otx2_nic *pfvf)
{
int last = find_last_bit(pfvf->qos.qos_sq_bmap, pfvf->hw.tc_tx_queues);
return last == pfvf->hw.tc_tx_queues ? 0 : last + 1;
}
static void otx2_reset_qdisc(struct net_device *dev, u16 qid)
{
struct netdev_queue *dev_queue = netdev_get_tx_queue(dev, qid);
struct Qdisc *qdisc = rtnl_dereference(dev_queue->qdisc_sleeping);
if (!qdisc)
return;
spin_lock_bh(qdisc_lock(qdisc));
qdisc_reset(qdisc);
spin_unlock_bh(qdisc_lock(qdisc));
}
static void otx2_cfg_smq(struct otx2_nic *pfvf, struct otx2_qos_node *node,
int qid)
{
struct otx2_qos_node *tmp;
list_for_each_entry(tmp, &node->child_schq_list, list)
if (tmp->level == NIX_TXSCH_LVL_MDQ) {
otx2_qos_txschq_config(pfvf, tmp);
pfvf->qos.qid_to_sqmap[qid] = tmp->schq;
}
}
static int otx2_qos_leaf_del(struct otx2_nic *pfvf, u16 *classid, static int otx2_qos_leaf_del(struct otx2_nic *pfvf, u16 *classid,
struct netlink_ext_ack *extack) struct netlink_ext_ack *extack)
{ {
struct otx2_qos_node *node, *parent; struct otx2_qos_node *node, *parent;
int dwrr_del_node = false; int dwrr_del_node = false;
u16 qid, moved_qid;
u64 prio; u64 prio;
u16 qid;
netdev_dbg(pfvf->netdev, "TC_HTB_LEAF_DEL classid %04x\n", *classid); netdev_dbg(pfvf->netdev, "TC_HTB_LEAF_DEL classid %04x\n", *classid);
...@@ -1517,6 +1564,37 @@ static int otx2_qos_leaf_del(struct otx2_nic *pfvf, u16 *classid, ...@@ -1517,6 +1564,37 @@ static int otx2_qos_leaf_del(struct otx2_nic *pfvf, u16 *classid,
if (!parent->child_static_cnt) if (!parent->child_static_cnt)
parent->max_static_prio = 0; parent->max_static_prio = 0;
moved_qid = otx2_qos_cur_leaf_nodes(pfvf);
/* last node just deleted */
if (moved_qid == 0 || moved_qid == qid)
return 0;
moved_qid--;
node = otx2_sw_node_find_by_qid(pfvf, moved_qid);
if (!node)
return 0;
/* stop traffic to the old queue and disable
* SQ associated with it
*/
node->qid = OTX2_QOS_QID_INNER;
__clear_bit(moved_qid, pfvf->qos.qos_sq_bmap);
otx2_qos_disable_sq(pfvf, moved_qid);
otx2_reset_qdisc(pfvf->netdev, pfvf->hw.tx_queues + moved_qid);
/* enable SQ associated with qid and
* update the node
*/
otx2_cfg_smq(pfvf, node, qid);
otx2_qos_enable_sq(pfvf, qid);
__set_bit(qid, pfvf->qos.qos_sq_bmap);
node->qid = qid;
*classid = node->classid;
return 0; return 0;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment