diff mbox series

[v1] net/ice: fix the reversed priority of DCF switch rule

Message ID 20210802072517.942854-1-wenjun1.wu@intel.com (mailing list archive)
State Accepted
Delegated to: Qi Zhang
Headers show
Series [v1] net/ice: fix the reversed priority of DCF switch rule | expand

Checks

Context Check Description
ci/iol-abi-testing success Testing PASS
ci/iol-testing success Testing PASS
ci/intel-Testing success Testing PASS
ci/Intel-compilation success Compilation OK
ci/iol-mellanox-Performance success Performance Testing PASS
ci/iol-intel-Performance success Performance Testing PASS
ci/iol-intel-Functional success Functional Testing PASS
ci/github-robot success github build: passed
ci/checkpatch success coding style OK

Commit Message

Wenjun Wu Aug. 2, 2021, 7:25 a.m. UTC
This patch fixes the reversed priority of DCF switch rule. Priority 0 and 1
are supported, and priority 0 should be the highest priority.

Fixes: 2321e34c23b3 ("net/ice: support flow priority for DCF switch filter")

Signed-off-by: Wenjun Wu <wenjun1.wu@intel.com>
---
 drivers/net/ice/ice_switch_filter.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Zhang, Qi Z Aug. 10, 2021, 3:05 a.m. UTC | #1
> -----Original Message-----
> From: Wu, Wenjun1 <wenjun1.wu@intel.com>
> Sent: Monday, August 2, 2021 3:25 PM
> To: dev@dpdk.org; Yang, Qiming <qiming.yang@intel.com>; Zhang, Qi Z
> <qi.z.zhang@intel.com>; Zhang, Yuying <yuying.zhang@intel.com>
> Cc: Wu, Wenjun1 <wenjun1.wu@intel.com>
> Subject: [PATCH v1] net/ice: fix the reversed priority of DCF switch rule
> 
> This patch fixes the reversed priority of DCF switch rule. Priority 0 and 1 are
> supported, and priority 0 should be the highest priority.
> 
> Fixes: 2321e34c23b3 ("net/ice: support flow priority for DCF switch filter")
> 
> Signed-off-by: Wenjun Wu <wenjun1.wu@intel.com>

Acked-by: Qi Zhang <qi.z.zhang@intel.com>

Applied to dpdk-next-net-intel.

Thanks
Qi
Ferruh Yigit Aug. 13, 2021, 10:45 a.m. UTC | #2
On 8/2/2021 8:25 AM, Wenjun Wu wrote:
> This patch fixes the reversed priority of DCF switch rule. Priority 0 and 1
> are supported, and priority 0 should be the highest priority.
> 
> Fixes: 2321e34c23b3 ("net/ice: support flow priority for DCF switch filter")
> 

I guess 'Cc: stable@dpdk.org' tag is needed, will add in next-net.
Please check if it is needed next time, './devtools/check-git-log.sh' should help.

@Aaron, @Thomas, Can we add 'check-git-log.sh' checks to the CI?

Thanks,
ferruh

> Signed-off-by: Wenjun Wu <wenjun1.wu@intel.com>
> ---
>  drivers/net/ice/ice_switch_filter.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/net/ice/ice_switch_filter.c b/drivers/net/ice/ice_switch_filter.c
> index eeed386c63..f222cb9cb0 100644
> --- a/drivers/net/ice/ice_switch_filter.c
> +++ b/drivers/net/ice/ice_switch_filter.c
> @@ -1572,7 +1572,7 @@ ice_switch_parse_dcf_action(struct ice_dcf_adapter *ad,
>  	rule_info->sw_act.src = rule_info->sw_act.vsi_handle;
>  	rule_info->sw_act.flag = ICE_FLTR_RX;
>  	rule_info->rx = 1;
> -	rule_info->priority = priority + 5;
> +	rule_info->priority = 6 - priority;
>  
>  	return 0;
>  }
>
Thomas Monjalon Aug. 13, 2021, 1:21 p.m. UTC | #3
13/08/2021 12:45, Ferruh Yigit:
> On 8/2/2021 8:25 AM, Wenjun Wu wrote:
> > This patch fixes the reversed priority of DCF switch rule. Priority 0 and 1
> > are supported, and priority 0 should be the highest priority.
> > 
> > Fixes: 2321e34c23b3 ("net/ice: support flow priority for DCF switch filter")
> > 
> 
> I guess 'Cc: stable@dpdk.org' tag is needed, will add in next-net.
> Please check if it is needed next time, './devtools/check-git-log.sh' should help.
> 
> @Aaron, @Thomas, Can we add 'check-git-log.sh' checks to the CI?

We need first to fix the rule for the patches touching "common/".
There is a patch in progress.
Aaron Conole Aug. 23, 2021, 12:41 p.m. UTC | #4
Thomas Monjalon <thomas@monjalon.net> writes:

> 13/08/2021 12:45, Ferruh Yigit:
>> On 8/2/2021 8:25 AM, Wenjun Wu wrote:
>> > This patch fixes the reversed priority of DCF switch rule. Priority 0 and 1
>> > are supported, and priority 0 should be the highest priority.
>> > 
>> > Fixes: 2321e34c23b3 ("net/ice: support flow priority for DCF switch filter")
>> > 
>> 
>> I guess 'Cc: stable@dpdk.org' tag is needed, will add in next-net.
>> Please check if it is needed next time, './devtools/check-git-log.sh' should help.
>> 
>> @Aaron, @Thomas, Can we add 'check-git-log.sh' checks to the CI?
>
> We need first to fix the rule for the patches touching "common/".
> There is a patch in progress.

I think it's possible to add - I don't think it will be too much work.
diff mbox series

Patch

diff --git a/drivers/net/ice/ice_switch_filter.c b/drivers/net/ice/ice_switch_filter.c
index eeed386c63..f222cb9cb0 100644
--- a/drivers/net/ice/ice_switch_filter.c
+++ b/drivers/net/ice/ice_switch_filter.c
@@ -1572,7 +1572,7 @@  ice_switch_parse_dcf_action(struct ice_dcf_adapter *ad,
 	rule_info->sw_act.src = rule_info->sw_act.vsi_handle;
 	rule_info->sw_act.flag = ICE_FLTR_RX;
 	rule_info->rx = 1;
-	rule_info->priority = priority + 5;
+	rule_info->priority = 6 - priority;
 
 	return 0;
 }