* [dpdk-dev] [PATCH] doc: remove repeated repeated words
@ 2021-11-05 21:10 Thomas Monjalon
2021-11-05 21:18 ` Thomas Monjalon
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: Thomas Monjalon @ 2021-11-05 21:10 UTC (permalink / raw)
To: dev
Cc: bernard.iremonger, jerinj, david.hunt, roy.fan.zhang,
remy.horton, stable, Kiran Kumar K, Nithin Dabilpuram,
Radu Nicolau, Akhil Goyal, John McNamara, Pavan Nikhilesh,
Sergio Gonzalez Monroy, Maryam Tahhan, John J Browne
The script devtools/check-dup-words.sh can detect some words
which are duplicated.
Fixes: fdec9301f52d ("doc: add flow classify guides")
Cc: bernard.iremonger@intel.com
Fixes: 4dc6d8e63c16 ("doc: add graph library guide")
Cc: jerinj@marvell.com
Fixes: 30d3aa861db5 ("doc: rework VM power manager user guide")
Cc: david.hunt@intel.com
Fixes: 0d547ed03717 ("examples/ipsec-secgw: support configuration file")
Cc: roy.fan.zhang@intel.com
Fixes: e64833f2273a ("examples/l2fwd-keepalive: add sample application")
Cc: remy.horton@intel.com
Cc: stable@dpdk.org
Signed-off-by: Thomas Monjalon <thomas@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.
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.
diff --git a/doc/guides/prog_guide/graph_lib.rst b/doc/guides/prog_guide/graph_lib.rst
index fcff9c4286..1cfdc86433 100644
--- a/doc/guides/prog_guide/graph_lib.rst
+++ b/doc/guides/prog_guide/graph_lib.rst
@@ -220,7 +220,7 @@ the user needs to update the context of the node hence access to
``struct rte_node *`` memory.
``rte_graph_foreach_node()``, ``rte_graph_node_get()``,
-``rte_graph_node_get_by_name()`` APIs can be used to to get the
+``rte_graph_node_get_by_name()`` APIs can be used to get the
``struct rte_node*``. ``rte_graph_foreach_node()`` iterator function works on
``struct rte_graph *`` fast-path graph object while others works on graph ID or name.
diff --git a/doc/guides/sample_app_ug/ipsec_secgw.rst b/doc/guides/sample_app_ug/ipsec_secgw.rst
index 2304141ce8..c53ee7c386 100644
--- a/doc/guides/sample_app_ug/ipsec_secgw.rst
+++ b/doc/guides/sample_app_ug/ipsec_secgw.rst
@@ -443,7 +443,7 @@ where each options means:
* *protect <SA_idx>*: the specified traffic is protected by SA rule
with id SA_idx
- * *bypass*: the specified traffic traffic is bypassed
+ * *bypass*: the specified traffic is bypassed
* *discard*: the specified traffic is discarded
``<priority>``
diff --git a/doc/guides/sample_app_ug/keep_alive.rst b/doc/guides/sample_app_ug/keep_alive.rst
index b6d75c8a80..a907ff36a1 100644
--- a/doc/guides/sample_app_ug/keep_alive.rst
+++ b/doc/guides/sample_app_ug/keep_alive.rst
@@ -78,7 +78,7 @@ options.
Explanation
-----------
-The following sections provide some explanation of the The
+The following sections provide some explanation of the
Keep-Alive/'Liveliness' conceptual scheme. As mentioned in the
overview section, the initialization and run-time paths are very
similar to those of the :doc:`l2_forward_real_virtual`.
diff --git a/doc/guides/sample_app_ug/vm_power_management.rst b/doc/guides/sample_app_ug/vm_power_management.rst
index 35afdac63f..7160b6a63a 100644
--- a/doc/guides/sample_app_ug/vm_power_management.rst
+++ b/doc/guides/sample_app_ug/vm_power_management.rst
@@ -236,7 +236,7 @@ Compiling and Running the Host Application
Compiling the Host Application
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
-For information on compiling the DPDK and sample applications, see
+For information on compiling the DPDK and sample applications,
see :doc:`compiling`.
The application is located in the ``vm_power_manager`` subdirectory.
--
2.33.0
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [dpdk-dev] [PATCH] doc: remove repeated repeated words
2021-11-05 21:10 [dpdk-dev] [PATCH] doc: remove repeated repeated words Thomas Monjalon
@ 2021-11-05 21:18 ` Thomas Monjalon
2021-11-06 9:01 ` David Marchand
2021-11-24 16:21 ` David Marchand
2 siblings, 0 replies; 6+ messages in thread
From: Thomas Monjalon @ 2021-11-05 21:18 UTC (permalink / raw)
To: dev
Cc: bernard.iremonger, jerinj, david.hunt, roy.fan.zhang,
remy.horton, stable, Kiran Kumar K, Nithin Dabilpuram,
Radu Nicolau, Akhil Goyal, John McNamara, Pavan Nikhilesh,
Sergio Gonzalez Monroy, Maryam Tahhan, John J Browne,
david.marchand
05/11/2021 22:10, Thomas Monjalon:
> The script devtools/check-dup-words.sh can detect some words
> which are duplicated.
After double check, it seems this script is not in the repository,
probably because of the false positives.
I will change this sentence in the v2.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [dpdk-dev] [PATCH] doc: remove repeated repeated words
2021-11-05 21:10 [dpdk-dev] [PATCH] doc: remove repeated repeated words Thomas Monjalon
2021-11-05 21:18 ` Thomas Monjalon
@ 2021-11-06 9:01 ` David Marchand
2021-11-06 9:53 ` Thomas Monjalon
2021-11-24 16:21 ` David Marchand
2 siblings, 1 reply; 6+ messages in thread
From: David Marchand @ 2021-11-06 9:01 UTC (permalink / raw)
To: Thomas Monjalon
Cc: dev, Iremonger, Bernard, Jerin Jacob Kollanukkaran, David Hunt,
Fan Zhang, remy.horton, dpdk stable, Kiran Kumar K,
Nithin Dabilpuram, Radu Nicolau, Akhil Goyal, John McNamara,
Pavan Nikhilesh, Sergio Gonzalez Monroy, Maryam Tahhan,
John J Browne
On Fri, Nov 5, 2021 at 10:11 PM Thomas Monjalon <thomas@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.iremonger@intel.com
> Fixes: 4dc6d8e63c16 ("doc: add graph library guide")
> Cc: jerinj@marvell.com
> Fixes: 30d3aa861db5 ("doc: rework VM power manager user guide")
> Cc: david.hunt@intel.com
> Fixes: 0d547ed03717 ("examples/ipsec-secgw: support configuration file")
> Cc: roy.fan.zhang@intel.com
> Fixes: e64833f2273a ("examples/l2fwd-keepalive: add sample application")
> Cc: remy.horton@intel.com
> Cc: stable@dpdk.org
>
> Signed-off-by: Thomas Monjalon <thomas@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
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [dpdk-dev] [PATCH] doc: remove repeated repeated words
2021-11-06 9:01 ` David Marchand
@ 2021-11-06 9:53 ` Thomas Monjalon
2021-11-24 16:05 ` David Marchand
0 siblings, 1 reply; 6+ messages in thread
From: Thomas Monjalon @ 2021-11-06 9:53 UTC (permalink / raw)
To: David Marchand
Cc: dev, Iremonger, Bernard, Jerin Jacob Kollanukkaran, David Hunt,
Fan Zhang, remy.horton, dpdk stable, Kiran Kumar K,
Nithin Dabilpuram, Radu Nicolau, Akhil Goyal, John McNamara,
Pavan Nikhilesh, Sergio Gonzalez Monroy, Maryam Tahhan,
John J Browne
06/11/2021 10:01, David Marchand:
> On Fri, Nov 5, 2021 at 10:11 PM Thomas Monjalon <thomas@monjalon.net> wrote:
> > --- 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.
I don't understand the meaning of the capital letter.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [dpdk-dev] [PATCH] doc: remove repeated repeated words
2021-11-06 9:53 ` Thomas Monjalon
@ 2021-11-24 16:05 ` David Marchand
0 siblings, 0 replies; 6+ messages in thread
From: David Marchand @ 2021-11-24 16:05 UTC (permalink / raw)
To: Thomas Monjalon
Cc: dev, Iremonger, Bernard, Jerin Jacob Kollanukkaran, David Hunt,
Fan Zhang, remy.horton, dpdk stable, Kiran Kumar K,
Nithin Dabilpuram, Radu Nicolau, Akhil Goyal, John McNamara,
Pavan Nikhilesh, Sergio Gonzalez Monroy, Maryam Tahhan,
John J Browne
On Sat, Nov 6, 2021 at 6:54 PM Thomas Monjalon <thomas@monjalon.net> wrote:
>
> 06/11/2021 10:01, David Marchand:
> > On Fri, Nov 5, 2021 at 10:11 PM Thomas Monjalon <thomas@monjalon.net> wrote:
> > > --- 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.
>
> I don't understand the meaning of the capital letter.
I don't get the intention either.
I will go with your patch.
--
David Marchand
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [dpdk-dev] [PATCH] doc: remove repeated repeated words
2021-11-05 21:10 [dpdk-dev] [PATCH] doc: remove repeated repeated words Thomas Monjalon
2021-11-05 21:18 ` Thomas Monjalon
2021-11-06 9:01 ` David Marchand
@ 2021-11-24 16:21 ` David Marchand
2 siblings, 0 replies; 6+ messages in thread
From: David Marchand @ 2021-11-24 16:21 UTC (permalink / raw)
To: Thomas Monjalon
Cc: dev, Iremonger, Bernard, Jerin Jacob Kollanukkaran, David Hunt,
Fan Zhang, dpdk stable, Kiran Kumar K, Nithin Dabilpuram,
Radu Nicolau, Akhil Goyal, John McNamara, Pavan Nikhilesh,
Sergio Gonzalez Monroy, Maryam Tahhan, John J Browne
On Fri, Nov 5, 2021 at 10:11 PM Thomas Monjalon <thomas@monjalon.net> wrote:
>
> The script devtools/check-dup-words.sh can detect some words
> which are duplicated.
Updated as:
Some duplicate words were detected with a script.
>
> Fixes: fdec9301f52d ("doc: add flow classify guides")
> Fixes: 4dc6d8e63c16 ("doc: add graph library guide")
> Fixes: 30d3aa861db5 ("doc: rework VM power manager user guide")
> Fixes: 0d547ed03717 ("examples/ipsec-secgw: support configuration file")
> Fixes: e64833f2273a ("examples/l2fwd-keepalive: add sample application")
> Cc: stable@dpdk.org
>
> Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
Acked-by: David Marchand <david.marchand@redhat.com>
Applied, thanks.
--
David Marchand
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2021-11-24 16:22 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-11-05 21:10 [dpdk-dev] [PATCH] doc: remove repeated repeated words Thomas Monjalon
2021-11-05 21:18 ` Thomas Monjalon
2021-11-06 9:01 ` David Marchand
2021-11-06 9:53 ` Thomas Monjalon
2021-11-24 16:05 ` David Marchand
2021-11-24 16:21 ` David Marchand
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).