Subject: Re: [net-next PATCH 5/5] octeontx2-pf: Add support for HTB offload

From: Maxim Mikityanskiy <[email-masked]>
Date: Fri, 13 Jan 2023 14:18:45 +0200

On Thu, Jan 12, 2023 at 11:01:20PM +0530, Hariprasad Kelam wrote:
> From: Naveen Mamindlapalli <[email-masked]>
> 
> This patch registers callbacks to support HTB offload.
> 
> Below are features supported,
> 
> - supports traffic shaping on the given class by honoring rate and ceil
> configuration.
> 
> - supports traffic scheduling,  which prioritizes different types of
> traffic based on strict priority values.
> 
> - supports the creation of leaf to inner classes such that parent node
> rate limits apply to all child nodes.
> 
> Signed-off-by: Naveen Mamindlapalli <[email-masked]>
> Signed-off-by: Hariprasad Kelam <[email-masked]>
> Signed-off-by: Sunil Kovvuri Goutham <[email-masked]>
> ---

Great job! Nice to see the second user of HTB offload.

(Ccing Mellanox folks for more eyes on the series [1].)

[1]: https://lore.kernel.org/all/[email-masked]/

> +	case TC_HTB_NODE_MODIFY:
> +		fallthrough;
> +	default:
> +		return -EOPNOTSUPP;

Is there any technical difficulty or hardware limitation preventing from
implementing modifications?