]> git.dujemihanovic.xyz Git - linux.git/commit
octeontx2-pf: Reuse Transmit queue/Send queue index of HTB class
authorHariprasad Kelam <hkelam@marvell.com>
Wed, 8 May 2024 07:09:35 +0000 (12:39 +0530)
committerJakub Kicinski <kuba@kernel.org>
Sat, 11 May 2024 01:49:15 +0000 (18:49 -0700)
commit04fb71cc5f1866f391a9c21ea634217e065ae525
tree8668dbac7ebeab07cf53f3d91d8a6e53899d5a28
parent9c1bbc7ea1a719d2edd616f676b1adf73c1000a6
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: Hariprasad Kelam <hkelam@marvell.com>
Reviewed-by: Simon Horman <horms@kernel.org>
Link: https://lore.kernel.org/r/20240508070935.11501-1-hkelam@marvell.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/ethernet/marvell/octeontx2/nic/qos.c