Re: [dpdk-dev] [PATCH 2/3] net/ice: fix flow action validation

2019-07-18 Thread Zhang, Qi Z
> -Original Message- > From: Yang, Qiming > Sent: Thursday, July 18, 2019 11:07 AM > To: Wang, Ying A ; Zhang, Qi Z > > Cc: dev@dpdk.org; sta...@dpdk.org > Subject: RE: [PATCH 2/3] net/ice: fix flow action validation > > > > -Original Message- > From: Wang, Ying A > Sent: Thu

Re: [dpdk-dev] [PATCH 2/3] net/ice: fix flow action validation

2019-07-17 Thread Yang, Qiming
-Original Message- From: Wang, Ying A Sent: Thursday, July 18, 2019 9:39 AM To: Zhang, Qi Z Cc: Yang, Qiming ; dev@dpdk.org; Wang, Ying A ; sta...@dpdk.org Subject: [PATCH 2/3] net/ice: fix flow action validation Action is a list. We should check each element of the action rather tha

Re: [dpdk-dev] [PATCH 2/3] net/ice: fix flow action validation

2019-07-17 Thread Yang, Qiming
Hi, Ying For the action check reorganization, please according to Beilei's comments, use array to stand the action element list. And as we discussed before, it's better to change in the next release. Qiming -Original Message- From: Wang, Ying A Sent: Thursday, July 18, 2019 9:39 AM To:

Re: [dpdk-dev] [PATCH 2/3] net/ice: fix flow action validation

2019-07-17 Thread Ye Xiaolong
On 07/18, Wang Ying A wrote: >Action is a list. We should check each element of the action >rather than the first one. >This patch fixes this issue. > >Fixes: d76116a4678f ("net/ice: add generic flow API") >Cc: sta...@dpdk.org > >Signed-off-by: Wang Ying A >--- > drivers/net/ice/ice_generic_flow.c

[dpdk-dev] [PATCH 2/3] net/ice: fix flow action validation

2019-07-17 Thread Wang Ying A
Action is a list. We should check each element of the action rather than the first one. This patch fixes this issue. Fixes: d76116a4678f ("net/ice: add generic flow API") Cc: sta...@dpdk.org Signed-off-by: Wang Ying A --- drivers/net/ice/ice_generic_flow.c | 39 -