doc: add new feature for connection tracking
Checks
Commit Message
The feature support of connection tracking is added in the feature
table of networking drivers. The feature overview part is also
updated.
Signed-off-by: Bing Zhao <bingz@nvidia.com>
---
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(+)
Comments
On 5/18/2021 1:52 PM, Bing Zhao wrote:
> The feature support of connection tracking is added in the feature
> table of networking drivers. The feature overview part is also
> updated.
>
> Signed-off-by: Bing Zhao <bingz@nvidia.com>
Hi Bing,
Thomas is suggestion another table to list flow API item/actions in separate
table, can this change can be replaced by it?
18/05/2021 15:16, Ferruh Yigit:
> On 5/18/2021 1:52 PM, Bing Zhao wrote:
> > The feature support of connection tracking is added in the feature
> > table of networking drivers. The feature overview part is also
> > updated.
> >
> > Signed-off-by: Bing Zhao <bingz@nvidia.com>
>
> 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.
Hi Ferruh,
> -----Original Message-----
> From: Ferruh Yigit <ferruh.yigit@intel.com>
> Sent: Tuesday, May 18, 2021 9:16 PM
> To: Bing Zhao <bingz@nvidia.com>; Ori Kam <orika@nvidia.com>; Slava
> Ovsiienko <viacheslavo@nvidia.com>; Matan Azrad <matan@nvidia.com>;
> NBU-Contact-Thomas Monjalon <thomas@monjalon.net>
> Cc: dev@dpdk.org; Raslan Darawsheh <rasland@nvidia.com>;
> andrew.rybchenko@oktetlabs.ru
> Subject: Re: [PATCH] doc: add new feature for connection tracking
>
> External email: Use caution opening links or attachments
>
>
> On 5/18/2021 1:52 PM, Bing Zhao wrote:
> > The feature support of connection tracking is added in the feature
> > table of networking drivers. The feature overview part is also
> > updated.
> >
> > Signed-off-by: Bing Zhao <bingz@nvidia.com>
>
> 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
18/05/2021 15:33, Bing Zhao:
> From: Ferruh Yigit <ferruh.yigit@intel.com>
> > On 5/18/2021 1:52 PM, Bing Zhao wrote:
> > > The feature support of connection tracking is added in the feature
> > > table of networking drivers. The feature overview part is also
> > > updated.
> > >
> > > Signed-off-by: Bing Zhao <bingz@nvidia.com>
> >
> > 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/20210518132844.3779728-3-thomas@monjalon.net/
Thanks a lot to you all.
> -----Original Message-----
> From: Thomas Monjalon <thomas@monjalon.net>
> Sent: Tuesday, May 18, 2021 9:19 PM
> To: Bing Zhao <bingz@nvidia.com>; Ferruh Yigit
> <ferruh.yigit@intel.com>
> Cc: Ori Kam <orika@nvidia.com>; Slava Ovsiienko
> <viacheslavo@nvidia.com>; Matan Azrad <matan@nvidia.com>;
> dev@dpdk.org; Raslan Darawsheh <rasland@nvidia.com>;
> andrew.rybchenko@oktetlabs.ru
> 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:
> > On 5/18/2021 1:52 PM, Bing Zhao wrote:
> > > The feature support of connection tracking is added in the
> feature
> > > table of networking drivers. The feature overview part is also
> > > updated.
> > >
> > > Signed-off-by: Bing Zhao <bingz@nvidia.com>
> >
> > 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.
>
@@ -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
@@ -66,6 +66,7 @@ Module EEPROM dump =
Registers dump =
LED =
Multiprocess aware =
+Connection tracking =
FreeBSD =
Linux =
Windows =
@@ -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