diff mbox series

ethdev: fix flow API support for P4-programmable devices

Message ID 20231027091742.3755532-1-wenjing.qiao@intel.com (mailing list archive)
State Accepted, archived
Delegated to: Ferruh Yigit
Headers
Series ethdev: fix flow API support for P4-programmable devices |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/loongarch-compilation success Compilation OK
ci/loongarch-unit-testing success Unit Testing PASS
ci/github-robot: build success github build: passed
ci/Intel-compilation success Compilation OK
ci/intel-Testing success Testing PASS
ci/iol-intel-Performance success Performance Testing PASS
ci/iol-mellanox-Performance success Performance Testing PASS
ci/iol-compile-amd64-testing success Testing PASS
ci/iol-intel-Functional success Functional Testing PASS
ci/iol-unit-amd64-testing success Testing PASS
ci/iol-unit-arm64-testing success Testing PASS
ci/iol-broadcom-Functional success Functional Testing PASS
ci/iol-broadcom-Performance success Performance Testing PASS
ci/iol-compile-arm64-testing success Testing PASS
ci/intel-Functional success Functional PASS
ci/iol-sample-apps-testing success Testing PASS

Commit Message

Wenjing Qiao Oct. 27, 2023, 9:17 a.m. UTC 
  From: Wenjing Qiao <wenjing.qiao@intel.com>

"program" action should also be added in structure rte_flow_desc_action.

Fixes: 8f1953f1914d ("ethdev: add flow API for P4-programmable devices")

Signed-off-by: Wenjing Qiao <wenjing.qiao@intel.com>
---
 lib/ethdev/rte_flow.c | 2 ++
 1 file changed, 2 insertions(+)

Comments

Ori Kam Oct. 29, 2023, 7:17 a.m. UTC | #1 
Hi Wenjing,

> -----Original Message-----
> From: wenjing.qiao@intel.com <wenjing.qiao@intel.com>
> Sent: Friday, October 27, 2023 12:18 PM
> 
> From: Wenjing Qiao <wenjing.qiao@intel.com>
> 
> "program" action should also be added in structure rte_flow_desc_action.
> 
> Fixes: 8f1953f1914d ("ethdev: add flow API for P4-programmable devices")
> 
> Signed-off-by: Wenjing Qiao <wenjing.qiao@intel.com>
> ---
>  lib/ethdev/rte_flow.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/lib/ethdev/rte_flow.c b/lib/ethdev/rte_flow.c
> index 3a67f1aaba..4d6c28ee0e 100644
> --- a/lib/ethdev/rte_flow.c
> +++ b/lib/ethdev/rte_flow.c
> @@ -267,6 +267,8 @@ static const struct rte_flow_desc_data
> rte_flow_desc_action[] = {
>  	MK_FLOW_ACTION(IPV6_EXT_REMOVE, sizeof(struct
> rte_flow_action_ipv6_ext_remove)),
>  	MK_FLOW_ACTION(INDIRECT_LIST,
>  		       sizeof(struct rte_flow_action_indirect_list)),
> +	MK_FLOW_ACTION(PROG,
> +		       sizeof(struct rte_flow_action_prog)),
>  };
> 
>  int
> --
> 2.34.1

Acked-by: Ori Kam <orika@nvidia.com>
Best,
Ori
Ferruh Yigit Oct. 31, 2023, 11:04 a.m. UTC | #2 
On 10/29/2023 7:17 AM, Ori Kam wrote:
> Hi Wenjing,
> 
>> -----Original Message-----
>> From: wenjing.qiao@intel.com <wenjing.qiao@intel.com>
>> Sent: Friday, October 27, 2023 12:18 PM
>>
>> From: Wenjing Qiao <wenjing.qiao@intel.com>
>>
>> "program" action should also be added in structure rte_flow_desc_action.
>>
>> Fixes: 8f1953f1914d ("ethdev: add flow API for P4-programmable devices")
>>
>> Signed-off-by: Wenjing Qiao <wenjing.qiao@intel.com>
> Acked-by: Ori Kam <orika@nvidia.com>
> 
> 

Applied to dpdk-next-net/main, thanks.
diff mbox series

Patch

diff --git a/lib/ethdev/rte_flow.c b/lib/ethdev/rte_flow.c
index 3a67f1aaba..4d6c28ee0e 100644
--- a/lib/ethdev/rte_flow.c
+++ b/lib/ethdev/rte_flow.c
@@ -267,6 +267,8 @@  static const struct rte_flow_desc_data rte_flow_desc_action[] = {
 	MK_FLOW_ACTION(IPV6_EXT_REMOVE, sizeof(struct rte_flow_action_ipv6_ext_remove)),
 	MK_FLOW_ACTION(INDIRECT_LIST,
 		       sizeof(struct rte_flow_action_indirect_list)),
+	MK_FLOW_ACTION(PROG,
+		       sizeof(struct rte_flow_action_prog)),
 };
 
 int