On Fri, Nov 5, 2021 at 10:11 PM Thomas Monjalon <tho...@monjalon.net> wrote:
>
> The script devtools/check-dup-words.sh can detect some words
> which are duplicated.
>
Err, that's one of our private checks.
Good to pass it every once in a while.
But checkpatch.pl itself is supposed to do a similar job for new changes.
> Fixes: fdec9301f52d ("doc: add flow classify guides")
> Cc: bernard.iremon...@intel.com
> Fixes: 4dc6d8e63c16 ("doc: add graph library guide")
> Cc: jer...@marvell.com
> Fixes: 30d3aa861db5 ("doc: rework VM power manager user guide")
> Cc: david.h...@intel.com
> Fixes: 0d547ed03717 ("examples/ipsec-secgw: support configuration file")
> Cc: roy.fan.zh...@intel.com
> Fixes: e64833f2273a ("examples/l2fwd-keepalive: add sample application")
> Cc: remy.hor...@intel.com
> Cc: sta...@dpdk.org
>
> Signed-off-by: Thomas Monjalon <tho...@monjalon.net>
> ---
> doc/guides/prog_guide/flow_classify_lib.rst | 2 +-
> doc/guides/prog_guide/graph_lib.rst | 2 +-
> doc/guides/sample_app_ug/ipsec_secgw.rst | 2 +-
> doc/guides/sample_app_ug/keep_alive.rst | 2 +-
> doc/guides/sample_app_ug/vm_power_management.rst | 2 +-
> 5 files changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/doc/guides/prog_guide/flow_classify_lib.rst
> b/doc/guides/prog_guide/flow_classify_lib.rst
> index f0ed5a1a04..7dae0bc8c6 100644
> --- a/doc/guides/prog_guide/flow_classify_lib.rst
> +++ b/doc/guides/prog_guide/flow_classify_lib.rst
> @@ -366,7 +366,7 @@ Packet Matching
> ~~~~~~~~~~~~~~~
>
> The ``rte_flow_classifier_query`` API is used to find packets which match a
> -given flow Flow rule in the table.
> +given flow rule in the table.
I think the "Flow rule" form was intended.
> This API calls the flow_classify_run internal function which calls the
> ``table.ops.f_lookup`` API to see if any packets in a burst match any
> of the Flow rules in the table.
^^^^
See here.
Otherwise patch lgtm.
--
David Marchand
