DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH v1 1/4] common/iavf: add QFI fields for GTPU UL and DL
@ 2021-08-17  7:16 Haiyue Wang
  2021-08-17  7:16 ` [dpdk-dev] [PATCH v1 2/4] common/iavf: add proto hdr field support for L4 checksum Haiyue Wang
                   ` (6 more replies)
  0 siblings, 7 replies; 21+ messages in thread
From: Haiyue Wang @ 2021-08-17  7:16 UTC (permalink / raw)
  To: dev; +Cc: Junfeng Guo, Haiyue Wang, Jingjing Wu, Beilei Xing

From: Junfeng Guo <junfeng.guo@intel.com>

Add virtchnl fields QFI of GTPU UL/DL for AVF FDIR.

Signed-off-by: Junfeng Guo <junfeng.guo@intel.com>
Signed-off-by: Haiyue Wang <haiyue.wang@intel.com>
---
 drivers/common/iavf/virtchnl.h | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/drivers/common/iavf/virtchnl.h b/drivers/common/iavf/virtchnl.h
index 1cf0866124..9fa5e3e891 100644
--- a/drivers/common/iavf/virtchnl.h
+++ b/drivers/common/iavf/virtchnl.h
@@ -1642,6 +1642,11 @@ enum virtchnl_proto_hdr_field {
 	/* IPv6 Extension Fragment */
 	VIRTCHNL_PROTO_HDR_IPV6_EH_FRAG_PKID =
 		PROTO_HDR_FIELD_START(VIRTCHNL_PROTO_HDR_IPV6_EH_FRAG),
+	/* GTPU_DWN/UP */
+	VIRTCHNL_PROTO_HDR_GTPU_DWN_QFI =
+		PROTO_HDR_FIELD_START(VIRTCHNL_PROTO_HDR_GTPU_EH_PDU_DWN),
+	VIRTCHNL_PROTO_HDR_GTPU_UP_QFI =
+		PROTO_HDR_FIELD_START(VIRTCHNL_PROTO_HDR_GTPU_EH_PDU_UP),
 };
 
 struct virtchnl_proto_hdr {
-- 
2.33.0


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

* [dpdk-dev] [PATCH v1 2/4] common/iavf: add proto hdr field support for L4 checksum
  2021-08-17  7:16 [dpdk-dev] [PATCH v1 1/4] common/iavf: add QFI fields for GTPU UL and DL Haiyue Wang
@ 2021-08-17  7:16 ` Haiyue Wang
  2021-08-29 11:38   ` Zhang, Qi Z
  2021-09-03 10:43   ` Ferruh Yigit
  2021-08-17  7:16 ` [dpdk-dev] [PATCH v1 3/4] common/iavf: remove the FDIR query opcode Haiyue Wang
                   ` (5 subsequent siblings)
  6 siblings, 2 replies; 21+ messages in thread
From: Haiyue Wang @ 2021-08-17  7:16 UTC (permalink / raw)
  To: dev; +Cc: Alvin Zhang, Haiyue Wang, Jingjing Wu, Beilei Xing

From: Alvin Zhang <alvinx.zhang@intel.com>

Add TCP/UDP/SCTP header checksum field selectors.

Signed-off-by: Alvin Zhang <alvinx.zhang@intel.com>
Signed-off-by: Haiyue Wang <haiyue.wang@intel.com>
---
 drivers/common/iavf/virtchnl.h | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/drivers/common/iavf/virtchnl.h b/drivers/common/iavf/virtchnl.h
index 9fa5e3e891..c56c668cff 100644
--- a/drivers/common/iavf/virtchnl.h
+++ b/drivers/common/iavf/virtchnl.h
@@ -1598,14 +1598,17 @@ enum virtchnl_proto_hdr_field {
 	VIRTCHNL_PROTO_HDR_TCP_SRC_PORT =
 		PROTO_HDR_FIELD_START(VIRTCHNL_PROTO_HDR_TCP),
 	VIRTCHNL_PROTO_HDR_TCP_DST_PORT,
+	VIRTCHNL_PROTO_HDR_TCP_CHKSUM,
 	/* UDP */
 	VIRTCHNL_PROTO_HDR_UDP_SRC_PORT =
 		PROTO_HDR_FIELD_START(VIRTCHNL_PROTO_HDR_UDP),
 	VIRTCHNL_PROTO_HDR_UDP_DST_PORT,
+	VIRTCHNL_PROTO_HDR_UDP_CHKSUM,
 	/* SCTP */
 	VIRTCHNL_PROTO_HDR_SCTP_SRC_PORT =
 		PROTO_HDR_FIELD_START(VIRTCHNL_PROTO_HDR_SCTP),
 	VIRTCHNL_PROTO_HDR_SCTP_DST_PORT,
+	VIRTCHNL_PROTO_HDR_SCTP_CHKSUM,
 	/* GTPU_IP */
 	VIRTCHNL_PROTO_HDR_GTPU_IP_TEID =
 		PROTO_HDR_FIELD_START(VIRTCHNL_PROTO_HDR_GTPU_IP),
-- 
2.33.0


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

* [dpdk-dev] [PATCH v1 3/4] common/iavf: remove the FDIR query opcode
  2021-08-17  7:16 [dpdk-dev] [PATCH v1 1/4] common/iavf: add QFI fields for GTPU UL and DL Haiyue Wang
  2021-08-17  7:16 ` [dpdk-dev] [PATCH v1 2/4] common/iavf: add proto hdr field support for L4 checksum Haiyue Wang
@ 2021-08-17  7:16 ` Haiyue Wang
  2021-08-29 11:38   ` Zhang, Qi Z
  2021-08-17  7:16 ` [dpdk-dev] [PATCH v1 4/4] common/iavf: update the driver version Haiyue Wang
                   ` (4 subsequent siblings)
  6 siblings, 1 reply; 21+ messages in thread
From: Haiyue Wang @ 2021-08-17  7:16 UTC (permalink / raw)
  To: dev; +Cc: Haiyue Wang, Jingjing Wu, Beilei Xing

The VIRTCHNL_OP_QUERY_FDIR_FILTER opcode is not used, so remove it.

Signed-off-by: Haiyue Wang <haiyue.wang@intel.com>
---
 drivers/common/iavf/virtchnl.h | 38 ----------------------------------
 1 file changed, 38 deletions(-)

diff --git a/drivers/common/iavf/virtchnl.h b/drivers/common/iavf/virtchnl.h
index c56c668cff..83f51d889f 100644
--- a/drivers/common/iavf/virtchnl.h
+++ b/drivers/common/iavf/virtchnl.h
@@ -146,7 +146,6 @@ enum virtchnl_ops {
 	VIRTCHNL_OP_DEL_RSS_CFG = 46,
 	VIRTCHNL_OP_ADD_FDIR_FILTER = 47,
 	VIRTCHNL_OP_DEL_FDIR_FILTER = 48,
-	VIRTCHNL_OP_QUERY_FDIR_FILTER = 49,
 	VIRTCHNL_OP_GET_MAX_RSS_QREGION = 50,
 	VIRTCHNL_OP_GET_OFFLOAD_VLAN_V2_CAPS = 51,
 	VIRTCHNL_OP_ADD_VLAN_V2 = 52,
@@ -244,8 +243,6 @@ static inline const char *virtchnl_op_str(enum virtchnl_ops v_opcode)
 		return "VIRTCHNL_OP_ADD_FDIR_FILTER";
 	case VIRTCHNL_OP_DEL_FDIR_FILTER:
 		return "VIRTCHNL_OP_DEL_FDIR_FILTER";
-	case VIRTCHNL_OP_QUERY_FDIR_FILTER:
-		return "VIRTCHNL_OP_QUERY_FDIR_FILTER";
 	case VIRTCHNL_OP_GET_MAX_RSS_QREGION:
 		return "VIRTCHNL_OP_GET_MAX_RSS_QREGION";
 	case VIRTCHNL_OP_ENABLE_QUEUES_V2:
@@ -1733,20 +1730,6 @@ struct virtchnl_fdir_rule {
 
 VIRTCHNL_CHECK_STRUCT_LEN(2604, virtchnl_fdir_rule);
 
-/* query information to retrieve fdir rule counters.
- * PF will fill out this structure to reset counter.
- */
-struct virtchnl_fdir_query_info {
-	u32 match_packets_valid:1;
-	u32 match_bytes_valid:1;
-	u32 reserved:30;  /* Reserved, must be zero. */
-	u32 pad;
-	u64 matched_packets; /* Number of packets for this rule. */
-	u64 matched_bytes;   /* Number of bytes through this rule. */
-};
-
-VIRTCHNL_CHECK_STRUCT_LEN(24, virtchnl_fdir_query_info);
-
 /* Status returned to VF after VF requests FDIR commands
  * VIRTCHNL_FDIR_SUCCESS
  * VF FDIR related request is successfully done by PF
@@ -1879,24 +1862,6 @@ struct virtchnl_queue_tc_mapping {
 
 VIRTCHNL_CHECK_STRUCT_LEN(12, virtchnl_queue_tc_mapping);
 
-/* VIRTCHNL_OP_QUERY_FDIR_FILTER
- * VF sends this request to PF by filling out vsi_id,
- * flow_id and reset_counter. PF will return query_info
- * and query_status to VF.
- */
-struct virtchnl_fdir_query {
-	u16 vsi_id;   /* INPUT */
-	u16 pad1[3];
-	u32 flow_id;  /* INPUT */
-	u32 reset_counter:1; /* INPUT */
-	struct virtchnl_fdir_query_info query_info; /* OUTPUT */
-
-	/* see enum virtchnl_fdir_prgm_status; OUTPUT */
-	s32 status;
-	u32 pad2;
-};
-
-VIRTCHNL_CHECK_STRUCT_LEN(48, virtchnl_fdir_query);
 
 /* TX and RX queue types are valid in legacy as well as split queue models.
  * With Split Queue model, 2 additional types are introduced - TX_COMPLETION
@@ -2254,9 +2219,6 @@ virtchnl_vc_validate_vf_msg(struct virtchnl_version_info *ver, u32 v_opcode,
 	case VIRTCHNL_OP_DEL_FDIR_FILTER:
 		valid_len = sizeof(struct virtchnl_fdir_del);
 		break;
-	case VIRTCHNL_OP_QUERY_FDIR_FILTER:
-		valid_len = sizeof(struct virtchnl_fdir_query);
-		break;
 	case VIRTCHNL_OP_GET_QOS_CAPS:
 		break;
 	case VIRTCHNL_OP_CONFIG_QUEUE_TC_MAP:
-- 
2.33.0


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

* [dpdk-dev] [PATCH v1 4/4] common/iavf: update the driver version
  2021-08-17  7:16 [dpdk-dev] [PATCH v1 1/4] common/iavf: add QFI fields for GTPU UL and DL Haiyue Wang
  2021-08-17  7:16 ` [dpdk-dev] [PATCH v1 2/4] common/iavf: add proto hdr field support for L4 checksum Haiyue Wang
  2021-08-17  7:16 ` [dpdk-dev] [PATCH v1 3/4] common/iavf: remove the FDIR query opcode Haiyue Wang
@ 2021-08-17  7:16 ` Haiyue Wang
  2021-08-29 11:38   ` Zhang, Qi Z
  2021-08-29 11:37 ` [dpdk-dev] [PATCH v1 1/4] common/iavf: add QFI fields for GTPU UL and DL Zhang, Qi Z
                   ` (3 subsequent siblings)
  6 siblings, 1 reply; 21+ messages in thread
From: Haiyue Wang @ 2021-08-17  7:16 UTC (permalink / raw)
  To: dev; +Cc: Haiyue Wang, Jingjing Wu, Beilei Xing

Update the driver version to trace the change.

Signed-off-by: Haiyue Wang <haiyue.wang@intel.com>
---
 drivers/common/iavf/README | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/common/iavf/README b/drivers/common/iavf/README
index 611fdcea94..89bdbc827e 100644
--- a/drivers/common/iavf/README
+++ b/drivers/common/iavf/README
@@ -6,7 +6,7 @@ Intel® IAVF driver
 =================
 
 This directory contains source code of FreeBSD IAVF driver of version
-cid-avf.2021.04.29.tar.gz released by the team which develops
+cid-avf.2021.08.16.tar.gz released by the team which develops
 basic drivers for any IAVF NIC. The directory of base/ contains the
 original source package.
 
-- 
2.33.0


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

* Re: [dpdk-dev] [PATCH v1 1/4] common/iavf: add QFI fields for GTPU UL and DL
  2021-08-17  7:16 [dpdk-dev] [PATCH v1 1/4] common/iavf: add QFI fields for GTPU UL and DL Haiyue Wang
                   ` (2 preceding siblings ...)
  2021-08-17  7:16 ` [dpdk-dev] [PATCH v1 4/4] common/iavf: update the driver version Haiyue Wang
@ 2021-08-29 11:37 ` Zhang, Qi Z
  2021-09-03 10:41 ` Ferruh Yigit
                   ` (2 subsequent siblings)
  6 siblings, 0 replies; 21+ messages in thread
From: Zhang, Qi Z @ 2021-08-29 11:37 UTC (permalink / raw)
  To: Wang, Haiyue, dev; +Cc: Guo, Junfeng, Wang, Haiyue, Wu, Jingjing, Xing, Beilei



> -----Original Message-----
> From: dev <dev-bounces@dpdk.org> On Behalf Of Haiyue Wang
> Sent: Tuesday, August 17, 2021 3:17 PM
> To: dev@dpdk.org
> Cc: Guo, Junfeng <junfeng.guo@intel.com>; Wang, Haiyue
> <haiyue.wang@intel.com>; Wu, Jingjing <jingjing.wu@intel.com>; Xing, Beilei
> <beilei.xing@intel.com>
> Subject: [dpdk-dev] [PATCH v1 1/4] common/iavf: add QFI fields for GTPU UL
> and DL
> 
> From: Junfeng Guo <junfeng.guo@intel.com>
> 
> Add virtchnl fields QFI of GTPU UL/DL for AVF FDIR.
> 
> Signed-off-by: Junfeng Guo <junfeng.guo@intel.com>
> Signed-off-by: Haiyue Wang <haiyue.wang@intel.com>

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

Applied to dpdk-next-net-intel.

Thanks
Qi


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

* Re: [dpdk-dev] [PATCH v1 2/4] common/iavf: add proto hdr field support for L4 checksum
  2021-08-17  7:16 ` [dpdk-dev] [PATCH v1 2/4] common/iavf: add proto hdr field support for L4 checksum Haiyue Wang
@ 2021-08-29 11:38   ` Zhang, Qi Z
  2021-09-03 10:43   ` Ferruh Yigit
  1 sibling, 0 replies; 21+ messages in thread
From: Zhang, Qi Z @ 2021-08-29 11:38 UTC (permalink / raw)
  To: Wang, Haiyue, dev; +Cc: Zhang, AlvinX, Wang, Haiyue, Wu, Jingjing, Xing, Beilei



> -----Original Message-----
> From: dev <dev-bounces@dpdk.org> On Behalf Of Haiyue Wang
> Sent: Tuesday, August 17, 2021 3:17 PM
> To: dev@dpdk.org
> Cc: Zhang, AlvinX <alvinx.zhang@intel.com>; Wang, Haiyue
> <haiyue.wang@intel.com>; Wu, Jingjing <jingjing.wu@intel.com>; Xing, Beilei
> <beilei.xing@intel.com>
> Subject: [dpdk-dev] [PATCH v1 2/4] common/iavf: add proto hdr field support for
> L4 checksum
> 
> From: Alvin Zhang <alvinx.zhang@intel.com>
> 
> Add TCP/UDP/SCTP header checksum field selectors.
> 
> Signed-off-by: Alvin Zhang <alvinx.zhang@intel.com>
> Signed-off-by: Haiyue Wang <haiyue.wang@intel.com>

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

Applied to dpdk-next-net-intel.

Thanks
Qi


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

* Re: [dpdk-dev] [PATCH v1 3/4] common/iavf: remove the FDIR query opcode
  2021-08-17  7:16 ` [dpdk-dev] [PATCH v1 3/4] common/iavf: remove the FDIR query opcode Haiyue Wang
@ 2021-08-29 11:38   ` Zhang, Qi Z
  0 siblings, 0 replies; 21+ messages in thread
From: Zhang, Qi Z @ 2021-08-29 11:38 UTC (permalink / raw)
  To: Wang, Haiyue, dev; +Cc: Wang, Haiyue, Wu, Jingjing, Xing, Beilei



> -----Original Message-----
> From: dev <dev-bounces@dpdk.org> On Behalf Of Haiyue Wang
> Sent: Tuesday, August 17, 2021 3:17 PM
> To: dev@dpdk.org
> Cc: Wang, Haiyue <haiyue.wang@intel.com>; Wu, Jingjing
> <jingjing.wu@intel.com>; Xing, Beilei <beilei.xing@intel.com>
> Subject: [dpdk-dev] [PATCH v1 3/4] common/iavf: remove the FDIR query
> opcode
> 
> The VIRTCHNL_OP_QUERY_FDIR_FILTER opcode is not used, so remove it.
> 
> Signed-off-by: Haiyue Wang <haiyue.wang@intel.com>

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

Applied to dpdk-next-net-intel.

Thanks
Qi


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

* Re: [dpdk-dev] [PATCH v1 4/4] common/iavf: update the driver version
  2021-08-17  7:16 ` [dpdk-dev] [PATCH v1 4/4] common/iavf: update the driver version Haiyue Wang
@ 2021-08-29 11:38   ` Zhang, Qi Z
  0 siblings, 0 replies; 21+ messages in thread
From: Zhang, Qi Z @ 2021-08-29 11:38 UTC (permalink / raw)
  To: Wang, Haiyue, dev; +Cc: Wang, Haiyue, Wu, Jingjing, Xing, Beilei



> -----Original Message-----
> From: dev <dev-bounces@dpdk.org> On Behalf Of Haiyue Wang
> Sent: Tuesday, August 17, 2021 3:17 PM
> To: dev@dpdk.org
> Cc: Wang, Haiyue <haiyue.wang@intel.com>; Wu, Jingjing
> <jingjing.wu@intel.com>; Xing, Beilei <beilei.xing@intel.com>
> Subject: [dpdk-dev] [PATCH v1 4/4] common/iavf: update the driver version
> 
> Update the driver version to trace the change.
> 
> Signed-off-by: Haiyue Wang <haiyue.wang@intel.com>

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

Applied to dpdk-next-net-intel.

Thanks
Qi


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

* Re: [dpdk-dev] [PATCH v1 1/4] common/iavf: add QFI fields for GTPU UL and DL
  2021-08-17  7:16 [dpdk-dev] [PATCH v1 1/4] common/iavf: add QFI fields for GTPU UL and DL Haiyue Wang
                   ` (3 preceding siblings ...)
  2021-08-29 11:37 ` [dpdk-dev] [PATCH v1 1/4] common/iavf: add QFI fields for GTPU UL and DL Zhang, Qi Z
@ 2021-09-03 10:41 ` Ferruh Yigit
  2021-09-06  5:51 ` [dpdk-dev] [PATCH v2 0/4] iavf base code update Haiyue Wang
  2021-09-07  7:33 ` [dpdk-dev] [PATCH v3 0/4] iavf base code update Haiyue Wang
  6 siblings, 0 replies; 21+ messages in thread
From: Ferruh Yigit @ 2021-09-03 10:41 UTC (permalink / raw)
  To: Haiyue Wang, dev; +Cc: Junfeng Guo, Jingjing Wu, Beilei Xing

On 8/17/2021 8:16 AM, Haiyue Wang wrote:
> From: Junfeng Guo <junfeng.guo@intel.com>
> 
> Add virtchnl fields QFI of GTPU UL/DL for AVF FDIR.

Can you please put what 'QFI' stands for in the commit log, I assume it is "QoS
Flow Identifier"?

What does it mean to add new protocol fields to the virtchnl?
And commit log mentions from 'FDIR', is this change updates anything in the flow
director support of the driver?

> 
> Signed-off-by: Junfeng Guo <junfeng.guo@intel.com>
> Signed-off-by: Haiyue Wang <haiyue.wang@intel.com>
> ---
>  drivers/common/iavf/virtchnl.h | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/drivers/common/iavf/virtchnl.h b/drivers/common/iavf/virtchnl.h
> index 1cf0866124..9fa5e3e891 100644
> --- a/drivers/common/iavf/virtchnl.h
> +++ b/drivers/common/iavf/virtchnl.h
> @@ -1642,6 +1642,11 @@ enum virtchnl_proto_hdr_field {
>  	/* IPv6 Extension Fragment */
>  	VIRTCHNL_PROTO_HDR_IPV6_EH_FRAG_PKID =
>  		PROTO_HDR_FIELD_START(VIRTCHNL_PROTO_HDR_IPV6_EH_FRAG),
> +	/* GTPU_DWN/UP */
> +	VIRTCHNL_PROTO_HDR_GTPU_DWN_QFI =
> +		PROTO_HDR_FIELD_START(VIRTCHNL_PROTO_HDR_GTPU_EH_PDU_DWN),
> +	VIRTCHNL_PROTO_HDR_GTPU_UP_QFI =
> +		PROTO_HDR_FIELD_START(VIRTCHNL_PROTO_HDR_GTPU_EH_PDU_UP),
>  };
>  
>  struct virtchnl_proto_hdr {
> 


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

* Re: [dpdk-dev] [PATCH v1 2/4] common/iavf: add proto hdr field support for L4 checksum
  2021-08-17  7:16 ` [dpdk-dev] [PATCH v1 2/4] common/iavf: add proto hdr field support for L4 checksum Haiyue Wang
  2021-08-29 11:38   ` Zhang, Qi Z
@ 2021-09-03 10:43   ` Ferruh Yigit
  1 sibling, 0 replies; 21+ messages in thread
From: Ferruh Yigit @ 2021-09-03 10:43 UTC (permalink / raw)
  To: Haiyue Wang, dev; +Cc: Alvin Zhang, Jingjing Wu, Beilei Xing

On 8/17/2021 8:16 AM, Haiyue Wang wrote:
> From: Alvin Zhang <alvinx.zhang@intel.com>
> 
> Add TCP/UDP/SCTP header checksum field selectors.
> 

Hi Alvin, Haiyue,

Can you please describe a little more, what is the impact of adding proto hdr
checksum field selector?

Is this also something related to the flow director as other patches in this set?

Thanks,
ferruh

> Signed-off-by: Alvin Zhang <alvinx.zhang@intel.com>
> Signed-off-by: Haiyue Wang <haiyue.wang@intel.com>
> ---
>  drivers/common/iavf/virtchnl.h | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/drivers/common/iavf/virtchnl.h b/drivers/common/iavf/virtchnl.h
> index 9fa5e3e891..c56c668cff 100644
> --- a/drivers/common/iavf/virtchnl.h
> +++ b/drivers/common/iavf/virtchnl.h
> @@ -1598,14 +1598,17 @@ enum virtchnl_proto_hdr_field {
>  	VIRTCHNL_PROTO_HDR_TCP_SRC_PORT =
>  		PROTO_HDR_FIELD_START(VIRTCHNL_PROTO_HDR_TCP),
>  	VIRTCHNL_PROTO_HDR_TCP_DST_PORT,
> +	VIRTCHNL_PROTO_HDR_TCP_CHKSUM,
>  	/* UDP */
>  	VIRTCHNL_PROTO_HDR_UDP_SRC_PORT =
>  		PROTO_HDR_FIELD_START(VIRTCHNL_PROTO_HDR_UDP),
>  	VIRTCHNL_PROTO_HDR_UDP_DST_PORT,
> +	VIRTCHNL_PROTO_HDR_UDP_CHKSUM,
>  	/* SCTP */
>  	VIRTCHNL_PROTO_HDR_SCTP_SRC_PORT =
>  		PROTO_HDR_FIELD_START(VIRTCHNL_PROTO_HDR_SCTP),
>  	VIRTCHNL_PROTO_HDR_SCTP_DST_PORT,
> +	VIRTCHNL_PROTO_HDR_SCTP_CHKSUM,
>  	/* GTPU_IP */
>  	VIRTCHNL_PROTO_HDR_GTPU_IP_TEID =
>  		PROTO_HDR_FIELD_START(VIRTCHNL_PROTO_HDR_GTPU_IP),
> 


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

* [dpdk-dev] [PATCH v2 0/4] iavf base code update
  2021-08-17  7:16 [dpdk-dev] [PATCH v1 1/4] common/iavf: add QFI fields for GTPU UL and DL Haiyue Wang
                   ` (4 preceding siblings ...)
  2021-09-03 10:41 ` Ferruh Yigit
@ 2021-09-06  5:51 ` Haiyue Wang
  2021-09-06  5:51   ` [dpdk-dev] [PATCH v2 1/4] common/iavf: add QFI fields for GTPU UL and DL Haiyue Wang
                     ` (3 more replies)
  2021-09-07  7:33 ` [dpdk-dev] [PATCH v3 0/4] iavf base code update Haiyue Wang
  6 siblings, 4 replies; 21+ messages in thread
From: Haiyue Wang @ 2021-09-06  5:51 UTC (permalink / raw)
  To: dev; +Cc: Haiyue Wang

v2: update the commit message.

Alvin Zhang (1):
  common/iavf: add proto hdr field support for L4 checksum

Haiyue Wang (2):
  common/iavf: remove the FDIR query opcode
  common/iavf: update the driver version

Junfeng Guo (1):
  common/iavf: add QFI fields for GTPU UL and DL

 drivers/common/iavf/README     |  2 +-
 drivers/common/iavf/virtchnl.h | 46 ++++++----------------------------
 2 files changed, 9 insertions(+), 39 deletions(-)

-- 
2.33.0


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

* [dpdk-dev] [PATCH v2 1/4] common/iavf: add QFI fields for GTPU UL and DL
  2021-09-06  5:51 ` [dpdk-dev] [PATCH v2 0/4] iavf base code update Haiyue Wang
@ 2021-09-06  5:51   ` Haiyue Wang
  2021-09-06  5:51   ` [dpdk-dev] [PATCH v2 2/4] common/iavf: add proto hdr field support for L4 checksum Haiyue Wang
                     ` (2 subsequent siblings)
  3 siblings, 0 replies; 21+ messages in thread
From: Haiyue Wang @ 2021-09-06  5:51 UTC (permalink / raw)
  To: dev; +Cc: Junfeng Guo, Haiyue Wang, Jingjing Wu, Beilei Xing

From: Junfeng Guo <junfeng.guo@intel.com>

The QFI is 6-bit "QoS Flow Identifier" within the GTPU Extension Header.
Add virtchnl fields QFI of GTPU UL/DL for supporting the AVF FDIR.

Signed-off-by: Junfeng Guo <junfeng.guo@intel.com>
Signed-off-by: Haiyue Wang <haiyue.wang@intel.com>
---
 drivers/common/iavf/virtchnl.h | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/drivers/common/iavf/virtchnl.h b/drivers/common/iavf/virtchnl.h
index 1cf0866124..9fa5e3e891 100644
--- a/drivers/common/iavf/virtchnl.h
+++ b/drivers/common/iavf/virtchnl.h
@@ -1642,6 +1642,11 @@ enum virtchnl_proto_hdr_field {
 	/* IPv6 Extension Fragment */
 	VIRTCHNL_PROTO_HDR_IPV6_EH_FRAG_PKID =
 		PROTO_HDR_FIELD_START(VIRTCHNL_PROTO_HDR_IPV6_EH_FRAG),
+	/* GTPU_DWN/UP */
+	VIRTCHNL_PROTO_HDR_GTPU_DWN_QFI =
+		PROTO_HDR_FIELD_START(VIRTCHNL_PROTO_HDR_GTPU_EH_PDU_DWN),
+	VIRTCHNL_PROTO_HDR_GTPU_UP_QFI =
+		PROTO_HDR_FIELD_START(VIRTCHNL_PROTO_HDR_GTPU_EH_PDU_UP),
 };
 
 struct virtchnl_proto_hdr {
-- 
2.33.0


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

* [dpdk-dev] [PATCH v2 2/4] common/iavf: add proto hdr field support for L4 checksum
  2021-09-06  5:51 ` [dpdk-dev] [PATCH v2 0/4] iavf base code update Haiyue Wang
  2021-09-06  5:51   ` [dpdk-dev] [PATCH v2 1/4] common/iavf: add QFI fields for GTPU UL and DL Haiyue Wang
@ 2021-09-06  5:51   ` Haiyue Wang
  2021-09-06  5:51   ` [dpdk-dev] [PATCH v2 3/4] common/iavf: remove the FDIR query opcode Haiyue Wang
  2021-09-06  5:51   ` [dpdk-dev] [PATCH v2 4/4] common/iavf: update the driver version Haiyue Wang
  3 siblings, 0 replies; 21+ messages in thread
From: Haiyue Wang @ 2021-09-06  5:51 UTC (permalink / raw)
  To: dev; +Cc: Alvin Zhang, Haiyue Wang, Jingjing Wu, Beilei Xing

From: Alvin Zhang <alvinx.zhang@intel.com>

Add TCP/UDP/SCTP header checksum field selectors, they can be used in
creating FDIR or RSS rules related to TCP/UDP/SCTP header checksum.

Signed-off-by: Alvin Zhang <alvinx.zhang@intel.com>
Signed-off-by: Haiyue Wang <haiyue.wang@intel.com>
---
 drivers/common/iavf/virtchnl.h | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/drivers/common/iavf/virtchnl.h b/drivers/common/iavf/virtchnl.h
index 9fa5e3e891..c56c668cff 100644
--- a/drivers/common/iavf/virtchnl.h
+++ b/drivers/common/iavf/virtchnl.h
@@ -1598,14 +1598,17 @@ enum virtchnl_proto_hdr_field {
 	VIRTCHNL_PROTO_HDR_TCP_SRC_PORT =
 		PROTO_HDR_FIELD_START(VIRTCHNL_PROTO_HDR_TCP),
 	VIRTCHNL_PROTO_HDR_TCP_DST_PORT,
+	VIRTCHNL_PROTO_HDR_TCP_CHKSUM,
 	/* UDP */
 	VIRTCHNL_PROTO_HDR_UDP_SRC_PORT =
 		PROTO_HDR_FIELD_START(VIRTCHNL_PROTO_HDR_UDP),
 	VIRTCHNL_PROTO_HDR_UDP_DST_PORT,
+	VIRTCHNL_PROTO_HDR_UDP_CHKSUM,
 	/* SCTP */
 	VIRTCHNL_PROTO_HDR_SCTP_SRC_PORT =
 		PROTO_HDR_FIELD_START(VIRTCHNL_PROTO_HDR_SCTP),
 	VIRTCHNL_PROTO_HDR_SCTP_DST_PORT,
+	VIRTCHNL_PROTO_HDR_SCTP_CHKSUM,
 	/* GTPU_IP */
 	VIRTCHNL_PROTO_HDR_GTPU_IP_TEID =
 		PROTO_HDR_FIELD_START(VIRTCHNL_PROTO_HDR_GTPU_IP),
-- 
2.33.0


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

* [dpdk-dev] [PATCH v2 3/4] common/iavf: remove the FDIR query opcode
  2021-09-06  5:51 ` [dpdk-dev] [PATCH v2 0/4] iavf base code update Haiyue Wang
  2021-09-06  5:51   ` [dpdk-dev] [PATCH v2 1/4] common/iavf: add QFI fields for GTPU UL and DL Haiyue Wang
  2021-09-06  5:51   ` [dpdk-dev] [PATCH v2 2/4] common/iavf: add proto hdr field support for L4 checksum Haiyue Wang
@ 2021-09-06  5:51   ` Haiyue Wang
  2021-09-06  5:51   ` [dpdk-dev] [PATCH v2 4/4] common/iavf: update the driver version Haiyue Wang
  3 siblings, 0 replies; 21+ messages in thread
From: Haiyue Wang @ 2021-09-06  5:51 UTC (permalink / raw)
  To: dev; +Cc: Haiyue Wang, Qi Zhang, Jingjing Wu, Beilei Xing

The VIRTCHNL_OP_QUERY_FDIR_FILTER opcode is not used, so remove it.

Signed-off-by: Haiyue Wang <haiyue.wang@intel.com>
Acked-by: Qi Zhang <qi.z.zhang@intel.com>
---
 drivers/common/iavf/virtchnl.h | 38 ----------------------------------
 1 file changed, 38 deletions(-)

diff --git a/drivers/common/iavf/virtchnl.h b/drivers/common/iavf/virtchnl.h
index c56c668cff..83f51d889f 100644
--- a/drivers/common/iavf/virtchnl.h
+++ b/drivers/common/iavf/virtchnl.h
@@ -146,7 +146,6 @@ enum virtchnl_ops {
 	VIRTCHNL_OP_DEL_RSS_CFG = 46,
 	VIRTCHNL_OP_ADD_FDIR_FILTER = 47,
 	VIRTCHNL_OP_DEL_FDIR_FILTER = 48,
-	VIRTCHNL_OP_QUERY_FDIR_FILTER = 49,
 	VIRTCHNL_OP_GET_MAX_RSS_QREGION = 50,
 	VIRTCHNL_OP_GET_OFFLOAD_VLAN_V2_CAPS = 51,
 	VIRTCHNL_OP_ADD_VLAN_V2 = 52,
@@ -244,8 +243,6 @@ static inline const char *virtchnl_op_str(enum virtchnl_ops v_opcode)
 		return "VIRTCHNL_OP_ADD_FDIR_FILTER";
 	case VIRTCHNL_OP_DEL_FDIR_FILTER:
 		return "VIRTCHNL_OP_DEL_FDIR_FILTER";
-	case VIRTCHNL_OP_QUERY_FDIR_FILTER:
-		return "VIRTCHNL_OP_QUERY_FDIR_FILTER";
 	case VIRTCHNL_OP_GET_MAX_RSS_QREGION:
 		return "VIRTCHNL_OP_GET_MAX_RSS_QREGION";
 	case VIRTCHNL_OP_ENABLE_QUEUES_V2:
@@ -1733,20 +1730,6 @@ struct virtchnl_fdir_rule {
 
 VIRTCHNL_CHECK_STRUCT_LEN(2604, virtchnl_fdir_rule);
 
-/* query information to retrieve fdir rule counters.
- * PF will fill out this structure to reset counter.
- */
-struct virtchnl_fdir_query_info {
-	u32 match_packets_valid:1;
-	u32 match_bytes_valid:1;
-	u32 reserved:30;  /* Reserved, must be zero. */
-	u32 pad;
-	u64 matched_packets; /* Number of packets for this rule. */
-	u64 matched_bytes;   /* Number of bytes through this rule. */
-};
-
-VIRTCHNL_CHECK_STRUCT_LEN(24, virtchnl_fdir_query_info);
-
 /* Status returned to VF after VF requests FDIR commands
  * VIRTCHNL_FDIR_SUCCESS
  * VF FDIR related request is successfully done by PF
@@ -1879,24 +1862,6 @@ struct virtchnl_queue_tc_mapping {
 
 VIRTCHNL_CHECK_STRUCT_LEN(12, virtchnl_queue_tc_mapping);
 
-/* VIRTCHNL_OP_QUERY_FDIR_FILTER
- * VF sends this request to PF by filling out vsi_id,
- * flow_id and reset_counter. PF will return query_info
- * and query_status to VF.
- */
-struct virtchnl_fdir_query {
-	u16 vsi_id;   /* INPUT */
-	u16 pad1[3];
-	u32 flow_id;  /* INPUT */
-	u32 reset_counter:1; /* INPUT */
-	struct virtchnl_fdir_query_info query_info; /* OUTPUT */
-
-	/* see enum virtchnl_fdir_prgm_status; OUTPUT */
-	s32 status;
-	u32 pad2;
-};
-
-VIRTCHNL_CHECK_STRUCT_LEN(48, virtchnl_fdir_query);
 
 /* TX and RX queue types are valid in legacy as well as split queue models.
  * With Split Queue model, 2 additional types are introduced - TX_COMPLETION
@@ -2254,9 +2219,6 @@ virtchnl_vc_validate_vf_msg(struct virtchnl_version_info *ver, u32 v_opcode,
 	case VIRTCHNL_OP_DEL_FDIR_FILTER:
 		valid_len = sizeof(struct virtchnl_fdir_del);
 		break;
-	case VIRTCHNL_OP_QUERY_FDIR_FILTER:
-		valid_len = sizeof(struct virtchnl_fdir_query);
-		break;
 	case VIRTCHNL_OP_GET_QOS_CAPS:
 		break;
 	case VIRTCHNL_OP_CONFIG_QUEUE_TC_MAP:
-- 
2.33.0


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

* [dpdk-dev] [PATCH v2 4/4] common/iavf: update the driver version
  2021-09-06  5:51 ` [dpdk-dev] [PATCH v2 0/4] iavf base code update Haiyue Wang
                     ` (2 preceding siblings ...)
  2021-09-06  5:51   ` [dpdk-dev] [PATCH v2 3/4] common/iavf: remove the FDIR query opcode Haiyue Wang
@ 2021-09-06  5:51   ` Haiyue Wang
  3 siblings, 0 replies; 21+ messages in thread
From: Haiyue Wang @ 2021-09-06  5:51 UTC (permalink / raw)
  To: dev; +Cc: Haiyue Wang, Qi Zhang, Jingjing Wu, Beilei Xing

Update the driver version to trace the change.

Signed-off-by: Haiyue Wang <haiyue.wang@intel.com>
Acked-by: Qi Zhang <qi.z.zhang@intel.com>
---
 drivers/common/iavf/README | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/common/iavf/README b/drivers/common/iavf/README
index 611fdcea94..89bdbc827e 100644
--- a/drivers/common/iavf/README
+++ b/drivers/common/iavf/README
@@ -6,7 +6,7 @@ Intel® IAVF driver
 =================
 
 This directory contains source code of FreeBSD IAVF driver of version
-cid-avf.2021.04.29.tar.gz released by the team which develops
+cid-avf.2021.08.16.tar.gz released by the team which develops
 basic drivers for any IAVF NIC. The directory of base/ contains the
 original source package.
 
-- 
2.33.0


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

* [dpdk-dev] [PATCH v3 0/4] iavf base code update
  2021-08-17  7:16 [dpdk-dev] [PATCH v1 1/4] common/iavf: add QFI fields for GTPU UL and DL Haiyue Wang
                   ` (5 preceding siblings ...)
  2021-09-06  5:51 ` [dpdk-dev] [PATCH v2 0/4] iavf base code update Haiyue Wang
@ 2021-09-07  7:33 ` Haiyue Wang
  2021-09-07  7:33   ` [dpdk-dev] [PATCH v3 1/4] common/iavf: add QFI fields for GTPU UL and DL Haiyue Wang
                     ` (4 more replies)
  6 siblings, 5 replies; 21+ messages in thread
From: Haiyue Wang @ 2021-09-07  7:33 UTC (permalink / raw)
  To: dev; +Cc: Haiyue Wang

v3: adjust the commit title.
v2: update the commit message.

Alvin Zhang (1):
  common/iavf: enable hash calculation based on L4 checksum

Haiyue Wang (2):
  common/iavf: remove the FDIR query opcode
  common/iavf: update the driver version

Junfeng Guo (1):
  common/iavf: add QFI fields for GTPU UL and DL

 drivers/common/iavf/README     |  2 +-
 drivers/common/iavf/virtchnl.h | 46 ++++++----------------------------
 2 files changed, 9 insertions(+), 39 deletions(-)

-- 
2.33.0


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

* [dpdk-dev] [PATCH v3 1/4] common/iavf: add QFI fields for GTPU UL and DL
  2021-09-07  7:33 ` [dpdk-dev] [PATCH v3 0/4] iavf base code update Haiyue Wang
@ 2021-09-07  7:33   ` Haiyue Wang
  2021-09-07  7:33   ` [dpdk-dev] [PATCH v3 2/4] common/iavf: enable hash calculation based on L4 checksum Haiyue Wang
                     ` (3 subsequent siblings)
  4 siblings, 0 replies; 21+ messages in thread
From: Haiyue Wang @ 2021-09-07  7:33 UTC (permalink / raw)
  To: dev; +Cc: Junfeng Guo, Haiyue Wang, Jingjing Wu, Beilei Xing

From: Junfeng Guo <junfeng.guo@intel.com>

The QFI is 6-bit "QoS Flow Identifier" within the GTPU Extension Header.
Add virtchnl fields QFI of GTPU UL/DL for supporting the AVF FDIR.

Signed-off-by: Junfeng Guo <junfeng.guo@intel.com>
Signed-off-by: Haiyue Wang <haiyue.wang@intel.com>
---
 drivers/common/iavf/virtchnl.h | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/drivers/common/iavf/virtchnl.h b/drivers/common/iavf/virtchnl.h
index 1cf0866124..9fa5e3e891 100644
--- a/drivers/common/iavf/virtchnl.h
+++ b/drivers/common/iavf/virtchnl.h
@@ -1642,6 +1642,11 @@ enum virtchnl_proto_hdr_field {
 	/* IPv6 Extension Fragment */
 	VIRTCHNL_PROTO_HDR_IPV6_EH_FRAG_PKID =
 		PROTO_HDR_FIELD_START(VIRTCHNL_PROTO_HDR_IPV6_EH_FRAG),
+	/* GTPU_DWN/UP */
+	VIRTCHNL_PROTO_HDR_GTPU_DWN_QFI =
+		PROTO_HDR_FIELD_START(VIRTCHNL_PROTO_HDR_GTPU_EH_PDU_DWN),
+	VIRTCHNL_PROTO_HDR_GTPU_UP_QFI =
+		PROTO_HDR_FIELD_START(VIRTCHNL_PROTO_HDR_GTPU_EH_PDU_UP),
 };
 
 struct virtchnl_proto_hdr {
-- 
2.33.0


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

* [dpdk-dev] [PATCH v3 2/4] common/iavf: enable hash calculation based on L4 checksum
  2021-09-07  7:33 ` [dpdk-dev] [PATCH v3 0/4] iavf base code update Haiyue Wang
  2021-09-07  7:33   ` [dpdk-dev] [PATCH v3 1/4] common/iavf: add QFI fields for GTPU UL and DL Haiyue Wang
@ 2021-09-07  7:33   ` Haiyue Wang
  2021-09-07  7:33   ` [dpdk-dev] [PATCH v3 3/4] common/iavf: remove the FDIR query opcode Haiyue Wang
                     ` (2 subsequent siblings)
  4 siblings, 0 replies; 21+ messages in thread
From: Haiyue Wang @ 2021-09-07  7:33 UTC (permalink / raw)
  To: dev; +Cc: Alvin Zhang, Haiyue Wang, Jingjing Wu, Beilei Xing

From: Alvin Zhang <alvinx.zhang@intel.com>

Add TCP/UDP/SCTP header checksum field selectors, they can be used in
creating FDIR or RSS rules related to TCP/UDP/SCTP header checksum.

Signed-off-by: Alvin Zhang <alvinx.zhang@intel.com>
Signed-off-by: Haiyue Wang <haiyue.wang@intel.com>
---
 drivers/common/iavf/virtchnl.h | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/drivers/common/iavf/virtchnl.h b/drivers/common/iavf/virtchnl.h
index 9fa5e3e891..c56c668cff 100644
--- a/drivers/common/iavf/virtchnl.h
+++ b/drivers/common/iavf/virtchnl.h
@@ -1598,14 +1598,17 @@ enum virtchnl_proto_hdr_field {
 	VIRTCHNL_PROTO_HDR_TCP_SRC_PORT =
 		PROTO_HDR_FIELD_START(VIRTCHNL_PROTO_HDR_TCP),
 	VIRTCHNL_PROTO_HDR_TCP_DST_PORT,
+	VIRTCHNL_PROTO_HDR_TCP_CHKSUM,
 	/* UDP */
 	VIRTCHNL_PROTO_HDR_UDP_SRC_PORT =
 		PROTO_HDR_FIELD_START(VIRTCHNL_PROTO_HDR_UDP),
 	VIRTCHNL_PROTO_HDR_UDP_DST_PORT,
+	VIRTCHNL_PROTO_HDR_UDP_CHKSUM,
 	/* SCTP */
 	VIRTCHNL_PROTO_HDR_SCTP_SRC_PORT =
 		PROTO_HDR_FIELD_START(VIRTCHNL_PROTO_HDR_SCTP),
 	VIRTCHNL_PROTO_HDR_SCTP_DST_PORT,
+	VIRTCHNL_PROTO_HDR_SCTP_CHKSUM,
 	/* GTPU_IP */
 	VIRTCHNL_PROTO_HDR_GTPU_IP_TEID =
 		PROTO_HDR_FIELD_START(VIRTCHNL_PROTO_HDR_GTPU_IP),
-- 
2.33.0


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

* [dpdk-dev] [PATCH v3 3/4] common/iavf: remove the FDIR query opcode
  2021-09-07  7:33 ` [dpdk-dev] [PATCH v3 0/4] iavf base code update Haiyue Wang
  2021-09-07  7:33   ` [dpdk-dev] [PATCH v3 1/4] common/iavf: add QFI fields for GTPU UL and DL Haiyue Wang
  2021-09-07  7:33   ` [dpdk-dev] [PATCH v3 2/4] common/iavf: enable hash calculation based on L4 checksum Haiyue Wang
@ 2021-09-07  7:33   ` Haiyue Wang
  2021-09-07  7:33   ` [dpdk-dev] [PATCH v3 4/4] common/iavf: update the driver version Haiyue Wang
  2021-09-13  0:46   ` [dpdk-dev] [PATCH v3 0/4] iavf base code update Zhang, Qi Z
  4 siblings, 0 replies; 21+ messages in thread
From: Haiyue Wang @ 2021-09-07  7:33 UTC (permalink / raw)
  To: dev; +Cc: Haiyue Wang, Qi Zhang, Jingjing Wu, Beilei Xing

The VIRTCHNL_OP_QUERY_FDIR_FILTER opcode is not used, so remove it.

Signed-off-by: Haiyue Wang <haiyue.wang@intel.com>
Acked-by: Qi Zhang <qi.z.zhang@intel.com>
---
 drivers/common/iavf/virtchnl.h | 38 ----------------------------------
 1 file changed, 38 deletions(-)

diff --git a/drivers/common/iavf/virtchnl.h b/drivers/common/iavf/virtchnl.h
index c56c668cff..83f51d889f 100644
--- a/drivers/common/iavf/virtchnl.h
+++ b/drivers/common/iavf/virtchnl.h
@@ -146,7 +146,6 @@ enum virtchnl_ops {
 	VIRTCHNL_OP_DEL_RSS_CFG = 46,
 	VIRTCHNL_OP_ADD_FDIR_FILTER = 47,
 	VIRTCHNL_OP_DEL_FDIR_FILTER = 48,
-	VIRTCHNL_OP_QUERY_FDIR_FILTER = 49,
 	VIRTCHNL_OP_GET_MAX_RSS_QREGION = 50,
 	VIRTCHNL_OP_GET_OFFLOAD_VLAN_V2_CAPS = 51,
 	VIRTCHNL_OP_ADD_VLAN_V2 = 52,
@@ -244,8 +243,6 @@ static inline const char *virtchnl_op_str(enum virtchnl_ops v_opcode)
 		return "VIRTCHNL_OP_ADD_FDIR_FILTER";
 	case VIRTCHNL_OP_DEL_FDIR_FILTER:
 		return "VIRTCHNL_OP_DEL_FDIR_FILTER";
-	case VIRTCHNL_OP_QUERY_FDIR_FILTER:
-		return "VIRTCHNL_OP_QUERY_FDIR_FILTER";
 	case VIRTCHNL_OP_GET_MAX_RSS_QREGION:
 		return "VIRTCHNL_OP_GET_MAX_RSS_QREGION";
 	case VIRTCHNL_OP_ENABLE_QUEUES_V2:
@@ -1733,20 +1730,6 @@ struct virtchnl_fdir_rule {
 
 VIRTCHNL_CHECK_STRUCT_LEN(2604, virtchnl_fdir_rule);
 
-/* query information to retrieve fdir rule counters.
- * PF will fill out this structure to reset counter.
- */
-struct virtchnl_fdir_query_info {
-	u32 match_packets_valid:1;
-	u32 match_bytes_valid:1;
-	u32 reserved:30;  /* Reserved, must be zero. */
-	u32 pad;
-	u64 matched_packets; /* Number of packets for this rule. */
-	u64 matched_bytes;   /* Number of bytes through this rule. */
-};
-
-VIRTCHNL_CHECK_STRUCT_LEN(24, virtchnl_fdir_query_info);
-
 /* Status returned to VF after VF requests FDIR commands
  * VIRTCHNL_FDIR_SUCCESS
  * VF FDIR related request is successfully done by PF
@@ -1879,24 +1862,6 @@ struct virtchnl_queue_tc_mapping {
 
 VIRTCHNL_CHECK_STRUCT_LEN(12, virtchnl_queue_tc_mapping);
 
-/* VIRTCHNL_OP_QUERY_FDIR_FILTER
- * VF sends this request to PF by filling out vsi_id,
- * flow_id and reset_counter. PF will return query_info
- * and query_status to VF.
- */
-struct virtchnl_fdir_query {
-	u16 vsi_id;   /* INPUT */
-	u16 pad1[3];
-	u32 flow_id;  /* INPUT */
-	u32 reset_counter:1; /* INPUT */
-	struct virtchnl_fdir_query_info query_info; /* OUTPUT */
-
-	/* see enum virtchnl_fdir_prgm_status; OUTPUT */
-	s32 status;
-	u32 pad2;
-};
-
-VIRTCHNL_CHECK_STRUCT_LEN(48, virtchnl_fdir_query);
 
 /* TX and RX queue types are valid in legacy as well as split queue models.
  * With Split Queue model, 2 additional types are introduced - TX_COMPLETION
@@ -2254,9 +2219,6 @@ virtchnl_vc_validate_vf_msg(struct virtchnl_version_info *ver, u32 v_opcode,
 	case VIRTCHNL_OP_DEL_FDIR_FILTER:
 		valid_len = sizeof(struct virtchnl_fdir_del);
 		break;
-	case VIRTCHNL_OP_QUERY_FDIR_FILTER:
-		valid_len = sizeof(struct virtchnl_fdir_query);
-		break;
 	case VIRTCHNL_OP_GET_QOS_CAPS:
 		break;
 	case VIRTCHNL_OP_CONFIG_QUEUE_TC_MAP:
-- 
2.33.0


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

* [dpdk-dev] [PATCH v3 4/4] common/iavf: update the driver version
  2021-09-07  7:33 ` [dpdk-dev] [PATCH v3 0/4] iavf base code update Haiyue Wang
                     ` (2 preceding siblings ...)
  2021-09-07  7:33   ` [dpdk-dev] [PATCH v3 3/4] common/iavf: remove the FDIR query opcode Haiyue Wang
@ 2021-09-07  7:33   ` Haiyue Wang
  2021-09-13  0:46   ` [dpdk-dev] [PATCH v3 0/4] iavf base code update Zhang, Qi Z
  4 siblings, 0 replies; 21+ messages in thread
From: Haiyue Wang @ 2021-09-07  7:33 UTC (permalink / raw)
  To: dev; +Cc: Haiyue Wang, Qi Zhang, Jingjing Wu, Beilei Xing

Update the driver version to trace the change.

Signed-off-by: Haiyue Wang <haiyue.wang@intel.com>
Acked-by: Qi Zhang <qi.z.zhang@intel.com>
---
 drivers/common/iavf/README | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/common/iavf/README b/drivers/common/iavf/README
index 611fdcea94..89bdbc827e 100644
--- a/drivers/common/iavf/README
+++ b/drivers/common/iavf/README
@@ -6,7 +6,7 @@ Intel® IAVF driver
 =================
 
 This directory contains source code of FreeBSD IAVF driver of version
-cid-avf.2021.04.29.tar.gz released by the team which develops
+cid-avf.2021.08.16.tar.gz released by the team which develops
 basic drivers for any IAVF NIC. The directory of base/ contains the
 original source package.
 
-- 
2.33.0


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

* Re: [dpdk-dev] [PATCH v3 0/4] iavf base code update
  2021-09-07  7:33 ` [dpdk-dev] [PATCH v3 0/4] iavf base code update Haiyue Wang
                     ` (3 preceding siblings ...)
  2021-09-07  7:33   ` [dpdk-dev] [PATCH v3 4/4] common/iavf: update the driver version Haiyue Wang
@ 2021-09-13  0:46   ` Zhang, Qi Z
  4 siblings, 0 replies; 21+ messages in thread
From: Zhang, Qi Z @ 2021-09-13  0:46 UTC (permalink / raw)
  To: Wang, Haiyue, dev; +Cc: Wang, Haiyue



> -----Original Message-----
> From: dev <dev-bounces@dpdk.org> On Behalf Of Haiyue Wang
> Sent: Tuesday, September 7, 2021 3:33 PM
> To: dev@dpdk.org
> Cc: Wang, Haiyue <haiyue.wang@intel.com>
> Subject: [dpdk-dev] [PATCH v3 0/4] iavf base code update
> 
> v3: adjust the commit title.
> v2: update the commit message.
> 
> Alvin Zhang (1):
>   common/iavf: enable hash calculation based on L4 checksum
> 
> Haiyue Wang (2):
>   common/iavf: remove the FDIR query opcode
>   common/iavf: update the driver version
> 
> Junfeng Guo (1):
>   common/iavf: add QFI fields for GTPU UL and DL
> 
>  drivers/common/iavf/README     |  2 +-
>  drivers/common/iavf/virtchnl.h | 46 ++++++----------------------------
>  2 files changed, 9 insertions(+), 39 deletions(-)
> 
> --
> 2.33.0

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

Applied to dpdk-next-net-intel.

Thanks
Qi



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

end of thread, other threads:[~2021-09-13  0:46 UTC | newest]

Thread overview: 21+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-08-17  7:16 [dpdk-dev] [PATCH v1 1/4] common/iavf: add QFI fields for GTPU UL and DL Haiyue Wang
2021-08-17  7:16 ` [dpdk-dev] [PATCH v1 2/4] common/iavf: add proto hdr field support for L4 checksum Haiyue Wang
2021-08-29 11:38   ` Zhang, Qi Z
2021-09-03 10:43   ` Ferruh Yigit
2021-08-17  7:16 ` [dpdk-dev] [PATCH v1 3/4] common/iavf: remove the FDIR query opcode Haiyue Wang
2021-08-29 11:38   ` Zhang, Qi Z
2021-08-17  7:16 ` [dpdk-dev] [PATCH v1 4/4] common/iavf: update the driver version Haiyue Wang
2021-08-29 11:38   ` Zhang, Qi Z
2021-08-29 11:37 ` [dpdk-dev] [PATCH v1 1/4] common/iavf: add QFI fields for GTPU UL and DL Zhang, Qi Z
2021-09-03 10:41 ` Ferruh Yigit
2021-09-06  5:51 ` [dpdk-dev] [PATCH v2 0/4] iavf base code update Haiyue Wang
2021-09-06  5:51   ` [dpdk-dev] [PATCH v2 1/4] common/iavf: add QFI fields for GTPU UL and DL Haiyue Wang
2021-09-06  5:51   ` [dpdk-dev] [PATCH v2 2/4] common/iavf: add proto hdr field support for L4 checksum Haiyue Wang
2021-09-06  5:51   ` [dpdk-dev] [PATCH v2 3/4] common/iavf: remove the FDIR query opcode Haiyue Wang
2021-09-06  5:51   ` [dpdk-dev] [PATCH v2 4/4] common/iavf: update the driver version Haiyue Wang
2021-09-07  7:33 ` [dpdk-dev] [PATCH v3 0/4] iavf base code update Haiyue Wang
2021-09-07  7:33   ` [dpdk-dev] [PATCH v3 1/4] common/iavf: add QFI fields for GTPU UL and DL Haiyue Wang
2021-09-07  7:33   ` [dpdk-dev] [PATCH v3 2/4] common/iavf: enable hash calculation based on L4 checksum Haiyue Wang
2021-09-07  7:33   ` [dpdk-dev] [PATCH v3 3/4] common/iavf: remove the FDIR query opcode Haiyue Wang
2021-09-07  7:33   ` [dpdk-dev] [PATCH v3 4/4] common/iavf: update the driver version Haiyue Wang
2021-09-13  0:46   ` [dpdk-dev] [PATCH v3 0/4] iavf base code update Zhang, Qi Z

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