From: Andrew Rybchenko <andrew.rybchenko@oktetlabs.ru>
To: Michael Baum <michaelba@nvidia.com>, dev@dpdk.org
Cc: Matan Azrad <matan@nvidia.com>,
Raslan Darawsheh <rasland@nvidia.com>, Ori Kam <orika@nvidia.com>
Subject: Re: [PATCH v3 2/3] ethdev: add queue-based API to report aged flow rules
Date: Wed, 26 Oct 2022 22:15:43 +0300 [thread overview]
Message-ID: <e7bb114c-8f62-7347-5fd8-23bb0b2a5f68@oktetlabs.ru> (raw)
In-Reply-To: <20221019144904.2543586-3-michaelba@nvidia.com>
On 10/19/22 17:49, Michael Baum wrote:
> When application use queue-based flow rule management and operate the
> same flow rule on the same queue, e.g create/destroy/query, API of
> querying aged flow rules should also have queue id parameter just like
> other queue-based flow APIs.
>
> By this way, PMD can work in more optimized way since resources are
> isolated by queue and needn't synchronize.
>
> If application do use queue-based flow management but configure port
> without RTE_FLOW_PORT_FLAG_STRICT_QUEUE, which means application operate
> a given flow rule on different queues, the queue id parameter will
> be ignored.
>
> Signed-off-by: Michael Baum <michaelba@nvidia.com>
> Acked-by: Ori Kam <orika@nvidia.com>
Few minor notes below, other than that
Acked-by: Andrew Rybchenko <andrew.rybchenko@oktetlabs.ru>
[snip]
> diff --git a/doc/guides/testpmd_app_ug/testpmd_funcs.rst b/doc/guides/testpmd_app_ug/testpmd_funcs.rst
> index a8b99c8c19..8e21b2a5b7 100644
> --- a/doc/guides/testpmd_app_ug/testpmd_funcs.rst
> +++ b/doc/guides/testpmd_app_ug/testpmd_funcs.rst
> @@ -2894,9 +2894,10 @@ following sections.
> [meters_number {number}] [flags {number}]
>
> - Create a pattern template::
> +
unrelated style fixes, should be in a separate patch
> flow pattern_template {port_id} create [pattern_template_id {id}]
> [relaxed {boolean}] [ingress] [egress] [transfer]
> - template {item} [/ {item} [...]] / end
> + template {item} [/ {item} [...]] / end
unrelated style fixes
>
> - Destroy a pattern template::
>
> @@ -2995,6 +2996,10 @@ following sections.
>
> flow aged {port_id} [destroy]
>
> +- Enqueue list and destroy aged flow rules::
> +
> + flow queue {port_id} aged {queue_id} [destroy]
> +
> - Tunnel offload - create a tunnel stub::
>
> flow tunnel create {port_id} type {tunnel_type}
> @@ -4236,7 +4241,7 @@ Disabling isolated mode::
> testpmd>
>
> Dumping HW internal information
> -~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> +~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
unrelated changes
>
> ``flow dump`` dumps the hardware's internal representation information of
> all flows. It is bound to ``rte_flow_dev_dump()``::
> @@ -4252,10 +4257,10 @@ Otherwise, it will complain error occurred::
> Caught error type [...] ([...]): [...]
>
> Listing and destroying aged flow rules
> -~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> +~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>
> ``flow aged`` simply lists aged flow rules be get from api ``rte_flow_get_aged_flows``,
> -and ``destroy`` parameter can be used to destroy those flow rules in PMD.
> +and ``destroy`` parameter can be used to destroy those flow rules in PMD::
unrelated style fixes
>
> flow aged {port_id} [destroy]
>
> @@ -4290,7 +4295,7 @@ will be ID 3, ID 1, ID 0::
> 1 0 0 i--
> 0 0 0 i--
>
> -If attach ``destroy`` parameter, the command will destroy all the list aged flow rules.
> +If attach ``destroy`` parameter, the command will destroy all the list aged flow rules::
unrelated style fixes
[snip]
next prev parent reply other threads:[~2022-10-26 19:15 UTC|newest]
Thread overview: 32+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-09-21 14:54 [PATCH 0/3] ethdev: AGE action preparation Michael Baum
2022-09-21 14:54 ` [PATCH 1/3] ethdev: add strict queue to pre-configuration flow hints Michael Baum
2022-09-29 12:04 ` Ori Kam
2022-09-21 14:54 ` [PATCH 2/3] ethdev: add queue-based API to report aged flow rules Michael Baum
2022-09-29 12:04 ` Ori Kam
2022-09-21 14:54 ` [PATCH 3/3] ethdev: add structure for indirect AGE update Michael Baum
2022-09-29 12:39 ` Ori Kam
2022-10-03 8:03 ` Andrew Rybchenko
2022-10-03 8:30 ` Ori Kam
2022-10-03 13:17 ` Andrew Rybchenko
2022-10-03 15:13 ` Ori Kam
2022-10-04 6:36 ` Andrew Rybchenko
2022-10-19 13:09 ` Michael Baum
2022-09-29 8:40 ` [PATCH 0/3] ethdev: AGE action preparation Andrew Rybchenko
2022-10-19 13:12 ` [PATCH v2 " Michael Baum
2022-10-19 13:12 ` [PATCH v2 1/3] ethdev: add strict queue to pre-configuration flow hints Michael Baum
2022-10-19 13:12 ` [PATCH v2 2/3] ethdev: add queue-based API to report aged flow rules Michael Baum
2022-10-19 13:12 ` [PATCH v2 3/3] ethdev: add structure for indirect AGE update Michael Baum
2022-10-19 14:49 ` [PATCH v3 0/3] ethdev: AGE action preparation Michael Baum
2022-10-19 14:49 ` [PATCH v3 1/3] ethdev: add strict queue to pre-configuration flow hints Michael Baum
2022-10-26 19:10 ` Andrew Rybchenko
2022-10-19 14:49 ` [PATCH v3 2/3] ethdev: add queue-based API to report aged flow rules Michael Baum
2022-10-26 19:15 ` Andrew Rybchenko [this message]
2022-10-26 21:17 ` Michael Baum
2022-10-19 14:49 ` [PATCH v3 3/3] ethdev: add structure for indirect AGE update Michael Baum
2022-10-26 19:18 ` Andrew Rybchenko
2022-10-26 21:19 ` Michael Baum
2022-10-26 21:49 ` [PATCH v4 0/3] ethdev: AGE action preparation Michael Baum
2022-10-26 21:49 ` [PATCH v4 1/3] ethdev: add strict queue to pre-configuration flow hints Michael Baum
2022-10-26 21:49 ` [PATCH v4 2/3] ethdev: add queue-based API to report aged flow rules Michael Baum
2022-10-26 21:49 ` [PATCH v4 3/3] ethdev: add structure for indirect AGE update Michael Baum
2022-10-28 10:56 ` [PATCH v4 0/3] ethdev: AGE action preparation Andrew Rybchenko
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=e7bb114c-8f62-7347-5fd8-23bb0b2a5f68@oktetlabs.ru \
--to=andrew.rybchenko@oktetlabs.ru \
--cc=dev@dpdk.org \
--cc=matan@nvidia.com \
--cc=michaelba@nvidia.com \
--cc=orika@nvidia.com \
--cc=rasland@nvidia.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).