DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] doc: add new feature for connection tracking
@ 2021-05-18 12:52 Bing Zhao
  2021-05-18 13:16 ` Ferruh Yigit
  0 siblings, 1 reply; 6+ messages in thread
From: Bing Zhao @ 2021-05-18 12:52 UTC (permalink / raw)
  To: orika, viacheslavo, matan, thomas, ferruh.yigit
  Cc: dev, rasland, andrew.rybchenko

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(+)

diff --git a/doc/guides/nics/features.rst b/doc/guides/nics/features.rst
index f6d30d0af3..a7b8d250a6 100644
--- a/doc/guides/nics/features.rst
+++ b/doc/guides/nics/features.rst
@@ -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
diff --git a/doc/guides/nics/features/default.ini b/doc/guides/nics/features/default.ini
index 8046bd121e..0deb4ef547 100644
--- a/doc/guides/nics/features/default.ini
+++ b/doc/guides/nics/features/default.ini
@@ -66,6 +66,7 @@ Module EEPROM dump   =
 Registers dump       =
 LED                  =
 Multiprocess aware   =
+Connection tracking  =
 FreeBSD              =
 Linux                =
 Windows              =
diff --git a/doc/guides/nics/features/mlx5.ini b/doc/guides/nics/features/mlx5.ini
index ddd131da16..45dbe75d07 100644
--- a/doc/guides/nics/features/mlx5.ini
+++ b/doc/guides/nics/features/mlx5.ini
@@ -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
-- 
2.27.0


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

* Re: [dpdk-dev] [PATCH] doc: add new feature for connection tracking
  2021-05-18 12:52 [dpdk-dev] [PATCH] doc: add new feature for connection tracking Bing Zhao
@ 2021-05-18 13:16 ` Ferruh Yigit
  2021-05-18 13:18   ` Thomas Monjalon
  2021-05-18 13:33   ` Bing Zhao
  0 siblings, 2 replies; 6+ messages in thread
From: Ferruh Yigit @ 2021-05-18 13:16 UTC (permalink / raw)
  To: Bing Zhao, orika, viacheslavo, matan, thomas
  Cc: dev, rasland, andrew.rybchenko

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?

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

* Re: [dpdk-dev] [PATCH] doc: add new feature for connection tracking
  2021-05-18 13:16 ` Ferruh Yigit
@ 2021-05-18 13:18   ` Thomas Monjalon
  2021-05-18 13:49     ` Bing Zhao
  2021-05-18 13:33   ` Bing Zhao
  1 sibling, 1 reply; 6+ messages in thread
From: Thomas Monjalon @ 2021-05-18 13:18 UTC (permalink / raw)
  To: Bing Zhao, Ferruh Yigit
  Cc: orika, viacheslavo, matan, dev, rasland, andrew.rybchenko

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.



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

* Re: [dpdk-dev] [PATCH] doc: add new feature for connection tracking
  2021-05-18 13:16 ` Ferruh Yigit
  2021-05-18 13:18   ` Thomas Monjalon
@ 2021-05-18 13:33   ` Bing Zhao
  2021-05-18 13:44     ` Thomas Monjalon
  1 sibling, 1 reply; 6+ messages in thread
From: Bing Zhao @ 2021-05-18 13:33 UTC (permalink / raw)
  To: Ferruh Yigit, Ori Kam, Slava Ovsiienko, Matan Azrad,
	NBU-Contact-Thomas Monjalon
  Cc: dev, Raslan Darawsheh, andrew.rybchenko

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

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

* Re: [dpdk-dev] [PATCH] doc: add new feature for connection tracking
  2021-05-18 13:33   ` Bing Zhao
@ 2021-05-18 13:44     ` Thomas Monjalon
  0 siblings, 0 replies; 6+ messages in thread
From: Thomas Monjalon @ 2021-05-18 13:44 UTC (permalink / raw)
  To: Bing Zhao
  Cc: Ferruh Yigit, Ori Kam, Slava Ovsiienko, Matan Azrad, dev,
	Raslan Darawsheh, andrew.rybchenko

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/



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

* Re: [dpdk-dev] [PATCH] doc: add new feature for connection tracking
  2021-05-18 13:18   ` Thomas Monjalon
@ 2021-05-18 13:49     ` Bing Zhao
  0 siblings, 0 replies; 6+ messages in thread
From: Bing Zhao @ 2021-05-18 13:49 UTC (permalink / raw)
  To: NBU-Contact-Thomas Monjalon, Ferruh Yigit
  Cc: Ori Kam, Slava Ovsiienko, Matan Azrad, dev, Raslan Darawsheh,
	andrew.rybchenko

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


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

end of thread, other threads:[~2021-05-18 13:49 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-05-18 12:52 [dpdk-dev] [PATCH] doc: add new feature for connection tracking Bing Zhao
2021-05-18 13:16 ` Ferruh Yigit
2021-05-18 13:18   ` Thomas Monjalon
2021-05-18 13:49     ` Bing Zhao
2021-05-18 13:33   ` Bing Zhao
2021-05-18 13:44     ` Thomas Monjalon

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