Skip to contentSkip to conversation

doc: add new feature for connection tracking

PATCH In review Bing Zhao 3 participants 6 messages May 18, 2021

lore
Share & tools
lore

Bing Zhao

author
#0 lore raw

PATCH 1/1 doc: add new feature for connection tracking

The feature support of connection tracking is added in the feature table of networking drivers. The feature overview part is also updated.

---
 doc/guides/nics/features.rst         | 12 ++++++++++++
 doc/guides/nics/features/default.ini |  1 +
 doc/guides/nics/features/mlx5.ini    |  1 +
 3 files changed, 14 insertions(+)
doc/guides/nics/features.rst +12 -0
index f6d30d0af3..a7b8d250a6 100644
@@ -923,6 +923,18 @@ Supports to get Rx/Tx packet burst mode information.
 * **[implements] eth_dev_ops**: ``rx_burst_mode_get``, ``tx_burst_mode_get``.
 * **[related] API**: ``rte_eth_rx_burst_mode_get()``, ``rte_eth_tx_burst_mode_get()``.
 
+.. _nic_features_connection_tracking:
+
+Connection tracking
+-------------------
+
+Supports conntrack offload by HW module.
+
+* **[uses]       rte_flow_item**: ``rte_flow_item_conntrack``.
+* **[uses]       rte_flow_action**: ``rte_flow_action_conntrack``.
+* **[uses]       user config**: ``rte_flow_modify_conntrack``.
+* **[related] API**: ``rte_flow_action_handle_create()``, ``rte_flow_action_handle_destroy()``, ``rte_flow_action_handle_update()``, ``rte_flow_action_handle_query()``.
+
 .. _nic_features_other:
 
 Other dev ops not represented by a Feature
doc/guides/nics/features/default.ini +1 -0
index 8046bd121e..0deb4ef547 100644
@@ -66,6 +66,7 @@ Module EEPROM dump   =
 Registers dump       =
 LED                  =
 Multiprocess aware   =
+Connection tracking  =
 FreeBSD              =
 Linux                =
 Windows              =
doc/guides/nics/features/mlx5.ini +1 -0
index ddd131da16..45dbe75d07 100644
@@ -45,6 +45,7 @@ Stats per queue      = Y
 FW version           = Y
 Module EEPROM dump   = Y
 Multiprocess aware   = Y
+Connection tracking  = Y
 Linux                = Y
 Windows              = P
 ARMv8                = Y
Signature · 1 line
2.27.0
#1 lore raw
Quoted 5 lines · Bing Zhao

The feature support of connection tracking is added in the feature table of networking drivers. The feature overview part is also updated.

Hi Bing,

Thomas is suggestion another table to list flow API item/actions in separate table, can this change can be replaced by it?

#2 lore raw

18/05/2021 15:16, Ferruh Yigit:

Quoted 11 lines · Bing Zhao
Quoted 5 lines

The feature support of connection tracking is added in the feature table of networking drivers. The feature overview part is also updated.

Hi Bing,

Thomas is suggestion another table to list flow API item/actions in separate table, can this change can be replaced by it?

Correct, I am adding conntrack as part of rte_flow features table. Please don't add it in the global table.

author
#3 lore raw

Thanks a lot to you all.

Quoted 32 lines

-----Original Message-----
From: Thomas Monjalon <[email protected]>
Sent: Tuesday, May 18, 2021 9:19 PM
To: Bing Zhao <[email protected]>; Ferruh Yigit <[email protected]>
Cc: Ori Kam <[email protected]>; Slava Ovsiienko <[email protected]>; Matan Azrad <[email protected]>; [email protected]; Raslan Darawsheh <[email protected]>; [email protected]
Subject: Re: [PATCH] doc: add new feature for connection tracking

External email: Use caution opening links or attachments

18/05/2021 15:16, Ferruh Yigit:

Quoted 2 lines · Bing Zhao
Quoted 1 line

The feature support of connection tracking is added in the

feature

Quoted 8 lines
Quoted 4 lines

table of networking drivers. The feature overview part is also updated.

Hi Bing,

Thomas is suggestion another table to list flow API item/actions

in

Quoted 1 line

separate table, can this change can be replaced by it?

Correct, I am adding conntrack as part of rte_flow features table. Please don't add it in the global table.

author
#4 lore raw
In reply to Ferruh Yigit

Hi Ferruh,

Quoted 18 lines

-----Original Message-----
From: Ferruh Yigit <[email protected]>
Sent: Tuesday, May 18, 2021 9:16 PM
To: Bing Zhao <[email protected]>; Ori Kam <[email protected]>; Slava Ovsiienko <[email protected]>; Matan Azrad <[email protected]>; NBU-Contact-Thomas Monjalon <[email protected]>
Cc: [email protected]; Raslan Darawsheh <[email protected]>; [email protected]
Subject: Re: [PATCH] doc: add new feature for connection tracking

External email: Use caution opening links or attachments

Quoted 4 lines · Bing Zhao

The feature support of connection tracking is added in the feature table of networking drivers. The feature overview part is also updated.

Quoted 6 lines
Quoted 1 line

Hi Bing,

Thomas is suggestion another table to list flow API item/actions in separate table, can this change can be replaced by it?

It is OK for me. But I might need the comments from Thomas, Ori and ETHDEV maintainers since you all are much more familiar with the documents and their scopes.

@Thomas, is it OK?

Thanks

#5 lore raw

18/05/2021 15:33, Bing Zhao:

Quoted 8 lines

From: Ferruh Yigit <[email protected]>

Quoted 7 lines · Bing Zhao
Quoted 5 lines

The feature support of connection tracking is added in the feature table of networking drivers. The feature overview part is also updated.

Quoted 8 lines
Quoted 4 lines

Hi Bing,

Thomas is suggestion another table to list flow API item/actions in separate table, can this change can be replaced by it?

It is OK for me. But I might need the comments from Thomas, Ori and ETHDEV maintainers since you all are much more familiar with the documents and their scopes.

@Thomas, is it OK?

Yes, I mark this patch as rejected.

See how it is added in the new table:
https://patches.dpdk.org/project/dpdk/patch/[email protected]/