ACK: [SRU][F:linux-bluefield][PATCH] net/sched: act_ct: Offload only ASSURED connections

[Tim Gardner]

Acked-by: Tim Gardner <tim.gardner at canonical.com>

On 2/22/22 20:21, Bodong Wang wrote:
> From: Chris Mi <cmi at nvidia.com>
> 
> BugLink: https://bugs.launchpad.net/bugs/1961819
> 
> Short-lived connections increase the insertion rate requirements,
> fill the offload table and provide very limited offload value since
> they process a very small amount of packets. The ct ASSURED flag is
> designed to filter short-lived connections for early expiration.
> 
> Offload connections when they are ESTABLISHED and ASSURED.
> 
> Signed-off-by: Chris Mi <cmi at nvidia.com>
> Reviewed-by: Oz Shlomo <ozsh at nvidia.com>
> Reviewed-by: Roi Dayan <roid at nvidia.com>
> Signed-off-by: David S. Miller <davem at davemloft.net>
> (cherry picked from commit 43332cf97425a3e5508c827c82201ecc5ddd54e0)
> Signed-off-by: Bodong Wang <bodong at nvidia.com>
> ---
>   net/sched/act_ct.c | 3 ++-
>   1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/net/sched/act_ct.c b/net/sched/act_ct.c
> index 8a78cbd..16306b8 100644
> --- a/net/sched/act_ct.c
> +++ b/net/sched/act_ct.c
> @@ -388,7 +388,8 @@ static void tcf_ct_flow_table_process_conn(struct tcf_ct_flow_table *ct_ft,
>   {
>   	bool tcp = false;
>   
> -	if (ctinfo != IP_CT_ESTABLISHED && ctinfo != IP_CT_ESTABLISHED_REPLY)
> +	if ((ctinfo != IP_CT_ESTABLISHED && ctinfo != IP_CT_ESTABLISHED_REPLY) ||
> +	    !test_bit(IPS_ASSURED_BIT, &ct->status))
>   		return;
>   
>   	switch (nf_ct_protonum(ct)) {

-- 
-----------
Tim Gardner
Canonical, Inc