mbox series

[net-next,v2,0/4] net: mvneta: mqprio cleanups and shaping support

Message ID 20211126112056.849123-1-maxime.chevallier@bootlin.com (mailing list archive)
Headers show
Series net: mvneta: mqprio cleanups and shaping support | expand

Message

Maxime Chevallier Nov. 26, 2021, 11:20 a.m. UTC
Hello everyone,

This is the second version of the series that adds some improvements to the
existing mqprio implementation in mvneta, and adds support for
egress shaping offload.

The first 3 patches are some minor cleanups, such as using the
tc_mqprio_qopt_offload structure to get access to more offloading
options, cleaning the logic to detect whether or not we should offload
mqprio setting, and allowing to have a 1 to N mapping between TCs and
queues.

The last patch adds traffic shaping offload, using mvneta's per-queue
token buckets, allowing to limit rates from 10Kbps up to 5Gbps with
10Kbps increments.

This was tested only on an Armada 3720, with traffic up to 2.5Gbps.

Changes since V1 fixes the build for 32bits kernels, using the right
div helpers as suggested by Jakub.

Maxime Chevallier (4):
  net: mvneta: Use struct tc_mqprio_qopt_offload for MQPrio
    configuration
  net: mvneta: Don't force-set the offloading flag
  net: mvneta: Allow having more than one queue per TC
  net: mvneta: Add TC traffic shaping offload

 drivers/net/ethernet/marvell/mvneta.c | 160 +++++++++++++++++++++++---
 1 file changed, 144 insertions(+), 16 deletions(-)

Comments

patchwork-bot+netdevbpf@kernel.org Nov. 29, 2021, 12:20 p.m. UTC | #1
Hello:

This series was applied to netdev/net-next.git (master)
by David S. Miller <davem@davemloft.net>:

On Fri, 26 Nov 2021 12:20:52 +0100 you wrote:
> Hello everyone,
> 
> This is the second version of the series that adds some improvements to the
> existing mqprio implementation in mvneta, and adds support for
> egress shaping offload.
> 
> The first 3 patches are some minor cleanups, such as using the
> tc_mqprio_qopt_offload structure to get access to more offloading
> options, cleaning the logic to detect whether or not we should offload
> mqprio setting, and allowing to have a 1 to N mapping between TCs and
> queues.
> 
> [...]

Here is the summary with links:
  - [net-next,v2,1/4] net: mvneta: Use struct tc_mqprio_qopt_offload for MQPrio configuration
    https://git.kernel.org/netdev/net-next/c/75fa71e3acad
  - [net-next,v2,2/4] net: mvneta: Don't force-set the offloading flag
    https://git.kernel.org/netdev/net-next/c/e7ca75fe6662
  - [net-next,v2,3/4] net: mvneta: Allow having more than one queue per TC
    https://git.kernel.org/netdev/net-next/c/e9f7099d0730
  - [net-next,v2,4/4] net: mvneta: Add TC traffic shaping offload
    https://git.kernel.org/netdev/net-next/c/2551dc9e398c

You are awesome, thank you!