DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH v1] net/ice: fix the reversed priority of DCF switch rule
@ 2021-08-02  7:25 Wenjun Wu
  2021-08-10  3:05 ` Zhang, Qi Z
  2021-08-13 10:45 ` Ferruh Yigit
  0 siblings, 2 replies; 5+ messages in thread
From: Wenjun Wu @ 2021-08-02  7:25 UTC (permalink / raw)
  To: dev, qiming.yang, qi.z.zhang, yuying.zhang; +Cc: Wenjun Wu

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(-)

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;
 }
-- 
2.25.1


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [dpdk-dev] [PATCH v1] net/ice: fix the reversed priority of DCF switch rule
  2021-08-02  7:25 [dpdk-dev] [PATCH v1] net/ice: fix the reversed priority of DCF switch rule Wenjun Wu
@ 2021-08-10  3:05 ` Zhang, Qi Z
  2021-08-13 10:45 ` Ferruh Yigit
  1 sibling, 0 replies; 5+ messages in thread
From: Zhang, Qi Z @ 2021-08-10  3:05 UTC (permalink / raw)
  To: Wu, Wenjun1, dev, Yang, Qiming, Zhang, Yuying



> -----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


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [dpdk-dev] [PATCH v1] net/ice: fix the reversed priority of DCF switch rule
  2021-08-02  7:25 [dpdk-dev] [PATCH v1] net/ice: fix the reversed priority of DCF switch rule Wenjun Wu
  2021-08-10  3:05 ` Zhang, Qi Z
@ 2021-08-13 10:45 ` Ferruh Yigit
  2021-08-13 13:21   ` Thomas Monjalon
  1 sibling, 1 reply; 5+ messages in thread
From: Ferruh Yigit @ 2021-08-13 10:45 UTC (permalink / raw)
  To: Wenjun Wu, dev, qiming.yang, qi.z.zhang, yuying.zhang
  Cc: Thomas Monjalon, David Marchand, Aaron Conole, dpdklab

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;
>  }
> 


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [dpdk-dev] [PATCH v1] net/ice: fix the reversed priority of DCF switch rule
  2021-08-13 10:45 ` Ferruh Yigit
@ 2021-08-13 13:21   ` Thomas Monjalon
  2021-08-23 12:41     ` Aaron Conole
  0 siblings, 1 reply; 5+ messages in thread
From: Thomas Monjalon @ 2021-08-13 13:21 UTC (permalink / raw)
  To: Wenjun Wu, dev, qiming.yang, qi.z.zhang, yuying.zhang, Ferruh Yigit
  Cc: David Marchand, Aaron Conole, dpdklab

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.



^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [dpdk-dev] [PATCH v1] net/ice: fix the reversed priority of DCF switch rule
  2021-08-13 13:21   ` Thomas Monjalon
@ 2021-08-23 12:41     ` Aaron Conole
  0 siblings, 0 replies; 5+ messages in thread
From: Aaron Conole @ 2021-08-23 12:41 UTC (permalink / raw)
  To: Thomas Monjalon
  Cc: Wenjun Wu, dev, qiming.yang, qi.z.zhang, yuying.zhang,
	Ferruh Yigit, David Marchand, dpdklab

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.


^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2021-08-23 12:41 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-08-02  7:25 [dpdk-dev] [PATCH v1] net/ice: fix the reversed priority of DCF switch rule Wenjun Wu
2021-08-10  3:05 ` Zhang, Qi Z
2021-08-13 10:45 ` Ferruh Yigit
2021-08-13 13:21   ` Thomas Monjalon
2021-08-23 12:41     ` Aaron Conole

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).